Re: [PATCH][gnulib] Add the Sframe package

2022-11-25 Thread Jose E. Marchesi
> Weimin Pan wrote: >> and will withdraw the request of making Sframe part of Gnulib. > > OK. > > Just curious: How will you now offer the code to the applications > that want to incluse it? Through a shared library? Through a static > library? Through some source-code copy instructions? Binutil

Re: [PATCH][gnulib] Add the Sframe package

2022-11-24 Thread Bruno Haible
Weimin Pan wrote: > and will withdraw the request of making Sframe part of Gnulib. OK. Just curious: How will you now offer the code to the applications that want to incluse it? Through a shared library? Through a static library? Through some source-code copy instructions? Bruno

Re: [PATCH][gnulib] Add the Sframe package

2022-11-23 Thread Weimin Pan
On 11/18/2022 2:17 PM, Bruno Haible wrote: [Adding back Weimin to CC.] Jeffrey Walton wrote: I don't think it's a good idea to subsume libbacktrace and libunwind into Gnulib. First, from another project's point of view, the dependency has not changed. A dependency still exists. It has just m

Re: [PATCH][gnulib] Add the Sframe package

2022-11-19 Thread Bruno Haible
Weimin Pan wrote: > But the SFrame package that we'd like to be included in Gnulib > does not subsume or rely on either libbacktrace or libunwind. Maybe > I'm missing something here? That's not the major argument. Considerations that make Jeffrey and me think that the Sframe code should better be

Re: [PATCH][gnulib] Add the Sframe package

2022-11-19 Thread Bruno Haible
Weimin Pan wrote: > > I read the same thing in > > : > > "callee-saved registers other than FP are not needed for stack unwinding, > > and hence are not included in the .sframe section." > > but I don't understand it: If you don't have the conte

Re: [PATCH][gnulib] Add the Sframe package

2022-11-18 Thread Weimin Pan
On 11/17/2022 5:24 PM, Bruno Haible wrote: Weimin Pan wrote: We came to design the SFrame format (The S stands for `simple') due to some concrete requirements of a very big program that ships its own "online" stack tracer and unwinder to handle error conditions: 1) They wanted something simpl

Re: [PATCH][gnulib] Add the Sframe package

2022-11-18 Thread Weimin Pan
On 11/18/2022 2:17 PM, Bruno Haible wrote: [Adding back Weimin to CC.] Jeffrey Walton wrote: I don't think it's a good idea to subsume libbacktrace and libunwind into Gnulib. First, from another project's point of view, the dependency has not changed. A dependency still exists. It has just m

Re: [PATCH][gnulib] Add the Sframe package

2022-11-18 Thread Bruno Haible
[Adding back Weimin to CC.] Jeffrey Walton wrote: > I don't think it's a good idea to subsume libbacktrace and libunwind > into Gnulib. > > First, from another project's point of view, the dependency has not > changed. A dependency still exists. It has just moved around. > > Second, I'm guessing

Re: [PATCH][gnulib] Add the Sframe package

2022-11-18 Thread Jose E. Marchesi
> On Thu, Nov 17, 2022 at 8:24 PM Bruno Haible wrote: >> >> Weimin Pan wrote: >> > We came to design the SFrame format (The S stands for `simple') due to some >> > concrete requirements of a very big program that ships its own "online" >> > stack tracer and unwinder to handle error conditions: >

Re: [PATCH][gnulib] Add the Sframe package

2022-11-17 Thread Jeffrey Walton
On Thu, Nov 17, 2022 at 8:24 PM Bruno Haible wrote: > > Weimin Pan wrote: > > We came to design the SFrame format (The S stands for `simple') due to some > > concrete requirements of a very big program that ships its own "online" > > stack tracer and unwinder to handle error conditions: > > > > 1)

Re: [PATCH][gnulib] Add the Sframe package

2022-11-17 Thread Bruno Haible
Weimin Pan wrote: > We came to design the SFrame format (The S stands for `simple') due to some > concrete requirements of a very big program that ships its own "online" > stack tracer and unwinder to handle error conditions: > > 1) They wanted something simple to decode and simple to compute. Th

Re: [PATCH][gnulib] Add the Sframe package

2022-11-17 Thread Weimin Pan
Hi Bruno, Thanks for your comments. On 11/15/2022 4:52 PM, Bruno Haible wrote: Hello Weimin, The main use-case for this format are "online" debugging tools like stack tracers I'll appreciate anything that can help producing a universally working backtrace for C, since experience (e.g. from L

Re: [PATCH][gnulib] Add the Sframe package

2022-11-16 Thread Jose E. Marchesi
Hi Bruno. > Since this is a significant contribution (regardless whether it finally > goes into Gnulib [2] or into Binutils [3]), we will need a copyright > assignment > to the FSF for this code. Are you already aware how this works, in your > context > as an Oracle employee? Maybe José Marche

Re: [PATCH][gnulib] Add the Sframe package

2022-11-15 Thread Bruno Haible
Hello Weimin, > The main use-case for this format are > "online" debugging tools like stack tracers I'll appreciate anything that can help producing a universally working backtrace for C, since experience (e.g. from Lisp, Java, Python) has shown that such a backtrace function can tremendously imp

[PATCH][gnulib] Add the Sframe package

2022-11-15 Thread Weimin Pan
This patch adds two new modules to gnulib, related to the Simple Frame format (SFrame) recently introduced in binutils [1]. The SFrame format (Simple Frame format) represents the minimal necessary information for stack backtrace and it is stored in the .sframe section. The format's detailed defini