0-day bot a bit too verbose (was Re: [linux-review:Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139 1/1] drivers/hid/hid-led.c:60:3: error: expected specifier-qualifier-list before 'le16')

0-day bot a bit too verbose (was Re: [linux-review:Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139 1/1] drivers/hid/hid-led.c:60:3: error: expected specifier-qualifier-list before 'le16')

[Pavel Machek]

[-- Attachment #1: Type: text/plain, Size: 1185 bytes --]

Hi!

> tree:   https://github.com/0day-ci/linux Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139
> head:   596471d181ad324301428001db102c810df4a6de
> commit: 596471d181ad324301428001db102c810df4a6de [1/1] hid-led.c: remove unneccessary underscores
> config: openrisc-allmodconfig (attached as .config)
> compiler: or32-linux-gcc (GCC) 4.5.1-or32-1.0rc1
> reproduce:
>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 596471d181ad324301428001db102c810df4a6de
>         # save the attached .config to linux build tree
>         make.cross ARCH=openrisc 
> 
> All errors (new ones prefixed by >>):

I get it, it does not compile anywhere, because I made a mistake. Ok?

But I know now, and if it does not compile on x86, there's very very
little chance that it would compile on openrisc.

Can we stop the announcements after first or second failure?

Thanks,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

Re: 0-day bot a bit too verbose (was Re: [linux-review:Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139 1/1] drivers/hid/hid-led.c:60:3: error: expected specifier-qualifier-list before 'le16')

[Fengguang Wu]

Hi Pavel,

On Mon, Oct 03, 2016 at 01:22:10PM +0200, Pavel Machek wrote:
>Hi!
>
>> tree:   https://github.com/0day-ci/linux Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139
>> head:   596471d181ad324301428001db102c810df4a6de
>> commit: 596471d181ad324301428001db102c810df4a6de [1/1] hid-led.c: remove unneccessary underscores
>> config: openrisc-allmodconfig (attached as .config)
>> compiler: or32-linux-gcc (GCC) 4.5.1-or32-1.0rc1
>> reproduce:
>>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>>         chmod +x ~/bin/make.cross
>>         git checkout 596471d181ad324301428001db102c810df4a6de
>>         # save the attached .config to linux build tree
>>         make.cross ARCH=openrisc
>>
>> All errors (new ones prefixed by >>):
>
>I get it, it does not compile anywhere, because I made a mistake. Ok?
>
>But I know now, and if it does not compile on x86, there's very very
>little chance that it would compile on openrisc.
>
>Can we stop the announcements after first or second failure?

Sorry for the extra noises!

The build bot will stop complaining after 3 reports on the same
commit. Shall I further reduce the threshold to 1 or 2? It'll risk
hiding real different/independent errors though.

Thanks,
Fengguang

Re: 0-day bot a bit too verbose (was Re: [linux-review:Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139 1/1] drivers/hid/hid-led.c:60:3: error: expected specifier-qualifier-list before 'le16')

[Pavel Machek]

[-- Attachment #1: Type: text/plain, Size: 1708 bytes --]

On Mon 2016-10-03 21:52:38, Fengguang Wu wrote:
> Hi Pavel,
> 
> On Mon, Oct 03, 2016 at 01:22:10PM +0200, Pavel Machek wrote:
> >Hi!
> >
> >>tree:   https://github.com/0day-ci/linux Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139
> >>head:   596471d181ad324301428001db102c810df4a6de
> >>commit: 596471d181ad324301428001db102c810df4a6de [1/1] hid-led.c: remove unneccessary underscores
> >>config: openrisc-allmodconfig (attached as .config)
> >>compiler: or32-linux-gcc (GCC) 4.5.1-or32-1.0rc1
> >>reproduce:
> >>        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
> >>        chmod +x ~/bin/make.cross
> >>        git checkout 596471d181ad324301428001db102c810df4a6de
> >>        # save the attached .config to linux build tree
> >>        make.cross ARCH=openrisc
> >>
> >>All errors (new ones prefixed by >>):
> >
> >I get it, it does not compile anywhere, because I made a mistake. Ok?
> >
> >But I know now, and if it does not compile on x86, there's very very
> >little chance that it would compile on openrisc.
> >
> >Can we stop the announcements after first or second failure?
> 
> Sorry for the extra noises!
> 
> The build bot will stop complaining after 3 reports on the same
> commit. Shall I further reduce the threshold to 1 or 2? It'll risk
> hiding real different/independent errors though.

I'd reduce the threshold to 2. When I got third report, I thought "hmm
and now I'm going to get 500 more.".

Thanks,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

Re: [kbuild-all] 0-day bot a bit too verbose (was Re: [linux-review:Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139 1/1] drivers/hid/hid-led.c:60:3: error: expected specifier-qualifier-list before 'le16')

[Fengguang Wu]

Hi Pavel,

On Wed, Oct 19, 2016 at 12:37:12PM +0200, Pavel Machek wrote:
>On Mon 2016-10-03 21:52:38, Fengguang Wu wrote:
>> Hi Pavel,
>>
>> On Mon, Oct 03, 2016 at 01:22:10PM +0200, Pavel Machek wrote:
>> >Hi!
>> >
>> >>tree:   https://github.com/0day-ci/linux Pavel-Machek/hid-led-c-remove-unneccessary-underscores/20161003-162139
>> >>head:   596471d181ad324301428001db102c810df4a6de
>> >>commit: 596471d181ad324301428001db102c810df4a6de [1/1] hid-led.c: remove unneccessary underscores
>> >>config: openrisc-allmodconfig (attached as .config)
>> >>compiler: or32-linux-gcc (GCC) 4.5.1-or32-1.0rc1
>> >>reproduce:
>> >>        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>> >>        chmod +x ~/bin/make.cross
>> >>        git checkout 596471d181ad324301428001db102c810df4a6de
>> >>        # save the attached .config to linux build tree
>> >>        make.cross ARCH=openrisc
>> >>
>> >>All errors (new ones prefixed by >>):
>> >
>> >I get it, it does not compile anywhere, because I made a mistake. Ok?
>> >
>> >But I know now, and if it does not compile on x86, there's very very
>> >little chance that it would compile on openrisc.
>> >
>> >Can we stop the announcements after first or second failure?
>>
>> Sorry for the extra noises!
>>
>> The build bot will stop complaining after 3 reports on the same
>> commit. Shall I further reduce the threshold to 1 or 2? It'll risk
>> hiding real different/independent errors though.
>
>I'd reduce the threshold to 2. When I got third report, I thought "hmm
>and now I'm going to get 500 more.".

OK, reduced the threshold to 2. FYI we'll send another private report
hours later to summarize the various kind of errors if that threshold
is triggered.

Thanks,
Fengguang