Re: [PATCH] Doc: process: Getting rid of OProfile url enlisting
Randy Dunlap <[email protected]>
| Newsgroups | org.kernel.vger.workflows,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On 8/16/26 1:24 AM, Manuel Ebner wrote: > On Sun, 2026-08-16 at 03:23 +0530, Bhaskar Chowdhury wrote: >> Aren't we done with this damn tool from a kernel perspective?? Thought,the link >> could be removed. > > This would finalize (fix?) this commit 8e9fa2f21151 (oprofiled version > output line removed from the list) > > Maybe add some reasoning (To reduce it's OS jitter or because it's a leftover link) > > And if it's the way to handle translations remove: > Documentation/translations/it_IT/process/changes.rst:- <https://oprofile.sf.net/download/> > Documentation/translations/pt_BR/process/changes.rst:- <https://oprofile.sf.net/download/> > > I have no clue about the technical side, that's why I'm asking: > Can this be removed aswell? > arch/sparc/kernel/nmi.c:atomic_t nmi_active = ATOMIC_INIT(0); /* oprofile uses this */ It looks like nmi_active is used in that source file regardless of oprofile. I'd say No. Other than the comment itself, which can be deleted if oprofile support is going (gone) away. >> Signed-off-by: Bhaskar Chowdhury <[email protected]> >> --- >> Documentation/process/changes.rst | 4 ---- >> 1 file changed, 4 deletions(-) >> >> diff --git a/Documentation/process/changes.rst b/Documentation/process/changes.rst >> index 0aa232b117b5..336c07190c9b 100644 >> --- a/Documentation/process/changes.rst >> +++ b/Documentation/process/changes.rst >> @@ -556,10 +556,6 @@ Ip-route2 >> >> - <https://www.kernel.org/pub/linux/utils/net/iproute2/> >> >> -OProfile >> --------- >> - >> -- <https://oprofile.sf.net/download/> >> >> Kernel documentation >> ******************** >> -- >> 2.54.0 >> > -- ~Randy