From: Jarkko Nikula <jarkko.nikula@linux.intel.com>
To: Arnd Bergmann <arnd@arndb.de>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
x86@kernel.org
Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Geert Uytterhoeven" <geert@linux-m68k.org>,
"Alexandre Belloni" <alexandre.belloni@free-electrons.com>,
"Alessandro Zummo" <a.zummo@towertech.it>,
rtc-linux@googlegroups.com, "Borislav Petkov" <bp@suse.de>,
linux-kernel@vger.kernel.org
Subject: [rtc-linux] Re: [PATCH] x86/hpet: fix typo from rtc cleanup
Date: Wed, 10 Aug 2016 11:22:46 +0300 [thread overview]
Message-ID: <adba492a-ad23-1376-4383-b819e43f786e@linux.intel.com> (raw)
In-Reply-To: <20160809195528.1604312-1-arnd@arndb.de>
Hi
On 08/09/2016 10:54 PM, Arnd Bergmann wrote:
> Ville Syrj=C3=A4l=C3=A4 reports "The first time I run hwclock after reboo=
ting
> I get this:
>
> open("/dev/rtc", O_RDONLY) =3D 3
> ioctl(3, PHN_SET_REGS or RTC_UIE_ON, 0) =3D 0
> select(4, [3], NULL, NULL, {10, 0}) =3D 0 (Timeout)
> ioctl(3, PHN_NOT_OH or RTC_UIE_OFF, 0) =3D 0
> close(3) =3D 0
>
> On all subsequent runs I get this:
> open("/dev/rtc", O_RDONLY) =3D 3
> ioctl(3, PHN_SET_REGS or RTC_UIE_ON, 0) =3D -1 EINVAL (Invalid argument)
> ioctl(3, RTC_RD_TIME, 0x7ffd76b3ae70) =3D -1 EINVAL (Invalid argument)
> close(3) =3D 0"
>
> This was caused by a stupid typo in a patch that should have been
> a simple rename to move around contents of a header file, but
> accidentally wrote zeroes into the rtc rather than reading from
> it.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Reported-by: Ville Syrj=C3=A4l=C3=A4 <ville.syrjala@linux.intel.com>
> Tested-by: Ville Syrj=C3=A4l=C3=A4 <ville.syrjala@linux.intel.com>
> Fixes: 463a86304cae ("char/genrtc: x86: remove remnants of asm/rtc.h")
> ---
> arch/x86/kernel/hpet.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
I started seeing system reboots and lockups during kernel boot on a=20
prototype HW this week using v4.8-rc1 and bisected to this same commit.
This patch fixes the issue.
Tested-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
--=20
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
---=20
You received this message because you are subscribed to the Google Groups "=
rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an e=
mail to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
WARNING: multiple messages have this Message-ID (diff)
From: Jarkko Nikula <jarkko.nikula@linux.intel.com>
To: Arnd Bergmann <arnd@arndb.de>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
x86@kernel.org
Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Geert Uytterhoeven" <geert@linux-m68k.org>,
"Alexandre Belloni" <alexandre.belloni@free-electrons.com>,
"Alessandro Zummo" <a.zummo@towertech.it>,
rtc-linux@googlegroups.com, "Borislav Petkov" <bp@suse.de>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] x86/hpet: fix typo from rtc cleanup
Date: Wed, 10 Aug 2016 11:22:46 +0300 [thread overview]
Message-ID: <adba492a-ad23-1376-4383-b819e43f786e@linux.intel.com> (raw)
In-Reply-To: <20160809195528.1604312-1-arnd@arndb.de>
Hi
On 08/09/2016 10:54 PM, Arnd Bergmann wrote:
> Ville Syrjälä reports "The first time I run hwclock after rebooting
> I get this:
>
> open("/dev/rtc", O_RDONLY) = 3
> ioctl(3, PHN_SET_REGS or RTC_UIE_ON, 0) = 0
> select(4, [3], NULL, NULL, {10, 0}) = 0 (Timeout)
> ioctl(3, PHN_NOT_OH or RTC_UIE_OFF, 0) = 0
> close(3) = 0
>
> On all subsequent runs I get this:
> open("/dev/rtc", O_RDONLY) = 3
> ioctl(3, PHN_SET_REGS or RTC_UIE_ON, 0) = -1 EINVAL (Invalid argument)
> ioctl(3, RTC_RD_TIME, 0x7ffd76b3ae70) = -1 EINVAL (Invalid argument)
> close(3) = 0"
>
> This was caused by a stupid typo in a patch that should have been
> a simple rename to move around contents of a header file, but
> accidentally wrote zeroes into the rtc rather than reading from
> it.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Reported-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Tested-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Fixes: 463a86304cae ("char/genrtc: x86: remove remnants of asm/rtc.h")
> ---
> arch/x86/kernel/hpet.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
I started seeing system reboots and lockups during kernel boot on a
prototype HW this week using v4.8-rc1 and bisected to this same commit.
This patch fixes the issue.
Tested-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
next prev parent reply other threads:[~2016-08-10 8:22 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-09 19:54 [rtc-linux] [PATCH] x86/hpet: fix typo from rtc cleanup Arnd Bergmann
2016-08-09 19:54 ` Arnd Bergmann
2016-08-10 8:22 ` Jarkko Nikula [this message]
2016-08-10 8:22 ` Jarkko Nikula
2016-08-10 18:08 ` [tip:timers/urgent] x86/hpet: Fix /dev/rtc breakage caused by RTC cleanup tip-bot for Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=adba492a-ad23-1376-4383-b819e43f786e@linux.intel.com \
--to=jarkko.nikula@linux.intel.com \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@free-electrons.com \
--cc=arnd@arndb.de \
--cc=bp@suse.de \
--cc=geert@linux-m68k.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=rtc-linux@googlegroups.com \
--cc=tglx@linutronix.de \
--cc=ville.syrjala@linux.intel.com \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.