Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-26 Thread Daniel Mentz
On Mon, Sep 26, 2016 at 5:38 AM, Alexey Brodkin wrote: > On Fri, 2016-09-23 at 18:04 -0700, Daniel Mentz wrote: >> On Fri, Sep 23, 2016 at 3:38 AM, Alexey Brodkin >> wrote: >> > >> > On Thu, 2016-09-22 at 15:37 -0700, Daniel Mentz wrote: >> > > >> > > Sorry, that was a misunderstanding. Buildroot

Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-26 Thread Alexey Brodkin
Hi Daniel, On Fri, 2016-09-23 at 18:04 -0700, Daniel Mentz wrote: > On Fri, Sep 23, 2016 at 3:38 AM, Alexey Brodkin > wrote: > > > > On Thu, 2016-09-22 at 15:37 -0700, Daniel Mentz wrote: > > > > > > Sorry, that was a misunderstanding. Buildroot routinely runs the strip > > > command on .ko fil

Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-23 Thread Daniel Mentz
On Fri, Sep 23, 2016 at 3:38 AM, Alexey Brodkin wrote: > On Thu, 2016-09-22 at 15:37 -0700, Daniel Mentz wrote: >> Sorry, that was a misunderstanding. Buildroot routinely runs the strip >> command on .ko files before installing them on the target. I was only >> looking at the .ko files *after* run

Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-23 Thread Alexey Brodkin
Hi Daniel, On Thu, 2016-09-22 at 15:37 -0700, Daniel Mentz wrote: > On Thu, Sep 22, 2016 at 1:59 PM, Vineet Gupta > wrote: > > > > Hi Daniel, > > > > On 09/19/2016 06:21 PM, Daniel Mentz wrote: > > > > > > I confirmed that the .eh_frame section is present and that the > > > .debug_frame sectio

Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-22 Thread Daniel Mentz
On Thu, Sep 22, 2016 at 1:59 PM, Vineet Gupta wrote: > Hi Daniel, > > On 09/19/2016 06:21 PM, Daniel Mentz wrote: >> I confirmed that the .eh_frame section is present and that the >> .debug_frame section is absent. I also verified that the file size of >> the .ko files are small enough for our emb

modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-22 Thread Vineet Gupta
Hi Daniel, On 09/19/2016 06:21 PM, Daniel Mentz wrote: > I confirmed that the .eh_frame section is present and that the > .debug_frame section is absent. I also verified that the file size of > the .ko files are small enough for our embedded platform and that > unnecessary sections like .debug_inf

Re: [PATCH 0/3] ARC unwinder switch to .eh_frame

2016-09-22 Thread Vineet Gupta
Hi Daniel, On 09/19/2016 11:13 PM, Vineet Gupta wrote: > On 09/19/2016 06:21 PM, Daniel Mentz wrote: >> > Hi Vineet, >> > >> > Thank you for your patches. I applied them, and verified that >> > unwinding works with code that is compiled into the kernel image as >> > well as kernel modules. >> > I

Re: [PATCH 0/3] ARC unwinder switch to .eh_frame

2016-09-19 Thread Vineet Gupta
On 09/19/2016 06:21 PM, Daniel Mentz wrote: > Hi Vineet, > > Thank you for your patches. I applied them, and verified that > unwinding works with code that is compiled into the kernel image as > well as kernel modules. > I confirmed that the .eh_frame section is present and that the > .debug_frame

Re: [PATCH 0/3] ARC unwinder switch to .eh_frame

2016-09-19 Thread Daniel Mentz
Hi Vineet, Thank you for your patches. I applied them, and verified that unwinding works with code that is compiled into the kernel image as well as kernel modules. I confirmed that the .eh_frame section is present and that the .debug_frame section is absent. I also verified that the file size of

[PATCH 0/3] ARC unwinder switch to .eh_frame

2016-09-19 Thread Vineet Gupta
Hi Daniel, Give these a shot and let me know if they work fine for you. I'm working on some more improvements to unwinder which will follow a bit later. Thx, -Vineet Vineet Gupta (3): ARC: module: support R_ARC_32_PCREL relocation ARC: dw2 unwind: factor CIE specifics for .eh_frame/.debug_fr