> On Jun 24, 2023, at 3:13 AM, Andrew Jones <andrew.jones@xxxxxxxxx> wrote: > >> +extern char vector_stub_start, vector_stub_end; > > These aren't used until the next patch. > >> + >> +int backtrace_frame(const void *frame, const void **return_addrs, int max_depth) { > > '{' should be on its own line. I usually try to run the kernel's > checkpatch since we use the same style (except we're even more forgiving > for long lines). I usually do use checkpatch. I guess I got sloppy. I will fix these 2 issues. BTW: I used the get_maintainer script to get who to send the patches to and it included the depreciated kvmarm@xxxxxxxxxxxxxxxxxxxxx <mailto:kvmarm@xxxxxxxxxxxxxxxxxxxxx> .. Ugh.