xen-devel
[Xen-devel] Re: [PATCH] watchdog, SP5100: Check if firmware has set corr
To: |
Yinghai Lu <yinghai@xxxxxxxxxx> |
Subject: |
[Xen-devel] Re: [PATCH] watchdog, SP5100: Check if firmware has set correct value in tcobase. |
From: |
Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> |
Date: |
Wed, 16 Mar 2011 20:00:35 -0400 |
Cc: |
Priyanka Gupta <priyankag@xxxxxxxxxx>, Jeremy Fitzhardinge <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, linux-watchdog@xxxxxxxxxxxxxxx, Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Mike Waychison <mikew@xxxxxxxxxx>, Wim Van Sebroeck <wim@xxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx> |
Delivery-date: |
Wed, 16 Mar 2011 17:09:47 -0700 |
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxxx |
In-reply-to: |
<4D812919.30706@xxxxxxxxxx> |
List-help: |
<mailto:xen-devel-request@lists.xensource.com?subject=help> |
List-id: |
Xen developer discussion <xen-devel.lists.xensource.com> |
List-post: |
<mailto:xen-devel@lists.xensource.com> |
List-subscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
List-unsubscribe: |
<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
References: |
<alpine.DEB.2.00.1103111201470.2968@kaball-desktop> <20110311222129.GA3168@xxxxxxxxxxxx> <alpine.DEB.2.00.1103161150010.3382@kaball-desktop> <alpine.DEB.2.00.1103161442210.3382@kaball-desktop> <4D80F992.10603@xxxxxxxxxx> <alpine.DEB.2.00.1103161757040.3382@kaball-desktop> <4D812180.5030102@xxxxxxxxxx> <AANLkTimNva3c_-kGA+JLCPZgrKAbwqLKaN559cwTh7cq@xxxxxxxxxxxxxx> <4D812919.30706@xxxxxxxxxx> |
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Wed, Mar 16, 2011 at 02:18:17PM -0700, Yinghai Lu wrote:
>
>
> Stefano found SP5100 TCO watchdog driver using wrong address.
>
> [ 9.148536] SP5100 TCO timer: SP5100 TCO WatchDog Timer Driver v0.01
> [ 9.148628] DEBUG __ioremap_caller WARNING address=b8fe00 size=8 valid=1
> reserved=1
>
> and e820 said that range is RAM.
>
> We should check if we can use that reading out. BIOS could just program wrong
> address there.
>
> -v2: Mike pointed out one path need one release.
>
> Reported-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> Signed-off-by:Yinghai Lu <yinghai@xxxxxxxxxx>
> Acked-by: Mike Waychison <mikew@xxxxxxxxxx>
Tested-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Stefano,
this fixes my bootup issues with your:
xen: set max_pfn_mapped to the last pfn mapped
patch. Will try the full patchset tomorrow.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- [Xen-devel] [GIT PULL tip/x86/mm] xen/x86 fixes, Stefano Stabellini
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Konrad Rzeszutek Wilk
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Stefano Stabellini
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Stefano Stabellini
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Yinghai Lu
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Stefano Stabellini
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes ===> fix sp5100_tco mmio checking., Yinghai Lu
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes ===> fix sp5100_tco mmio checking., Mike Waychison
- [Xen-devel] [PATCH] watchdog, SP5100: Check if firmware has set correct value in tcobase., Yinghai Lu
- [Xen-devel] Re: [PATCH] watchdog, SP5100: Check if firmware has set correct value in tcobase.,
Konrad Rzeszutek Wilk <=
- [Xen-devel] Re: [PATCH] watchdog, SP5100: Check if firmware has set correct value in tcobase., Konrad Rzeszutek Wilk
- [Xen-devel] [PATCH -v3] watchdog, SP5100: Check if firmware has set correct value in tcobase., Yinghai Lu
- Re: [Xen-devel] [PATCH -v3] watchdog, SP5100: Check if firmware has set correct value in tcobase., Konrad Rzeszutek Wilk
- Re: [Xen-devel] [PATCH -v3] watchdog, SP5100: Check if firmware has set correct value in tcobase., Yinghai Lu
- [Xen-devel] Re: [PATCH] watchdog, SP5100: Check if firmware has set correct value in tcobase., Stefano Stabellini
- [Xen-devel] [PATCH -v3 -resend] watchdog, SP5100: Check if firmware has set correct value in tcobase., Yinghai Lu
- [Xen-devel] Re: [PATCH -v3 -resend] watchdog, SP5100: Check if firmware has set correct value in tcobase., Wim Van Sebroeck
- [Xen-devel] Re: [PATCH -v3 -resend] watchdog, SP5100: Check if firmware has set correct value in tcobase., Yinghai Lu
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Stefano Stabellini
- [Xen-devel] Re: [GIT PULL tip/x86/mm] xen/x86 fixes, Yinghai Lu
|
|
|