Re: [PATCH v2 06/13] um: Improve panic notifiers consistency and ordering
From: Johannes Berg
Date: Tue Aug 09 2022 - 15:18:50 EST
On Tue, 2022-08-09 at 16:03 -0300, Guilherme G. Piccoli wrote:
> On 09/08/2022 15:09, Johannes Berg wrote:
> > [...]
> > > > 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,
> > >
> >
> > Yeah, no objections, my previous comment was just a minor almost style
> > issue anyway.
> >
> > johannes
>
> Perfect, thank you! Let me take the opportunity to ask you something I'm
> asking all the maintainers involved here - do you prefer taking the
> patch through your tree, or to get it landed with the whole series, at
> once, from some maintainer?
>
Hm. I don't think we'd really care, but so far I was thinking - since
it's a series - it'd go through some appropriate tree all together. If
you think it should be applied separately, let us know.
johannes