All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] kgdb: fix gcc-11 warning on indentation
@ 2021-03-22 16:43 Arnd Bergmann
  2021-03-22 17:03 ` Daniel Thompson
  2021-03-22 17:04 ` Doug Anderson
  0 siblings, 2 replies; 10+ messages in thread
From: Arnd Bergmann @ 2021-03-22 16:43 UTC (permalink / raw)
  To: Jason Wessel, Daniel Thompson, Arnd Bergmann, Greg Kroah-Hartman,
	Ingo Molnar
  Cc: Douglas Anderson, Christian Brauner, kgdb-bugreport, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

gcc-11 starts warning about misleading indentation inside of macros:

drivers/misc/kgdbts.c: In function ‘kgdbts_break_test’:
drivers/misc/kgdbts.c:103:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
  103 |         if (verbose > 1) \
      |         ^~
drivers/misc/kgdbts.c:200:9: note: in expansion of macro ‘v2printk’
  200 |         v2printk("kgdbts: breakpoint complete\n");
      |         ^~~~~~~~
drivers/misc/kgdbts.c:105:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
  105 |                 touch_nmi_watchdog();   \
      |                 ^~~~~~~~~~~~~~~~~~

The code looks correct to me, so just reindent it for readability.

Fixes: e8d31c204e36 ("kgdb: add kgdb internal test suite")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/misc/kgdbts.c | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/drivers/misc/kgdbts.c b/drivers/misc/kgdbts.c
index 945701bce553..2e081a58da6c 100644
--- a/drivers/misc/kgdbts.c
+++ b/drivers/misc/kgdbts.c
@@ -95,19 +95,19 @@
 
 #include <asm/sections.h>
 
-#define v1printk(a...) do { \
-	if (verbose) \
-		printk(KERN_INFO a); \
-	} while (0)
-#define v2printk(a...) do { \
-	if (verbose > 1) \
-		printk(KERN_INFO a); \
-		touch_nmi_watchdog();	\
-	} while (0)
-#define eprintk(a...) do { \
-		printk(KERN_ERR a); \
-		WARN_ON(1); \
-	} while (0)
+#define v1printk(a...) do {		\
+	if (verbose)			\
+		printk(KERN_INFO a);	\
+} while (0)
+#define v2printk(a...) do {		\
+	if (verbose > 1)		\
+		printk(KERN_INFO a);	\
+	touch_nmi_watchdog();		\
+} while (0)
+#define eprintk(a...) do {		\
+	printk(KERN_ERR a);		\
+	WARN_ON(1);			\
+} while (0)
 #define MAX_CONFIG_LEN		40
 
 static struct kgdb_io kgdbts_io_ops;
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 16:43 [PATCH] kgdb: fix gcc-11 warning on indentation Arnd Bergmann
@ 2021-03-22 17:03 ` Daniel Thompson
  2021-03-22 18:03   ` Arnd Bergmann
  2021-03-22 17:04 ` Doug Anderson
  1 sibling, 1 reply; 10+ messages in thread
From: Daniel Thompson @ 2021-03-22 17:03 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Jason Wessel, Arnd Bergmann, Greg Kroah-Hartman, Ingo Molnar,
	Douglas Anderson, Christian Brauner, kgdb-bugreport,
	linux-kernel

On Mon, Mar 22, 2021 at 05:43:03PM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> gcc-11 starts warning about misleading indentation inside of macros:
> 
> drivers/misc/kgdbts.c: In function ‘kgdbts_break_test’:
> drivers/misc/kgdbts.c:103:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
>   103 |         if (verbose > 1) \
>       |         ^~
> drivers/misc/kgdbts.c:200:9: note: in expansion of macro ‘v2printk’
>   200 |         v2printk("kgdbts: breakpoint complete\n");
>       |         ^~~~~~~~
> drivers/misc/kgdbts.c:105:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
>   105 |                 touch_nmi_watchdog();   \
>       |                 ^~~~~~~~~~~~~~~~~~
> 
> The code looks correct to me, so just reindent it for readability.
> 
> Fixes: e8d31c204e36 ("kgdb: add kgdb internal test suite")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Acked-by: Daniel Thompson <daniel.thompson@linaro.org>

Which tree do you want to merge this one though? I've got nothing else
pending for this file so I am very relaxed about the route...


Daniel.


> ---
>  drivers/misc/kgdbts.c | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/misc/kgdbts.c b/drivers/misc/kgdbts.c
> index 945701bce553..2e081a58da6c 100644
> --- a/drivers/misc/kgdbts.c
> +++ b/drivers/misc/kgdbts.c
> @@ -95,19 +95,19 @@
>  
>  #include <asm/sections.h>
>  
> -#define v1printk(a...) do { \
> -	if (verbose) \
> -		printk(KERN_INFO a); \
> -	} while (0)
> -#define v2printk(a...) do { \
> -	if (verbose > 1) \
> -		printk(KERN_INFO a); \
> -		touch_nmi_watchdog();	\
> -	} while (0)
> -#define eprintk(a...) do { \
> -		printk(KERN_ERR a); \
> -		WARN_ON(1); \
> -	} while (0)
> +#define v1printk(a...) do {		\
> +	if (verbose)			\
> +		printk(KERN_INFO a);	\
> +} while (0)
> +#define v2printk(a...) do {		\
> +	if (verbose > 1)		\
> +		printk(KERN_INFO a);	\
> +	touch_nmi_watchdog();		\
> +} while (0)
> +#define eprintk(a...) do {		\
> +	printk(KERN_ERR a);		\
> +	WARN_ON(1);			\
> +} while (0)
>  #define MAX_CONFIG_LEN		40
>  
>  static struct kgdb_io kgdbts_io_ops;
> -- 
> 2.29.2
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 16:43 [PATCH] kgdb: fix gcc-11 warning on indentation Arnd Bergmann
  2021-03-22 17:03 ` Daniel Thompson
@ 2021-03-22 17:04 ` Doug Anderson
  2021-03-22 18:18   ` Arnd Bergmann
  2021-03-22 19:23   ` Daniel Thompson
  1 sibling, 2 replies; 10+ messages in thread
From: Doug Anderson @ 2021-03-22 17:04 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Jason Wessel, Daniel Thompson, Arnd Bergmann, Greg Kroah-Hartman,
	Ingo Molnar, Christian Brauner, kgdb-bugreport, LKML

Hi,

On Mon, Mar 22, 2021 at 9:43 AM Arnd Bergmann <arnd@kernel.org> wrote:
>
> -#define v1printk(a...) do { \
> -       if (verbose) \
> -               printk(KERN_INFO a); \
> -       } while (0)
> -#define v2printk(a...) do { \
> -       if (verbose > 1) \
> -               printk(KERN_INFO a); \
> -               touch_nmi_watchdog();   \
> -       } while (0)
> -#define eprintk(a...) do { \
> -               printk(KERN_ERR a); \
> -               WARN_ON(1); \
> -       } while (0)
> +#define v1printk(a...) do {            \

nit: In addition to the indentation change you're also lining up the
backslashes. Is that just personal preference, or is there some
official recommendation in the kernel? I don't really have a strong
opinion either way (IMO each style has its advantages).


> +       if (verbose)                    \
> +               printk(KERN_INFO a);    \
> +} while (0)
> +#define v2printk(a...) do {            \
> +       if (verbose > 1)                \
> +               printk(KERN_INFO a);    \
> +       touch_nmi_watchdog();           \

This touch_nmi_watchdog() is pretty wonky. I guess maybe the
assumption is that the "verbose level 2" prints are so chatty that the
printing might prevent us from touching the NMI watchdog in the way
that we normally do and thus we need an extra one here?

...but, in that case, I think the old code was _wrong_ and that the
intention was that the touch_nmi_watchdog() should only be if "verose
> 1" as the indentation implied. There doesn't feel like a reason to
touch the watchdog if we're not doing anything slow.

-Doug

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 17:03 ` Daniel Thompson
@ 2021-03-22 18:03   ` Arnd Bergmann
  2021-03-23  7:28     ` Greg Kroah-Hartman
  0 siblings, 1 reply; 10+ messages in thread
From: Arnd Bergmann @ 2021-03-22 18:03 UTC (permalink / raw)
  To: Daniel Thompson
  Cc: Jason Wessel, Greg Kroah-Hartman, Ingo Molnar, Douglas Anderson,
	Christian Brauner, kgdb-bugreport, Linux Kernel Mailing List

On Mon, Mar 22, 2021 at 6:03 PM Daniel Thompson
<daniel.thompson@linaro.org> wrote:
>
>
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>
> Which tree do you want to merge this one though? I've got nothing else
> pending for this file so I am very relaxed about the route...

I don't plan to merge any of the build fixes through my own tree.
If you don't have anything else, maybe Greg can pick it up
in the char-misc tree.

         Arnd

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 17:04 ` Doug Anderson
@ 2021-03-22 18:18   ` Arnd Bergmann
  2021-03-22 19:22     ` Doug Anderson
  2021-03-22 19:23   ` Daniel Thompson
  1 sibling, 1 reply; 10+ messages in thread
From: Arnd Bergmann @ 2021-03-22 18:18 UTC (permalink / raw)
  To: Doug Anderson
  Cc: Jason Wessel, Daniel Thompson, Greg Kroah-Hartman, Ingo Molnar,
	Christian Brauner, kgdb-bugreport, LKML

On Mon, Mar 22, 2021 at 6:07 PM Doug Anderson <dianders@chromium.org> wrote:
> On Mon, Mar 22, 2021 at 9:43 AM Arnd Bergmann <arnd@kernel.org> wrote:
> >
> > -#define v1printk(a...) do { \
> > -       if (verbose) \
> > -               printk(KERN_INFO a); \
> > -       } while (0)
> > -#define v2printk(a...) do { \
> > -       if (verbose > 1) \
> > -               printk(KERN_INFO a); \
> > -               touch_nmi_watchdog();   \
> > -       } while (0)
> > -#define eprintk(a...) do { \
> > -               printk(KERN_ERR a); \
> > -               WARN_ON(1); \
> > -       } while (0)
> > +#define v1printk(a...) do {            \
>
> nit: In addition to the indentation change you're also lining up the
> backslashes. Is that just personal preference, or is there some
> official recommendation in the kernel? I don't really have a strong
> opinion either way (IMO each style has its advantages).

I don't think there is an official recommendation, I just think the
style is more common, and it helped my figure out what the
indentation should look like in this case.

>
> > +       if (verbose)                    \
> > +               printk(KERN_INFO a);    \
> > +} while (0)
> > +#define v2printk(a...) do {            \
> > +       if (verbose > 1)                \
> > +               printk(KERN_INFO a);    \
> > +       touch_nmi_watchdog();           \
>
> This touch_nmi_watchdog() is pretty wonky. I guess maybe the
> assumption is that the "verbose level 2" prints are so chatty that the
> printing might prevent us from touching the NMI watchdog in the way
> that we normally do and thus we need an extra one here?
>
> ...but, in that case, I think the old code was _wrong_ and that the
> intention was that the touch_nmi_watchdog() should only be if "verose
> > 1" as the indentation implied. There doesn't feel like a reason to
> touch the watchdog if we're not doing anything slow.

No idea. It was like this in Jason's original version from 2008.

        Arnd

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 18:18   ` Arnd Bergmann
@ 2021-03-22 19:22     ` Doug Anderson
  2021-03-22 20:14       ` Jason Wessel
  0 siblings, 1 reply; 10+ messages in thread
From: Doug Anderson @ 2021-03-22 19:22 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Jason Wessel, Daniel Thompson, Greg Kroah-Hartman, Ingo Molnar,
	Christian Brauner, kgdb-bugreport, LKML

Hi,

On Mon, Mar 22, 2021 at 11:19 AM Arnd Bergmann <arnd@kernel.org> wrote:
>
> On Mon, Mar 22, 2021 at 6:07 PM Doug Anderson <dianders@chromium.org> wrote:
> > On Mon, Mar 22, 2021 at 9:43 AM Arnd Bergmann <arnd@kernel.org> wrote:
> > >
> > > -#define v1printk(a...) do { \
> > > -       if (verbose) \
> > > -               printk(KERN_INFO a); \
> > > -       } while (0)
> > > -#define v2printk(a...) do { \
> > > -       if (verbose > 1) \
> > > -               printk(KERN_INFO a); \
> > > -               touch_nmi_watchdog();   \
> > > -       } while (0)
> > > -#define eprintk(a...) do { \
> > > -               printk(KERN_ERR a); \
> > > -               WARN_ON(1); \
> > > -       } while (0)
> > > +#define v1printk(a...) do {            \
> >
> > nit: In addition to the indentation change you're also lining up the
> > backslashes. Is that just personal preference, or is there some
> > official recommendation in the kernel? I don't really have a strong
> > opinion either way (IMO each style has its advantages).
>
> I don't think there is an official recommendation, I just think the
> style is more common, and it helped my figure out what the
> indentation should look like in this case.

OK, makes sense. I just wasn't sure if there was some standard that I
wasn't aware of. Given that you have to touch all these lines anyway
then making them all pretty like this seems fine to me.


> > > +       if (verbose)                    \
> > > +               printk(KERN_INFO a);    \
> > > +} while (0)
> > > +#define v2printk(a...) do {            \
> > > +       if (verbose > 1)                \
> > > +               printk(KERN_INFO a);    \
> > > +       touch_nmi_watchdog();           \
> >
> > This touch_nmi_watchdog() is pretty wonky. I guess maybe the
> > assumption is that the "verbose level 2" prints are so chatty that the
> > printing might prevent us from touching the NMI watchdog in the way
> > that we normally do and thus we need an extra one here?
> >
> > ...but, in that case, I think the old code was _wrong_ and that the
> > intention was that the touch_nmi_watchdog() should only be if "verose
> > > 1" as the indentation implied. There doesn't feel like a reason to
> > touch the watchdog if we're not doing anything slow.
>
> No idea. It was like this in Jason's original version from 2008.

Yeah, I noticed the same. I'd be curious what Daneil (or Jason if he's
reading) says. I suppose i could always wait until your patch lands
and then send a new patch that puts it inside the "if" statement and
we can debate it then.

-Doug

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 17:04 ` Doug Anderson
  2021-03-22 18:18   ` Arnd Bergmann
@ 2021-03-22 19:23   ` Daniel Thompson
  1 sibling, 0 replies; 10+ messages in thread
From: Daniel Thompson @ 2021-03-22 19:23 UTC (permalink / raw)
  To: Doug Anderson
  Cc: Arnd Bergmann, Jason Wessel, Arnd Bergmann, Greg Kroah-Hartman,
	Ingo Molnar, Christian Brauner, kgdb-bugreport, LKML

On Mon, Mar 22, 2021 at 10:04:57AM -0700, Doug Anderson wrote:
> > +       if (verbose)                    \
> > +               printk(KERN_INFO a);    \
> > +} while (0)
> > +#define v2printk(a...) do {            \
> > +       if (verbose > 1)                \
> > +               printk(KERN_INFO a);    \
> > +       touch_nmi_watchdog();           \
> 
> This touch_nmi_watchdog() is pretty wonky. I guess maybe the
> assumption is that the "verbose level 2" prints are so chatty that the
> printing might prevent us from touching the NMI watchdog in the way
> that we normally do and thus we need an extra one here?
> 
> ...but, in that case, I think the old code was _wrong_ and that the
> intention was that the touch_nmi_watchdog() should only be if "verose
> > 1" as the indentation implied. There doesn't feel like a reason to
> touch the watchdog if we're not doing anything slow.

I'm not entirely sure I'd like to second guess the intent here. This
macro has been there since this file was introduced but several callers
have been added since then. We have to guess their intent too!

So, whilst I think you are probably right, v2printk() does appears in
places such as the single step test loop which makes it pretty
difficult to decide by inspection whether or not touching the watchdog
is useful.

It's something that could be further examined... but I'd be a little
reluctant to combine it directly with a whitespace change!


Daniel.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 19:22     ` Doug Anderson
@ 2021-03-22 20:14       ` Jason Wessel
  2021-03-22 20:26         ` Arnd Bergmann
  0 siblings, 1 reply; 10+ messages in thread
From: Jason Wessel @ 2021-03-22 20:14 UTC (permalink / raw)
  To: Doug Anderson, Arnd Bergmann
  Cc: Daniel Thompson, Greg Kroah-Hartman, Ingo Molnar,
	Christian Brauner, kgdb-bugreport, LKML



On 3/22/21 2:22 PM, Doug Anderson wrote:
> Hi,
> 
> On Mon, Mar 22, 2021 at 11:19 AM Arnd Bergmann <arnd@kernel.org> wrote:
>>
>> On Mon, Mar 22, 2021 at 6:07 PM Doug Anderson <dianders@chromium.org> wrote:
>>> On Mon, Mar 22, 2021 at 9:43 AM Arnd Bergmann <arnd@kernel.org> wrote:
>>>>
>>>> -#define v1printk(a...) do { \
>>>> -       if (verbose) \
>>>> -               printk(KERN_INFO a); \
>>>> -       } while (0)
>>>> -#define v2printk(a...) do { \
>>>> -       if (verbose > 1) \
>>>> -               printk(KERN_INFO a); \
>>>> -               touch_nmi_watchdog();   \
>>>> -       } while (0)
>>>> -#define eprintk(a...) do { \
>>>> -               printk(KERN_ERR a); \
>>>> -               WARN_ON(1); \
>>>> -       } while (0)
>>>> +#define v1printk(a...) do {            \
>>>
>>> nit: In addition to the indentation change you're also lining up the
>>> backslashes. Is that just personal preference, or is there some
>>> official recommendation in the kernel? I don't really have a strong
>>> opinion either way (IMO each style has its advantages).
>>
>> I don't think there is an official recommendation, I just think the
>> style is more common, and it helped my figure out what the
>> indentation should look like in this case.
> 
> OK, makes sense. I just wasn't sure if there was some standard that I
> wasn't aware of. Given that you have to touch all these lines anyway
> then making them all pretty like this seems fine to me.
> 
> 
>>>> +       if (verbose)                    \
>>>> +               printk(KERN_INFO a);    \
>>>> +} while (0)
>>>> +#define v2printk(a...) do {            \
>>>> +       if (verbose > 1)                \
>>>> +               printk(KERN_INFO a);    \
>>>> +       touch_nmi_watchdog();           \
>>>
>>> This touch_nmi_watchdog() is pretty wonky. I guess maybe the
>>> assumption is that the "verbose level 2" prints are so chatty that the
>>> printing might prevent us from touching the NMI watchdog in the way
>>> that we normally do and thus we need an extra one here?
>>>
>>> ...but, in that case, I think the old code was _wrong_ and that the
>>> intention was that the touch_nmi_watchdog() should only be if "verose
>>>> 1" as the indentation implied. There doesn't feel like a reason to
>>> touch the watchdog if we're not doing anything slow.
>>
>> No idea. It was like this in Jason's original version from 2008.
> 
> Yeah, I noticed the same. I'd be curious what Daneil (or Jason if he's
> reading) says. I suppose i could always wait until your patch lands
> and then send a new patch that puts it inside the "if" statement and
> we can debate it then.
> 


The original board this was developed with was a 32bit eeepc.

The intent was that when v2printk() was called for a verbose > 1
condition the touch_nmi_watchdog() was called.   The test case
where a whole lot of single steps are executed sequentially was not
letting the watchdog get reset by the normal kernel routine.
The serial port was so slow it was pretty easy to hit this problem
and it would just power cycle itself.

The original intent would have bee:

#define v2printk(a...) do {		\
	if (verbose > 1) {		\
		printk(KERN_INFO a);	\
		touch_nmi_watchdog();	\
	}				\
} while (0)


I'd guess this probably not the first time gcc-11 is finding brace
imbalances.

Cheers,
Jason.


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 20:14       ` Jason Wessel
@ 2021-03-22 20:26         ` Arnd Bergmann
  0 siblings, 0 replies; 10+ messages in thread
From: Arnd Bergmann @ 2021-03-22 20:26 UTC (permalink / raw)
  To: Jason Wessel
  Cc: Doug Anderson, Daniel Thompson, Greg Kroah-Hartman, Ingo Molnar,
	Christian Brauner, kgdb-bugreport, LKML

On Mon, Mar 22, 2021 at 9:14 PM Jason Wessel <jason.wessel@windriver.com> wrote:
>
> The original board this was developed with was a 32bit eeepc.
>
> The intent was that when v2printk() was called for a verbose > 1
> condition the touch_nmi_watchdog() was called.   The test case
> where a whole lot of single steps are executed sequentially was not
> letting the watchdog get reset by the normal kernel routine.
> The serial port was so slow it was pretty easy to hit this problem
> and it would just power cycle itself.
>
> The original intent would have bee:
>
> #define v2printk(a...) do {             \
>         if (verbose > 1) {              \
>                 printk(KERN_INFO a);    \
>                 touch_nmi_watchdog();   \
>         }                               \
> } while (0)

Ok, thanks for sharing how the code was intended to work. I'll let
you all come up with a decision on what should be done about it
now, I'm happy to send one or two patches to address both the
compiler warning, and the original mistake.

My feeling is it would be best to address the warning first,
pretty much with the patch I sent here, and to change the
behavior as a second patch.

That way the gcc-11 warning can be silenced in stable kernels by
backporting the first patch, while the second patch can be tried
out in new kernels first and might not get backported because the
existing behavior is not harmful.

> I'd guess this probably not the first time gcc-11 is finding brace
> imbalances.

There were only a handful of new -Wmisleading-indentation warnings
for gcc-11, the older compilers already caught every instance in normal
functions, while gcc-11 improved on finding them in macros as well.

       Arnd

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] kgdb: fix gcc-11 warning on indentation
  2021-03-22 18:03   ` Arnd Bergmann
@ 2021-03-23  7:28     ` Greg Kroah-Hartman
  0 siblings, 0 replies; 10+ messages in thread
From: Greg Kroah-Hartman @ 2021-03-23  7:28 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Daniel Thompson, Jason Wessel, Ingo Molnar, Douglas Anderson,
	Christian Brauner, kgdb-bugreport, Linux Kernel Mailing List

On Mon, Mar 22, 2021 at 07:03:45PM +0100, Arnd Bergmann wrote:
> On Mon, Mar 22, 2021 at 6:03 PM Daniel Thompson
> <daniel.thompson@linaro.org> wrote:
> >
> >
> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> >
> > Which tree do you want to merge this one though? I've got nothing else
> > pending for this file so I am very relaxed about the route...
> 
> I don't plan to merge any of the build fixes through my own tree.
> If you don't have anything else, maybe Greg can pick it up
> in the char-misc tree.

Yes, I can take it, thanks.

greg k-h

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-03-23  7:29 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-22 16:43 [PATCH] kgdb: fix gcc-11 warning on indentation Arnd Bergmann
2021-03-22 17:03 ` Daniel Thompson
2021-03-22 18:03   ` Arnd Bergmann
2021-03-23  7:28     ` Greg Kroah-Hartman
2021-03-22 17:04 ` Doug Anderson
2021-03-22 18:18   ` Arnd Bergmann
2021-03-22 19:22     ` Doug Anderson
2021-03-22 20:14       ` Jason Wessel
2021-03-22 20:26         ` Arnd Bergmann
2021-03-22 19:23   ` Daniel Thompson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.