Re: [PATCH v3] strip: keep .ctf section in stripped file

2023-03-07 Thread Guillermo E. Martinez via Elfutils-devel
On Tue, Mar 07, 2023 at 03:50:01PM +0100, Mark Wielaard wrote: > Hi Nick, Hi Guillermo, > Hi Mark, > On Sat, 2023-03-04 at 08:00 -0600, Guillermo E. Martinez wrote: > > Hello, > > > > On Fri, Mar 03, 2023 at 12:24:19PM +, Nick Clifton wrote: > > > Hi Mark, > > > > > > > > > I am mainly wo

Re: [PATCH v3] strip: keep .ctf section in stripped file

2023-03-04 Thread Guillermo E. Martinez via Elfutils-devel
Hello, On Fri, Mar 03, 2023 at 12:24:19PM +, Nick Clifton wrote: > Hi Mark, > > > > > I am mainly wondering why binutils strip already seems to keep ".ctf" > > > > sections (even without -g). > > > Right. But I am wondering how that mechanism works with binutils strip. > > Apparently we do s

Re: [PATCH v3] strip: keep .ctf section in stripped file

2023-03-02 Thread Guillermo E. Martinez via Elfutils-devel
Hello Mark, On Tue, Feb 28, 2023 at 03:27:13PM +0100, Mark Wielaard wrote: > Hi Nick, > > On Tue, 2023-02-28 at 12:59 +, Nick Clifton wrote: > > > O, this surprises me. I wasn't aware binutils strip keeps unallocated > > > sections by default. But apparently it does. It doesn't seem specific

Re: [PATCH v3] strip: keep .ctf section in stripped file

2023-02-24 Thread Guillermo E. Martinez via Elfutils-devel
On Fri, Feb 24, 2023 at 12:51:25PM +0100, Mark Wielaard wrote: > Hi Guillermo, > Hi Mark, > On Thu, Feb 23, 2023 at 12:42:37PM -0600, Guillermo E. Martinez via > Elfutils-devel wrote: > > This is the third version of the patch to avoid remove the CTF section in > > stri

[PATCH v3] strip: keep .ctf section in stripped file

2023-02-23 Thread Guillermo E. Martinez via Elfutils-devel
Hello, This is the third version of the patch to avoid remove the CTF section in stripped files. Changes from v2: - Rebased from master. Please let me know your thoughts. Kind regards, guillermo -- CTF debug format was designed to be present in stripped files, so this section should not be r

Re: [PATCHv2] strip: keep .ctf section in stripped file

2023-02-23 Thread Guillermo E. Martinez via Elfutils-devel
On Thu, Feb 23, 2023 at 12:04:47AM +0100, Mark Wielaard wrote: > Hi, Hi Mark, > > On Wed, Feb 22, 2023 at 11:12:07AM -0600, Guillermo E. Martinez wrote: > > The conclusion was basically not use section flags to identify which > > section should be stripped out or not, so, it requires other mecha

Re: [PATCHv2] strip: keep .ctf section in stripped file

2023-02-22 Thread Guillermo E. Martinez via Elfutils-devel
On Wed, Feb 22, 2023 at 05:42:45PM +0100, Mark Wielaard wrote: > Hi, > Hi Mark, > On Tue, 2022-12-20 at 22:35 +0100, Mark Wielaard wrote: > > On Wed, Jun 01, 2022 at 10:55:27AM -0500, Guillermo E. Martinez via > > Elfutils-devel wrote: > > > This is the second ver

[PATCHv2] strip: keep .ctf section in stripped file

2022-06-01 Thread Guillermo E. Martinez via Elfutils-devel
Hello, This is the second version patch to avoid remove the CTF section in stripped files. Changes from v1: - Add description in tests/run-strip-remove-keep-ctf.sh mentioning how to regenerate test input file (testfile-ctf) Please let me know your thoughts. Kind regard, guillermo CTF deb

Re: [PATCH] strip: keep .ctf section in stripped file

2022-05-31 Thread Guillermo E. Martinez via Elfutils-devel
On Tuesday, May 31, 2022 2:06:26 AM CDT Mark Wielaard wrote: Hi, Mark > > On Mon, May 30, 2022 at 09:26:19PM -0500, Guillermo E. Martinez via > Elfutils-devel wrote: > > Hello elfutils team, > > > > This patch is meant to avoid remove the CTF section in > > st

[PATCH] strip: keep .ctf section in stripped file

2022-05-30 Thread Guillermo E. Martinez via Elfutils-devel
Hello elfutils team, This patch is meant to avoid remove the CTF section in stripped files. Please let me know your thoughts. Kind regard, Guillermo CTF debug format was designed to be present in stripped files, so this section should not be removed, so a new --remove-ctf option is added to in