Re: [PATCH v11 3/5] arm64: Make the unwind loop in unwind() similar to other architectures

From: Mark Brown
Date: Thu Nov 25 2021 - 09:32:53 EST


On Tue, Nov 23, 2021 at 01:37:21PM -0600, madvenka@xxxxxxxxxxxxxxxxxxx wrote:

> unwind_start(&frame, fp, pc);
> while (unwind_continue(tsk, &frame, fn, data))
> unwind_next(tsk, &frame);

Other architectures seem to call their unwind_next() unwind_next_frame()
instead, and use a function unwind_done() rather than unwind_continue().
I appreciate that's actually a change carried through from one of the
earlier patches but might be worth considering. I don't really *mind*
that, though if there's any work on pulling more code out of the
architecture into the generic code it'll need revisiting in at least
some of the architectures.

Reviwed-by: Mark Brown <broonie@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature