Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1726420 > unrolled thread
| Started by | Thomas Gleixner <tglx@linutronix.de> |
|---|---|
| First post | 2017-09-05 09:20 +0200 |
| Last post | 2017-09-05 21:40 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: 915f3e3f76 ("mac80211_hwsim: Replace bogus hrtimer clockid"): BUG: kernel reboot-without-warning in test stage Thomas Gleixner <tglx@linutronix.de> - 2017-09-05 09:20 +0200
Re: 915f3e3f76 ("mac80211_hwsim: Replace bogus hrtimer clockid"): BUG: kernel reboot-without-warning in test stage Sebastian Andrzej Siewior <sebastian@breakpoint.cc> - 2017-09-05 18:50 +0200
Re: 915f3e3f76 ("mac80211_hwsim: Replace bogus hrtimer clockid"): BUG: kernel reboot-without-warning in test stage Thomas Gleixner <tglx@linutronix.de> - 2017-09-05 21:40 +0200
| From | Thomas Gleixner <tglx@linutronix.de> |
|---|---|
| Date | 2017-09-05 09:20 +0200 |
| Subject | Re: 915f3e3f76 ("mac80211_hwsim: Replace bogus hrtimer clockid"): BUG: kernel reboot-without-warning in test stage |
| Message-ID | <umgjq-87w-67@gated-at.bofh.it> |
On Tue, 5 Sep 2017, kernel test robot wrote: > Greetings, > > 0day kernel testing robot got the below dmesg and the first bad commit is > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > commit 915f3e3f76c05b2da93c4cc278eebc2d9219d9f4 > Author: Thomas Gleixner <tglx@linutronix.de> > AuthorDate: Sat Feb 25 11:27:37 2017 +0100 > Commit: Linus Torvalds <torvalds@linux-foundation.org> > CommitDate: Sat Feb 25 09:48:16 2017 -0800 > > mac80211_hwsim: Replace bogus hrtimer clockid > > mac80211_hwsim initializes a hrtimer with clockid CLOCK_MONOTONIC_RAW. > That's not supported. > > Use CLOCK_MONOTNIC instead. Sorry, no. That bisect is completely bogus. The commit in question merily replaces the unsupported clockid with a valid one. Thanks, tglx
[toc] | [next] | [standalone]
| From | Sebastian Andrzej Siewior <sebastian@breakpoint.cc> |
|---|---|
| Date | 2017-09-05 18:50 +0200 |
| Message-ID | <umpd1-5fU-33@gated-at.bofh.it> |
| In reply to | #1726420 |
On 2017-09-05 09:12:40 [+0200], Thomas Gleixner wrote: > Sorry, no. That bisect is completely bogus. The commit in question merily > replaces the unsupported clockid with a valid one. The bisect is correct. It just has problems to express itself properly. So the table says: | WARNING:at_kernel/time/hrtimer.c:#hrtimer_init | 7 | | | | | BUG:kernel_reboot-without-warning_in_test_stage | 0 | 12 | 6 | 6 | which means _before_ your commit it counted a warning in hrtimer_init() (an unsupported clock id was used). With your commit, the warning was gone and I *think* the userland then printed "BUG:kernel_reboot-without-warning_in_test_stage" because it had no warning. It seems that the bot learned to live with that warning which was around for more than three years. Now that you removed it, it seems to be a mistake to do so because nobody complained about it so far. > Thanks, > > tglx Sebastian
[toc] | [prev] | [next] | [standalone]
| From | Thomas Gleixner <tglx@linutronix.de> |
|---|---|
| Date | 2017-09-05 21:40 +0200 |
| Message-ID | <umrRv-76q-11@gated-at.bofh.it> |
| In reply to | #1726860 |
On Tue, 5 Sep 2017, Sebastian Andrzej Siewior wrote:
> On 2017-09-05 09:12:40 [+0200], Thomas Gleixner wrote:
> > Sorry, no. That bisect is completely bogus. The commit in question merily
> > replaces the unsupported clockid with a valid one.
>
> The bisect is correct. It just has problems to express itself properly. So
> the table says:
>
> | WARNING:at_kernel/time/hrtimer.c:#hrtimer_init | 7 | | | |
> | BUG:kernel_reboot-without-warning_in_test_stage | 0 | 12 | 6 | 6 |
>
> which means _before_ your commit it counted a warning in hrtimer_init()
> (an unsupported clock id was used). With your commit, the warning was
> gone and I *think* the userland then printed
> "BUG:kernel_reboot-without-warning_in_test_stage" because it had no
> warning.
> It seems that the bot learned to live with that warning which was around
> for more than three years. Now that you removed it, it seems to be a
> mistake to do so because nobody complained about it so far.
Thanks for the translation. I'll never learn to decode these reports.
tglx
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web