From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnaldo Carvalho de Melo Subject: Re: [PATCH 1/3] perf s390: generate system call table from asm/unistd.h Date: Mon, 11 Dec 2017 11:19:24 -0300 Message-ID: <20171211141924.GA3958@kernel.org> References: <1512635281-20733-1-git-send-email-brueckner@linux.vnet.ibm.com> <1512635281-20733-2-git-send-email-brueckner@linux.vnet.ibm.com> <20171207153040.GB3173@kernel.org> <20171207173731.GF3173@kernel.org> <20171211090639.GC4369@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail.kernel.org ([198.145.29.99]:51380 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752880AbdLKOT1 (ORCPT ); Mon, 11 Dec 2017 09:19:27 -0500 Content-Disposition: inline In-Reply-To: <20171211090639.GC4369@linux.vnet.ibm.com> Sender: linux-perf-users-owner@vger.kernel.org List-ID: To: Hendrik Brueckner Cc: Arnaldo Carvalho de Melo , Thomas-Mich Richter , Michael Petlan , Jiri Olsa , linux-perf-users@vger.kernel.org, linux-s390@vger.kernel.org Em Mon, Dec 11, 2017 at 10:06:39AM +0100, Hendrik Brueckner escreveu: > 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! - Arnaldo