Re: [PATCH 1/3] perf s390: generate system call table from asm/unistd.h

[Hendrik Brueckner <brueckner@linux.vnet.ibm.com>]

Hi Arnaldo,

On Thu, Dec 07, 2017 at 02:37:31PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Thu, Dec 07, 2017 at 12:30:40PM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Thu, Dec 07, 2017 at 09:27:59AM +0100, Hendrik Brueckner escreveu:
> > > This should speed up accessing new system calls introduced with
> > > the kernel rather than waiting for libaudit updates to include
> > > them.
> > 
> > So, this needs to be split in multiple patches, one grabbing the
> > tools/arch/s390/include/uapi/asm/unistd.h copy and adding it to
> > check-headers.sh (cool, you did it! :))
> > 
> > I'll take a stab at splitting it here.
> 
> So, splitted it into three patches and tested the generation, showing
> the steps and results on the second patch commit message:
> 
> [acme@jouet perf]$ git log -3 --oneline
> b91e81b8e967 (HEAD -> perf/core, acme.korg/perf/core) perf trace: Use generated syscall table on s390 too
> 8fe39fe5058d perf s390: Generate system call table from asm/unistd.h
> fdd7d74b2782 tools include s390: Grab a copy of arch/s390/include/uapi/asm/unistd.h
> [acme@jouet perf]$
> 
> And pushed it to my perf/core branch on:
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git
> 
> Please take a look and check that I didn't mess anything up :-)


They look fine.  I have tested them again.  Many thanks for splitting them.
I will try to follow your split guidelines for future commits.

Thanks,
  Hendrik