login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2010
»
November
»
11
Re: [PATCH] PCI hotplug: ibmphp: Add check to prevent reading beyond mapped area
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
[view in full thread]
From: Jesse Barnes
Subject:
Re: [PATCH] PCI hotplug: ibmphp: Add check to prevent reading beyond mapped area
Date: Thursday, November 11, 2010 - 10:09 am
On Mon, 08 Nov 2010 23:20:27 -0500 Steven Rostedt <rostedt@goodmis.org> wrote:
quoted text
> While testing various randconfigs with ktest.pl, I hit the following panic: > > BUG: unable to handle kernel paging request at f7e54b03 > IP: [<c0d63409>] ibmphp_access_ebda+0x101/0x19bb > > Adding printks, I found that the loop that reads the ebda blocks > can move out of the mapped section. > > ibmphp_access_ebda: start=f7e44c00 size=5120 end=f7e46000 > ibmphp_access_ebda: io_mem=f7e44d80 offset=384 > ibmphp_access_ebda: io_mem=f7e54b03 offset=65283 > > The start of the iomap was at f7e44c00 and had a size of 5120, > making the end f7e46000. We start with an offset of 0x180 or > 384, giving the first read at 0xf7e44d80. Reading that location > yields 65283, which is much bigger than the 5120 that was allocated > and makes the next read at f7e54b03 which is outside the mapped area. > > Perhaps this is a bug in the driver, or buggy hardware, but this patch > is more about not crashing my box on start up and just giving a warning > if it detects this error. > > This patch at least lets my box boot with just a warning. > > Cc: Jesse Barnes <jbarnes@virtuousgeek.org> > Cc: Chandru Siddalingappa <chandru@linux.vnet.ibm.com> > Signed-off-by: Steven Rostedt <rostedt@goodmis.org> > > --- > drivers/pci/hotplug/ibmphp_ebda.c | 6 ++++++ > 1 file changed, 6 insertions(+)
Applied to my for-linus branch, thanks. -- Jesse Barnes, Intel Open Source Technology Center --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
Messages in current thread:
[PATCH] PCI hotplug: ibmphp: Add check to prevent reading ...
, Steven Rostedt
, (Mon Nov 8, 9:20 pm)
Re: [PATCH] PCI hotplug: ibmphp: Add check to prevent read ...
, Jesse Barnes
, (Thu Nov 11, 10:09 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Mel Gorman
Re: [PATCH 1/4] vmstat: remove zone->lock from walk_zones_in_node
Guenter Roeck
Re: [lm-sensors] Location for thermal drivers
David Woodhouse
Re: RFC: Moving firmware blobs out of the kernel.
Siddha, Suresh B
Re: [PATCH 2.6.21 review I] [11/25] x86: default to physical mode on hotplug CPU k...
Peter Zijlstra
Re: [patch 4/6] mm: merge populate and nopage into fault (fixes nonlinear)
git-commits-head
:
Linux Kernel Mailing List
[MIPS] Fix potential latency problem due to non-atomic cpu_wait.
Linux Kernel Mailing List
USB: rename USB_SPEED_VARIABLE to USB_SPEED_WIRELESS
Linux Kernel Mailing List
lib/vsprintf.c: fix bug omitting minus sign of numbers (module_param)
Linux Kernel Mailing List
[Bluetooth] Initiate authentication during connection establishment
Linux Kernel Mailing List
[POWERPC] 4xx: Add ppc40x_defconfig
linux-netdev
:
MERCEDES
Your mail id has won 950,000.00 in the MERCEDES Benz Online Promo.for claims send:
David Miller
Re: [PATCH] xen/netfront: do not mark packets of length < MSS as GSO
David Miller
Re: skb_segment() questions
Shan Wei
[RFC PATCH net-next 2/5]IPv6:netfilter: Send an ICMPv6 "Fragment Reassembly Timeou...
Stanislaw Gruszka
[PATCH 1/4] bnx2x: use smp_mb() to keep ordering of read write operations
git
:
Nicolas Sebrecht
git-svn died of signal 11 (was "3 failures on test t9100 (svn)")
Junio C Hamano
Re: [PATCH 2/2] Add url.<base>.pushInsteadOf: URL rewriting for push only
Martin Langhoff
Re: [PATCH] GIT commit statistics.
Alexandre Julliard
[PATCH] gitweb: Put back shortlog instead of graphiclog in the project list.
Josh Triplett
[PATCH 2/2] Add url.<base>.pushInsteadOf: URL rewriting for push only
openbsd-misc
:
Taisto Qvist XX
Re: AMD GEODE LX-800 just works with kernel from install42.iso and kernelpanics wi...
Nico Meijer
Re: gOS Develop Kit with VIA pc-1 Processor Platform VIA C7-D
Andreas Bihlmaier
Re: jetway board sensors (Fintek F71805F)
admin
Drive a 2009 car from R799p/m
Antti Harri
Re: how to create a sha256 hash
Colocation donated by:
Syndicate