From: Simon Marchi <simark@simark.ca>
To: Kamil Rytarowski <n54@gmx.com>, gdb-patches@sourceware.org
Cc: tom@tromey.com, jhb@FreeBSD.org
Subject: Re: [PATCH v2] Add SVR4 psABI specific parser for AUXV entries
Date: Wed, 8 Apr 2020 20:04:25 -0400 [thread overview]
Message-ID: <46801343-faf4-7db0-5dbe-35bd23b14603@simark.ca> (raw)
In-Reply-To: <20200408145832.24243-1-n54@gmx.com>
> diff --git a/gdb/auxv.c b/gdb/auxv.c
> index c13d7a22eb9..28e8d2f704d 100644
> --- a/gdb/auxv.c
> +++ b/gdb/auxv.c
> @@ -248,34 +248,65 @@ memory_xfer_auxv (struct target_ops *ops,
> return procfs_xfer_auxv (readbuf, writebuf, offset, len, xfered_len);
> }
>
> -/* Read one auxv entry from *READPTR, not reading locations >= ENDPTR.
> - Return 0 if *READPTR is already at the end of the buffer.
> - Return -1 if there is insufficient buffer for a whole entry.
> - Return 1 if an entry was read into *TYPEP and *VALP. */
> -int
> -default_auxv_parse (struct target_ops *ops, gdb_byte **readptr,
> - gdb_byte *endptr, CORE_ADDR *typep, CORE_ADDR *valp)
> +/* This function compared to other auxv_parse functions: it takes the size of
> + the auxv type field as a parameter. */
> +
> +static int
> +generic_auxv_parse (struct gdbarch *gdbarch, gdb_byte **readptr,
> + gdb_byte *endptr, CORE_ADDR *typep, CORE_ADDR *valp,
> + int sizeof_auxv_type)
> {
> - const int sizeof_auxv_field = gdbarch_ptr_bit (target_gdbarch ())
> - / TARGET_CHAR_BIT;
> - const enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch ());
> + struct type *ptr_type = builtin_type (gdbarch)->builtin_data_ptr;
> + const int sizeof_auxv_val = TYPE_LENGTH (ptr_type);
> + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
> gdb_byte *ptr = *readptr;
>
> if (endptr == ptr)
> return 0;
>
> - if (endptr - ptr < sizeof_auxv_field * 2)
> + if (endptr - ptr < 2 * sizeof_auxv_val)
> return -1;
>
> - *typep = extract_unsigned_integer (ptr, sizeof_auxv_field, byte_order);
> - ptr += sizeof_auxv_field;
> - *valp = extract_unsigned_integer (ptr, sizeof_auxv_field, byte_order);
> - ptr += sizeof_auxv_field;
> + *typep = extract_unsigned_integer (ptr, sizeof_auxv_type, byte_order);
> + /* Even if the auxv type takes less space than an auxv value, there is
> + padding after the type such that the value is aligned on a multiple of
> + its size (and this is why we advance by `sizeof_auxv_val` and not
> + `sizeof_auxv_type`. */
The parenthesis in the comment is not closed.
> diff --git a/gdb/auxv.h b/gdb/auxv.h
> index a5a932ec80e..c1d11ea2c7b 100644
> --- a/gdb/auxv.h
> +++ b/gdb/auxv.h
> @@ -25,12 +25,27 @@
> /* See "include/elf/common.h" for the definition of valid AT_* values. */
>
> /* The default implementation of to_auxv_parse, used by the target
> - stack. */
> + stack.
>
> + Read one auxv entry from *READPTR, not reading locations >= ENDPTR.
> + Return 0 if *READPTR is already at the end of the buffer.
> + Return -1 if there is insufficient buffer for a whole entry.
> + Return 1 if an entry was read into *TYPEP and *VALP. */
> extern int default_auxv_parse (struct target_ops *ops, gdb_byte **readptr,
> gdb_byte *endptr, CORE_ADDR *typep,
> CORE_ADDR *valp);
>
> +/* The SVR4 psABI implementation of to_auxv_parse, that uses an int to
> + store the type rather than long as assumed by the default parser.
> +
> + Read one auxv entry from *READPTR, not reading locations >= ENDPTR.
> + Return 0 if *READPTR is already at the end of the buffer.
> + Return -1 if there is insufficient buffer for a whole entry.
> + Return 1 if an entry was read into *TYPEP and *VALP. */
> +extern int svr4_auxv_parse (struct gdbarch *gdbarch, gdb_byte **readptr,
> + gdb_byte *endptr, CORE_ADDR *typep,
> + CORE_ADDR *valp);
The last two lines are not properly aligned on the parenthesis.
Simon
next prev parent reply other threads:[~2020-04-09 0:04 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-08 2:14 [PATCH] " Kamil Rytarowski
2020-04-08 4:11 ` Simon Marchi
2020-04-08 14:59 ` Kamil Rytarowski
2020-04-08 14:58 ` [PATCH v2] " Kamil Rytarowski
2020-04-09 0:04 ` Simon Marchi [this message]
2020-04-09 0:57 ` Kamil Rytarowski
2020-04-09 0:56 ` [PATCH v3] " Kamil Rytarowski
2020-04-09 1:00 ` Simon Marchi
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=46801343-faf4-7db0-5dbe-35bd23b14603@simark.ca \
--to=simark@simark.ca \
--cc=gdb-patches@sourceware.org \
--cc=jhb@FreeBSD.org \
--cc=n54@gmx.com \
--cc=tom@tromey.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 read-only IMAP folder(s) and NNTP newsgroup(s).