public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Indu Bhagat <indu.bhagat@oracle.com>
To: binutils@sourceware.org
Subject: Re: [PATCH,RFC 3/7] libctfframe: add the CTF Frame library
Date: Sun, 8 May 2022 15:00:19 -0700	[thread overview]
Message-ID: <56857abe-32c2-9f16-0ac2-99a82ebf3d7d@oracle.com> (raw)
In-Reply-To: <20220507005223.3093035-4-indu.bhagat@oracle.com>

On 5/6/22 5:52 PM, Indu Bhagat wrote:
> From: Weimin Pan <weimin.pan@oracle.com>
> 
> libctfframe is a library that allows you to:
> - decode a .ctf_frame section
> - probe and inspect a .ctf_frame section
> - encode (and eventually write) a .ctf_frame section.
> 
> This library is currently being used by the linker, readelf and objdump.
> The file include/ctf-frame-api.h defines the user-facing APIs for decoding,
> encoding and probing .ctf_frame sections. A set of error codes together
> with their error message strings are also defined.
> 
> Endian flipping is performed automatically at read and write time, if
> cross-endianness is detected.
> 
> ChangeLog:
> 
> 	* Makefile.def: Add libctfframe as new module with its
> 	dependencies.
> 	* Makefile.in: Regenerated.
> 	* binutils/Makefile.am: Add libctfframe.
> 	* binutils/Makefile.in: Regenerated.
> 	* configure: Regenerated
> 	* configure.ac: Add libctfframe to host_libs.
> 	* libctfframe/Makefile.am: New file.
> 	* libctfframe/Makefile.in: New file.
> 	* libctfframe/aclocal.m4: New file.
> 	* libctfframe/config.h.in: New file.
> 	* libctfframe/configure: New file.
> 	* libctfframe/configure.ac: New file.
> 	* libctfframe/ctf-frame-error.c: New file.
> 	* libctfframe/ctf-frame-impl.h: New file.
> 	* libctfframe/ctf-frame.c: New file.
> 
> include/ChangeLog:
> 
> 	* ctf-frame-api.h: New file.
> ---
>   Makefile.def                  |     2 +
>   Makefile.in                   |  1288 ++-
>   binutils/Makefile.am          |     2 +
>   binutils/Makefile.in          |     1 +
>   configure                     |     2 +-
>   configure.ac                  |     2 +-
>   include/ctf-frame-api.h       |   210 +
>   libctfframe/Makefile.am       |    38 +
>   libctfframe/Makefile.in       |   915 ++
>   libctfframe/aclocal.m4        |  1241 +++
>   libctfframe/config.h.in       |   144 +
>   libctfframe/configure         | 15118 ++++++++++++++++++++++++++++++++
>   libctfframe/configure.ac      |    75 +
>   libctfframe/ctf-frame-error.c |    49 +
>   libctfframe/ctf-frame-impl.h  |    55 +
>   libctfframe/ctf-frame.c       |  1515 ++++
>   16 files changed, 20650 insertions(+), 7 deletions(-)
>   create mode 100644 include/ctf-frame-api.h
>   create mode 100644 libctfframe/Makefile.am
>   create mode 100644 libctfframe/Makefile.in
>   create mode 100644 libctfframe/aclocal.m4
>   create mode 100644 libctfframe/config.h.in
>   create mode 100755 libctfframe/configure
>   create mode 100644 libctfframe/configure.ac
>   create mode 100644 libctfframe/ctf-frame-error.c
>   create mode 100644 libctfframe/ctf-frame-impl.h
>   create mode 100644 libctfframe/ctf-frame.c
> 

Hi,

This patch did not make to the list as it exceeds the 400 KB limit

--------------

Your mail to 'Binutils' with the subject

     [PATCH,RFC 3/7] libctfframe: add the CTF Frame library

Is being held until the list moderator can review it for approval.

The reason it is being held:

     Message body is too big: 666389 bytes with a limit of 400 KB

-------------

What is the recommended way to address this ? Splitting this patch up 
with libctfframe/configure in a separate commit will not address the 
issue as the libctfframe/configure will still be larger than 400 KB.

Will this eventually get moderator approval or should I push it all in a 
publicly available branch on github ?

Thanks
Indu

  parent reply	other threads:[~2022-05-08 22:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-07  0:52 [PATCH,RFC 0/7] Definition and Implementation of CTF Frame format Indu Bhagat
2022-05-07  0:52 ` [PATCH,RFC 1/7] ctf-frame.h: Add CTF Frame format definition Indu Bhagat
2022-05-07  0:52 ` [PATCH,RFC 2/7] gas: generate .ctf_frame Indu Bhagat
2022-05-07  0:52 ` [PATCH,RFC 4/7] bfd: linker: merge .ctf_frame sections Indu Bhagat
2022-05-07  0:52 ` [PATCH,RFC 5/7] readelf/objdump: support for CTF Frame section Indu Bhagat
2022-05-07  0:52 ` [PATCH,RFC 6/7] unwinder: generate backtrace using CTF Frame format Indu Bhagat
2022-05-07  0:52 ` [PATCH, RFC 7/7] gdb: sim: buildsystem changes to accommodate libctfframe Indu Bhagat
     [not found] ` <20220507005223.3093035-4-indu.bhagat@oracle.com>
2022-05-08 22:00   ` Indu Bhagat [this message]
2022-05-17 13:44     ` [PATCH,RFC 3/7] libctfframe: add the CTF Frame library Jan Beulich
2022-05-13 13:08 ` [PATCH,RFC 0/7] Definition and Implementation of CTF Frame format Michael Matz
2022-05-13 17:50   ` [PATCH, RFC " Indu Bhagat
2022-05-09 17:41 [PATCH,RFC 3/7] libctfframe: add the CTF Frame library Indu Bhagat

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=56857abe-32c2-9f16-0ac2-99a82ebf3d7d@oracle.com \
    --to=indu.bhagat@oracle.com \
    --cc=binutils@sourceware.org \
    /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).