On Tue, Sep 27, 2022, David Matlack wrote: > Bail out of test_dump_stack() if the stack trace is empty rather than > invoking addr2line with zero addresses. The problem with the latter is > that addr2line will block waiting for addresses to be passed in via > stdin, e.g. if running a selftest from an interactive terminal. > > Opportunistically fix up the comment that mentions skipping 3 frames > since only 2 are skipped in the code, and move the call to backtrace() > down to where it is used. > > Cc: Vipin Sharma <vipinsh@xxxxxxxxxx> > Cc: Sean Christopherson <seanjc@xxxxxxxxxx> > Signed-off-by: David Matlack <dmatlack@xxxxxxxxxx> > --- In case Paolo sees the error of his ways ;-) Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx>