tools/perf: Fix man page formatting for perf lock

[PATCH] tools/perf: Fix man page formatting for perf lock

Posted by Nick Forrington 2 years, 1 month ago

This makes "CONTENTION" a top level section (rather than a subsection of
"INFO").

Fixes: 79079f21f50a ("perf lock: Add -k and -F options to 'contention' subcommand")
Signed-off-by: Nick Forrington <nick.forrington@arm.com>
---
 tools/perf/Documentation/perf-lock.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/Documentation/perf-lock.txt b/tools/perf/Documentation/perf-lock.txt
index 503abcba1438..f5938d616d75 100644
--- a/tools/perf/Documentation/perf-lock.txt
+++ b/tools/perf/Documentation/perf-lock.txt
@@ -119,7 +119,7 @@ INFO OPTIONS
 
 
 CONTENTION OPTIONS
---------------
+------------------
 
 -k::
 --key=<value>::
-- 
2.42.0

Re: [PATCH] tools/perf: Fix man page formatting for perf lock

Posted by James Clark 2 years, 1 month ago


On 02/11/2023 16:11, Nick Forrington wrote:
> This makes "CONTENTION" a top level section (rather than a subsection of
> "INFO").
> 
> Fixes: 79079f21f50a ("perf lock: Add -k and -F options to 'contention' subcommand")
> Signed-off-by: Nick Forrington <nick.forrington@arm.com>
> ---
>  tools/perf/Documentation/perf-lock.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/Documentation/perf-lock.txt b/tools/perf/Documentation/perf-lock.txt
> index 503abcba1438..f5938d616d75 100644
> --- a/tools/perf/Documentation/perf-lock.txt
> +++ b/tools/perf/Documentation/perf-lock.txt
> @@ -119,7 +119,7 @@ INFO OPTIONS
>  
>  
>  CONTENTION OPTIONS
> ---------------
> +------------------
>  
>  -k::
>  --key=<value>::

Reviewed-by: James Clark <james.clark@arm.com>

Re: [PATCH] tools/perf: Fix man page formatting for perf lock

Posted by Arnaldo Carvalho de Melo 2 years, 1 month ago

Em Tue, Nov 28, 2023 at 11:08:54AM +0000, James Clark escreveu:
> 
> 
> On 02/11/2023 16:11, Nick Forrington wrote:
> > This makes "CONTENTION" a top level section (rather than a subsection of
> > "INFO").
> > 
> > Fixes: 79079f21f50a ("perf lock: Add -k and -F options to 'contention' subcommand")
> > Signed-off-by: Nick Forrington <nick.forrington@arm.com>
> > ---
> >  tools/perf/Documentation/perf-lock.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tools/perf/Documentation/perf-lock.txt b/tools/perf/Documentation/perf-lock.txt
> > index 503abcba1438..f5938d616d75 100644
> > --- a/tools/perf/Documentation/perf-lock.txt
> > +++ b/tools/perf/Documentation/perf-lock.txt
> > @@ -119,7 +119,7 @@ INFO OPTIONS
> >  
> >  
> >  CONTENTION OPTIONS
> > ---------------
> > +------------------
> >  
> >  -k::
> >  --key=<value>::
> 
> Reviewed-by: James Clark <james.clark@arm.com>

Thanks, applied to perf-tools-next.

- Arnaldo

Re: [PATCH] tools/perf: Fix man page formatting for perf lock

Posted by Ian Rogers 2 years, 1 month ago

On Thu, Nov 2, 2023 at 9:12 AM Nick Forrington <nick.forrington@arm.com> wrote:
>
> This makes "CONTENTION" a top level section (rather than a subsection of
> "INFO").
>
> Fixes: 79079f21f50a ("perf lock: Add -k and -F options to 'contention' subcommand")
> Signed-off-by: Nick Forrington <nick.forrington@arm.com>

Tested-by: Ian Rogers <irogers@google.com>

Thanks,
Ian

> ---
>  tools/perf/Documentation/perf-lock.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/Documentation/perf-lock.txt b/tools/perf/Documentation/perf-lock.txt
> index 503abcba1438..f5938d616d75 100644
> --- a/tools/perf/Documentation/perf-lock.txt
> +++ b/tools/perf/Documentation/perf-lock.txt
> @@ -119,7 +119,7 @@ INFO OPTIONS
>
>
>  CONTENTION OPTIONS
> ---------------
> +------------------
>
>  -k::
>  --key=<value>::
> --
> 2.42.0
>