Messages in this thread | | | From | "Song Bao Hua (Barry Song)" <> | Subject | RE: [PATCH v2 2/2] Documentation: scheduler: ask users to set sched_debug cmdline | Date | Fri, 13 Nov 2020 11:41:57 +0000 |
| |
> -----Original Message----- > From: Valentin Schneider [mailto:valentin.schneider@arm.com] > Sent: Saturday, November 14, 2020 12:34 AM > To: Song Bao Hua (Barry Song) <song.bao.hua@hisilicon.com> > Cc: corbet@lwn.net; linux-doc@vger.kernel.org; linux-kernel@vger.kernel.org; > Linuxarm <linuxarm@huawei.com>; Ingo Molnar <mingo@redhat.com>; Peter > Zijlstra <peterz@infradead.org>; Mel Gorman <mgorman@suse.de>; Adrian > Freund <adrian@freund.io> > Subject: Re: [PATCH v2 2/2] Documentation: scheduler: ask users to set > sched_debug cmdline > > > On 12/11/20 22:31, Barry Song wrote: > > From: Valentin Schneider <valentin.schneider@arm.com> > > > > To print the schedule debug information, users need to set sched_debug > > cmdline or enable it by sysfs entry. > > > > Cc: Ingo Molnar <mingo@redhat.com> > > Cc: Peter Zijlstra <peterz@infradead.org> > > Cc: Mel Gorman <mgorman@suse.de> > > Cc: Adrian Freund <adrian@freund.io> > > Signed-off-by: Valentin Schneider <valentin.schneider@arm.com> > > FYI, forging SoB, or any 'tag' other than Suggested-by / Reported-by is > frowned up. Now I do appreciate your effort to give me credit, but in this > case you should've used Suggested-by.
As this part is actually your code, so I put an --author=" Valentin ..." in git commit so that git can show you are the author. Actually I was not sure if SoB was proper when I made it. My intention was only telling people the code was from you :-)
> > Moreso, this really is a trivial change and doesn't deserve a separate > patch, so I would say just squash that into patch 1, and add > > Reviewed-by: Valentin Schneider <valentin.schneider@arm.com> > > to it :)
Cool, thanks.
> > > Signed-off-by: Barry Song <song.bao.hua@hisilicon.com> > > --- > > Documentation/scheduler/sched-domains.rst | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/scheduler/sched-domains.rst > b/Documentation/scheduler/sched-domains.rst > > index 55983b7e1098..8582fa5e9170 100644 > > --- a/Documentation/scheduler/sched-domains.rst > > +++ b/Documentation/scheduler/sched-domains.rst > > @@ -74,6 +74,8 @@ for a given topology level by creating a > sched_domain_topology_level array and > > calling set_sched_topology() with this array as the parameter. > > > > The sched-domains debugging infrastructure can be enabled by enabling > > -CONFIG_SCHED_DEBUG. This enables an error checking parse of the sched > domains > > -which should catch most possible errors (described above). It also prints out > > -the domain structure in a visual format. > > +CONFIG_SCHED_DEBUG and adding 'sched_debug' to your cmdline. If you > forgot to > > +tweak your cmdline, you can also flip the /sys/kernel/debug/sched_debug > > +knob. This enables an error checking parse of the sched domains which > should > > +catch most possible errors (described above). It also prints out the domain > > +structure in a visual format.
Thanks Barry
| |