LinuxLists.cc - The robot is broken (was: Re: [PATCH 1/3] USB: serial: return errors from break handling)

2023-06-04 12:59:01

Subject: The robot is broken (was: Re: [PATCH 1/3] USB: serial: return errors from break handling)

On Sat, Jun 03, 2023 at 10:40:42AM +0800, kernel test robot wrote:
> BCC: [email protected]
> CC: [email protected]
> In-Reply-To: <[email protected]>
> References: <[email protected]>
> TO: Johan Hovold <[email protected]>
> TO: Johan Hovold <[email protected]>
> CC: "Greg Kroah-Hartman" <[email protected]>
> CC: [email protected]
> CC: [email protected]
> CC: Corey Minyard <[email protected]>
>
> Hi Johan,
>
> kernel test robot noticed the following build warnings:

> New smatch warnings:
> drivers/usb/serial/io_edgeport.c:1601 edge_break() error: uninitialized symbol 'status'.

Thanks for the report. Now fixed up in v2:

https://lore.kernel.org/lkml/[email protected]

The robot currently appears to be broken though as this report was never
sent to me or the relevant mailing lists (and I just happened to see it
by chance).

Looks like there's a stray newline in the output of your scripts which
causes the mail headers above to be included in the body instead.

Johan

2023-06-05 04:49:08

by Philip Li

[permalink] [raw]

Subject: Re: The robot is broken (was: Re: [PATCH 1/3] USB: serial: return errors from break handling)

On Sun, Jun 04, 2023 at 02:41:27PM +0200, Johan Hovold wrote:
> On Sat, Jun 03, 2023 at 10:40:42AM +0800, kernel test robot wrote:
> > BCC: [email protected]
> > CC: [email protected]
> > In-Reply-To: <[email protected]>
> > References: <[email protected]>
> > TO: Johan Hovold <[email protected]>
> > TO: Johan Hovold <[email protected]>
> > CC: "Greg Kroah-Hartman" <[email protected]>
> > CC: [email protected]
> > CC: [email protected]
> > CC: Corey Minyard <[email protected]>
> >
> > Hi Johan,
> >
> > kernel test robot noticed the following build warnings:
>
> > New smatch warnings:
> > drivers/usb/serial/io_edgeport.c:1601 edge_break() error: uninitialized symbol 'status'.
>
> Thanks for the report. Now fixed up in v2:
>
> https://lore.kernel.org/lkml/[email protected]
>
> The robot currently appears to be broken though as this report was never
> sent to me or the relevant mailing lists (and I just happened to see it
> by chance).

Thanks a lot Johan for checking this. For now, the bot has 2 similar mailing lists,
one is [email protected], which contains the high confidence report that
is sent directly to author and related CC. Another one is this [email protected],
that we will send low confidence issues here and they will only be really reported
out after manual check. So we can reduce false positives.

But sometimes, we may miss some reports of real issues, sorry about this.

>
> Looks like there's a stray newline in the output of your scripts which
> causes the mail headers above to be included in the body instead.

Thanks, yes, this is by design, when the report is confirmed, it helps to send
out the report to actual users by removing this extra line.

>
> Johan
>

2023-06-05 07:02:26

by Johan Hovold

[permalink] [raw]

Subject: Re: The robot is broken (was: Re: [PATCH 1/3] USB: serial: return errors from break handling)

On Mon, Jun 05, 2023 at 12:30:58PM +0800, Philip Li wrote:
> On Sun, Jun 04, 2023 at 02:41:27PM +0200, Johan Hovold wrote:

> > The robot currently appears to be broken though as this report was never
> > sent to me or the relevant mailing lists (and I just happened to see it
> > by chance).
>
> Thanks a lot Johan for checking this. For now, the bot has 2 similar mailing lists,
> one is [email protected], which contains the high confidence report that
> is sent directly to author and related CC. Another one is this [email protected],
> that we will send low confidence issues here and they will only be really reported
> out after manual check. So we can reduce false positives.

Ah, thanks for the explanation and sorry about the noise.

Johan

2023-06-05 07:54:23

by Dan Carpenter

[permalink] [raw]