public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH v3 00/17] Catch syscall group
@ 2015-04-26  1:25 Gabriel Krisman Bertazi
  2015-04-26  1:25 ` [PATCH v3 01/17] Implemement support for groups of syscalls in the xml-syscall interface Gabriel Krisman Bertazi
                   ` (18 more replies)
  0 siblings, 19 replies; 43+ messages in thread
From: Gabriel Krisman Bertazi @ 2015-04-26  1:25 UTC (permalink / raw)
  To: gdb-patches; +Cc: dje, sergiodj, Gabriel Krisman Bertazi

Hello,

This is the third version of the catch syscall group patches.  This
version presents the following modifications, other than fixing the
things Sergio and Doug pointed out in previous reviews:

* Created syscall groups for every architecture supported by catch
  syscall.

The previous version only implemented syscalls for amd64.  I used a
script to generate the xmls, and based the group field information on
strace, so please share your thoughts if you disagree with any group.

I babysitted the xml generation scripts but I was not able to actually
test them on architectures other than amd64 and i386.  It would be very
nice if someone could actually run catch-syscall.exp on other
architectures.

* Now, catch syscall group tests are performed on every architecture
that supports catch-syscall and not only on amd64.

I'm also not resending the documentation patch because Eli already
approved it (twice :)

I've split patches in several parts to ease review.  I'll squash
them all together before pushing upstream.

Please, tell me what you think!

Gabriel Krisman Bertazi (17):
  Implemement support for groups of syscalls in the xml-syscall    
    interface.
  Add support to catch groups of syscalls.
  Add tests for catching groups of syscalls on supported    
    architectures.
  Create syscall groups for amd64.
  Create syscall groups for ppc.
  Create syscall groups for ppc64.
  Create syscall groups for aarch64.
  Create syscall groups for arm.
  Create syscall groups for bfin.
  Create syscall groups for i386.
  Create syscall groups for mips-n32.
  Create syscall groups for mips-n64.
  Create syscall groups for mips-o32.
  Create syscall groups for s390.
  Create syscall groups for s390x.
  Create syscall groups for sparc.
  Create syscall groups for sparc64.

 gdb/break-catch-syscall.c                |  94 ++++++-
 gdb/syscalls/aarch64-linux.xml           | 314 +++++++++++----------
 gdb/syscalls/amd64-linux.xml             | 364 ++++++++++++------------
 gdb/syscalls/arm-linux.xml               | 470 ++++++++++++++++---------------
 gdb/syscalls/bfin-linux.xml              | 368 ++++++++++++------------
 gdb/syscalls/gdb-syscalls.dtd            |   3 +-
 gdb/syscalls/i386-linux.xml              | 346 ++++++++++++-----------
 gdb/syscalls/mips-n32-linux.xml          | 374 ++++++++++++------------
 gdb/syscalls/mips-n64-linux.xml          | 362 ++++++++++++------------
 gdb/syscalls/mips-o32-linux.xml          | 398 +++++++++++++-------------
 gdb/syscalls/ppc-linux.xml               | 316 ++++++++++-----------
 gdb/syscalls/ppc64-linux.xml             | 288 +++++++++----------
 gdb/syscalls/s390-linux.xml              | 342 +++++++++++-----------
 gdb/syscalls/s390x-linux.xml             | 324 ++++++++++-----------
 gdb/syscalls/sparc-linux.xml             | 344 +++++++++++-----------
 gdb/syscalls/sparc64-linux.xml           | 332 +++++++++++-----------
 gdb/testsuite/gdb.base/catch-syscall.exp |  41 +++
 gdb/xml-syscall.c                        | 233 ++++++++++++++-
 gdb/xml-syscall.h                        |  16 ++
 19 files changed, 2860 insertions(+), 2469 deletions(-)

-- 
1.9.3

^ permalink raw reply	[flat|nested] 43+ messages in thread
* Re: [PATCH v4 4/5] Include group information in xml syscall files.
@ 2015-05-16  0:33 Doug Evans
  0 siblings, 0 replies; 43+ messages in thread
From: Doug Evans @ 2015-05-16  0:33 UTC (permalink / raw)
  To: Gabriel Krisman Bertazi; +Cc: Sergio Durigan Junior, Pedro Alves, gdb-patches

Gabriel Krisman Bertazi writes:
  > Doug Evans <dje@google.com> writes:
  >
  > > I would have expected syscalls/Makefile.in,
  > > with requisite changes to generate Makefile in some configure.ac file.
  >
  > Hi Doug,
  >
  > I saw your other message and I'll surely answer and apply your
  > suggestions as soon as I can, but let me make a quick question and
  > comment on my design here :)
  >
  > I tried to follow the exact same design we have in gdb/Features.  There,
  > we only have a Makefile that doesn't depend on the configure script.  If
  > I understand correctly, all it takes to generate the files is a simple
  > 'make blah'.  I tried to reproduce this behavior here.

Ugh.  I see.
Well I guess I can't ask for more here.

Note that all I'm asking for (effectively) is that
your Makefile get renamed as Makefile.in,
and that the real Makefile get created at build time
in the build directory along with the other makefiles.
That does mean that the makefile has to cope with
being in the buildir and not srcdir, but that's doable.

All the --enable-maintainer-mode switch does is turn
on dependencies so that all I have to do is edit
one of those source files and then the normal make
will DTRT to regenerate the files and then
continue with the rest of the build.
IOW, with maintainer mode turned on there is no
extra separate manual step to regenerate the files.

  > Even though I understand the obvious benefits of using the configure
  > script, is there any actual differences between the expected behavior in
  > gdb/Features/Makefile that justifies it being a simple Makefile?  If
  > not, I'll probably work on a follow-up patch adjusting it, as well.

The benefit is that the build continues to Just Work,
even when modifying these files (whereas otherwise one might not notice
that these source files are special and have to spend time figuring
out why one can't just edit the file and type "make").

Obviously, we don't, necessarily, want to impose on everyone
building from trunk to require the additional dependencies
(e.g., xsltproc), and thus the configure switch.

btw,
I can imagine keying this off of --enable-maintainer-mode
could annoy someone who mostly works with binutils
but still builds gdb. I'd be happy with --enable-gdb-maintainer-mode.
I'd say no need to change to use it now, and we can switch to
using the configure switch later.
But the more we wait the more inertia we have to overcome.

  > Btw, can any of you guys provide me with more information about the
  > maintainers mode?  I couldn't find many explanations in the Internals
  > wiki.  I can update the wiki once I have a better understanding of
  > it.

I doubt there is more documentation than this:

http://www.gnu.org/software/automake/manual/html_node/maintainer_002dmode.html#maintainer_002dmode

^ permalink raw reply	[flat|nested] 43+ messages in thread

end of thread, other threads:[~2015-05-16  0:33 UTC | newest]

Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-26  1:25 [PATCH v3 00/17] Catch syscall group Gabriel Krisman Bertazi
2015-04-26  1:25 ` [PATCH v3 01/17] Implemement support for groups of syscalls in the xml-syscall interface Gabriel Krisman Bertazi
2015-04-26  1:25 ` [PATCH v3 03/17] Add tests for catching groups of syscalls on supported architectures Gabriel Krisman Bertazi
2015-04-26 18:44   ` Sergio Durigan Junior
2015-04-26  1:25 ` [PATCH v3 02/17] Add support to catch groups of syscalls Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 12/17] Create syscall groups for mips-n64 Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 08/17] Create syscall groups for arm Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 06/17] Create syscall groups for ppc64 Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 10/17] Create syscall groups for i386 Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 07/17] Create syscall groups for aarch64 Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 05/17] Create syscall groups for ppc Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 04/17] Create syscall groups for amd64 Gabriel Krisman Bertazi
2015-04-26  1:26 ` [PATCH v3 09/17] Create syscall groups for bfin Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 11/17] Create syscall groups for mips-n32 Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 14/17] Create syscall groups for s390 Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 15/17] Create syscall groups for s390x Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 16/17] Create syscall groups for sparc Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 17/17] Create syscall groups for sparc64 Gabriel Krisman Bertazi
2015-04-26  1:47 ` [PATCH v3 13/17] Create syscall groups for mips-o32 Gabriel Krisman Bertazi
2015-04-26 18:58 ` [PATCH v3 00/17] Catch syscall group Sergio Durigan Junior
2015-04-28 11:24 ` Pedro Alves
2015-04-29  0:45   ` Sergio Durigan Junior
2015-04-29 10:44     ` Pedro Alves
2015-05-04  2:34       ` Gabriel Krisman Bertazi
2015-05-06 14:38         ` Pedro Alves
2015-05-10 18:34           ` Gabriel Krisman Bertazi
2015-05-10 19:01             ` Sergio Durigan Junior
2015-05-11  0:28               ` [PATCH v4 0/4] catch " Gabriel Krisman Bertazi
2015-05-11  0:28                 ` [PATCH v4 5/5] Update documentation on catching a group of related syscalls Gabriel Krisman Bertazi
2015-05-11  0:40                   ` Gabriel Krisman Bertazi
2015-05-13 10:30                     ` Pedro Alves
2015-05-13 16:40                       ` Eli Zaretskii
2015-05-11  0:28                 ` [PATCH v4 1/5] Implemement support for groups of syscalls in the xml-syscall interface Gabriel Krisman Bertazi
2015-05-11  0:28                 ` [PATCH v4 3/5] Add tests for catching groups of syscalls on supported architectures Gabriel Krisman Bertazi
2015-05-11  0:28                 ` [PATCH v4 4/5] Include group information in xml syscall files Gabriel Krisman Bertazi
2015-05-12 21:42                   ` Doug Evans
2015-05-13  1:17                     ` Gabriel Krisman Bertazi
2015-05-13 10:43                   ` Pedro Alves
2015-05-11  0:28                 ` [PATCH v4 2/5] Add support to catch groups of syscalls Gabriel Krisman Bertazi
2015-05-13 10:38                   ` Pedro Alves
2015-05-13 10:47                 ` [PATCH v4 0/4] catch syscall group Pedro Alves
2015-05-11 11:39               ` [PATCH v3 00/17] Catch " Pedro Alves
2015-05-16  0:33 [PATCH v4 4/5] Include group information in xml syscall files Doug Evans

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 read-only IMAP folder(s) and NNTP newsgroup(s).