On 19/07/2022 16:53, Guilherme G. Piccoli wrote: > Currently the panic notifiers from user mode linux don't follow > the convention for most of the other notifiers present in the > kernel (indentation, priority setting, numeric return). > More important, the priorities could be improved, since it's a > special case (userspace), hence we could run the notifiers earlier; > user mode linux shouldn't care much with other panic notifiers but > the ordering among the mconsole and arch notifier is important, > given that the arch one effectively triggers a core dump. > > Fix that by running the mconsole notifier as the first panic > notifier, followed by the architecture one (that coredumps). > > Cc: Anton Ivanov <[email protected]> > Cc: Johannes Berg <[email protected]> > Cc: Richard Weinberger <[email protected]> > Signed-off-by: Guilherme G. Piccoli <[email protected]> > > --- > > V2: > - Kept the notifier header to avoid implicit usage - thanks > Johannes for the suggestion! > > arch/um/drivers/mconsole_kern.c | 7 +++---- > arch/um/kernel/um_arch.c | 8 ++++---- > 2 files changed, 7 insertions(+), 8 deletions(-) > [...]
Hi Johannes, do you feel this one is good now, after your last review? Thanks in advance, Guilherme _______________________________________________ linux-um mailing list [email protected] http://lists.infradead.org/mailman/listinfo/linux-um
