From: Hendrik Brueckner <brueckner@linux.vnet.ibm.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Hendrik Brueckner <brueckner@linux.vnet.ibm.com>,
Arnaldo de Melo <acme@redhat.com>,
Thomas-Mich Richter <tmricht@linux.vnet.ibm.com>,
Michael Petlan <mpetlan@redhat.com>,
linux-perf-users@vger.kernel.org, Jiri Olsa <jolsa@redhat.com>
Subject: Re: [PATCH] perf test shell: Fix check open filename arg using 'perf trace'
Date: Thu, 7 Dec 2017 08:39:12 +0100 [thread overview]
Message-ID: <20171207073912.GB4889@linux.vnet.ibm.com> (raw)
In-Reply-To: <20171206163443.GA16781@kernel.org>
On Wed, Dec 06, 2017 at 01:34:43PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Dec 05, 2017 at 04:39:49PM +0100, Hendrik Brueckner escreveu:
> > On Thu, Nov 30, 2017 at 01:59:22PM -0200, Arnaldo de Melo wrote:
> > > So someone needs to implement syscalltbl__strglobmatch_next() for arches
> > > that don't HAVE_SYSCALL_TABLE, which is basically at this stage !x86.
>
> > > Shouldn't be that difficult, but remains to be done.
>
> > Thanks for pointing in that direction. I looked at it and will post a
> > patch to add syscall tables for s390 too.
>
> Will you do it the way I did for x86? That would be the best, as we
> would then reduce the need for audit-libs to be linked with perf.
It looks similar to x86 but at the time being, s390 does not have a
system call table like x86. So the input is different, the output
will follow what x86 provides to perf.
>
> We really need to reduce the number of libraries that perf links to :-\
I have prepared two patches to get rid of the auditlibs dependency for
those arch's that have system call tables. Going to post this today.
Thanks and kind regards,
Hendrik
next prev parent reply other threads:[~2017-12-07 7:39 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-29 17:27 [PATCH] perf test shell: Fix check open filename arg using 'perf trace' Michael Petlan
2017-11-30 10:28 ` Thomas-Mich Richter
2017-11-30 15:56 ` Arnaldo de Melo
2017-11-30 15:59 ` Arnaldo de Melo
2017-12-05 15:39 ` Hendrik Brueckner
2017-12-06 16:34 ` Arnaldo Carvalho de Melo
2017-12-07 7:39 ` Hendrik Brueckner [this message]
2017-12-01 2:33 ` Namhyung Kim
2017-12-01 15:16 ` Arnaldo Carvalho de Melo
2017-12-02 23:21 ` Michael Petlan
2017-12-05 23:18 ` Michael Petlan
2017-12-06 14:28 ` Arnaldo de Melo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171207073912.GB4889@linux.vnet.ibm.com \
--to=brueckner@linux.vnet.ibm.com \
--cc=acme@kernel.org \
--cc=acme@redhat.com \
--cc=jolsa@redhat.com \
--cc=linux-perf-users@vger.kernel.org \
--cc=mpetlan@redhat.com \
--cc=tmricht@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).