On 2/19/2010 1:54 PM, Benjamin Herrenschmidt wrote:
On Fri, 2010-02-19 at 14:43 -0200, Breno Leitao wrote:
Hi Ben,
I'd like to ask about this patch ? Should I re-submit ?
Thanks,
Breno Leitao wrote:
During a EEH recover, the pci_dev structure can be null, mainly if an
eeh event is detected
during recovery.
These patches supersede the previously submitted patch that implemented a fundamental reset bit field.
Please review and let me know of any concerns.
Signed-off-by: Mike Mason mm...@us.ibm.com
Signed-off-by: Richard Lary rl...@us.ibm.com
diff -uNrp a/include/linux/pci.h b
unrelated updates.
These patches supersede the previously submitted patch that implemented a fundamental reset bit field.
Please review and let me know of any concerns.
Signed-off-by: Mike Mason mm...@us.ibm.com
Signed-off-by: Richard Lary rl...@us.ibm.com
--- a/Documentation/PCI/pci-error
that implemented a
fundamental reset bit field.
Please review and let me know of any concerns.
Signed-off-by: Mike Mason mm...@us.ibm.com
Signed-off-by: Richard Lary rl...@us.ibm.com
diff -uNrp a/arch/powerpc/kernel/pci_64.c b/arch/powerpc/kernel/pci_64.c
--- a/arch/powerpc/kernel/pci_64.c 2009-07-13 14
Michael Ellerman wrote:
On Thu, 2009-07-16 at 09:33 -0700, Mike Mason wrote:
Michael Ellerman wrote:
On Wed, 2009-07-15 at 14:43 -0700, Mike Mason wrote:
This patch increments the device_node reference counter when an EEH
error occurs and decrements the counter when the event has been
handled
Michael Ellerman wrote:
On Wed, 2009-07-15 at 14:43 -0700, Mike Mason wrote:
This patch increments the device_node reference counter when an EEH
error occurs and decrements the counter when the event has been
handled. This is to prevent the device_node from being released until
This patch was simultaneously submitted to Red Hat for review. As a result of
that review, I'm withdrawing this patch and will submit a new version shortly.
Mike
Mike Mason wrote:
By default, EEH does what's known as a hot reset during error recovery
of a PCI Express device. We've found
callback.
Please review and let me know of any concerns.
Signed-off-by: Mike Mason mm...@us.ibm.com
diff -uNrp a/arch/powerpc/kernel/pci_64.c b/arch/powerpc/kernel/pci_64.c
--- a/arch/powerpc/kernel/pci_64.c 2009-07-13 14:25:24.0 -0700
+++ b/arch/powerpc/kernel/pci_64.c 2009-07-15 10
the device_node is
released too soon when an EEH event occurs during a dlpar remove, causing the
event handler to attempt to access bad memory locations.
Please review and let me know of any concerns.
Signed-off-by: Mike Mason mm...@us.ibm.com
--- a/arch/powerpc/platforms/pseries/eeh_event.c
only aware of one driver that has the requirement (qla2xxx). The
patch touches mostly EEH and pseries code, but does require a couple of minor
additions to the overall PCI error recovery framework.
Signed-off-by: Mike Mason mm...@us.ibm.com
--- a/arch/powerpc/include/asm/ppc-pci.h2009-06
Paul Mackerras wrote:
Mike Mason writes:
diff --git a/arch/powerpc/platforms/pseries/eeh_driver.c
b/arch/powerpc/platforms/pseries/eeh_driver.c
index 380420f..9a2a6e3 100644
--- a/arch/powerpc/platforms/pseries/eeh_driver.c
+++ b/arch/powerpc/platforms/pseries/eeh_driver.c
and should be accurately reflected by
setting dev-error_state to pci_channel_io_normal.
The current implementation of EEH driver does not do so and requires the
following patch to correct this deficiency.
Signed-off-by: Mike Mason mm...@us.ibm.com
diff --git a/arch/powerpc/platforms/pseries
Michael Ellerman wrote:
On Tue, 2009-02-10 at 13:12 -0800, Mike Mason wrote:
I'm resubmitting this patch with a couple changes
suggested by Michael Ellerman. 1) the new functions
should be static, and 2) some people may object to
including unrelated formating changes
in a different way than LSI/MSI interrupts. This patch
ensures only LSI interrupts are disabled/enabled.
Signed-off-by: Mike Mason mm...@us.ibm.com
Acked-by: Linas Vepstas linasveps...@gmail.com
--- arch/powerpc/platforms/pseries/eeh_driver.c-orig2009-02-10
07:12:31.0 -0800
+++ arch
into account that MSI-X interrupts are tracked
in a different way than LSI/MSI interrupts. This
patch ensures only LSI interrupts are disabled/enabled.
The patch also includes a couple minor formatting fixes.
Signed-off-by: Mike Mason mm...@us.ibm.com
--- linux-2.6.18.ppc64-orig/arch
, which at the very least should allow for a
more graceful shutdown. The patch also removes the msleep() within a
spinlock, which can lead to a deadlock and is not recommended.
Signed-off-by: Mike Mason [EMAIL PROTECTED]
Acked-by: Linas Vepstas [EMAIL PROTECTED]
--- powerpc.git/arch/powerpc
shutdown. The panic() is now wrapped in a DEBUG statement for development
purposes. The patch also removes the msleep() within a spinlock, which is not
allowed.
Signed-off-by: Mike Mason [EMAIL PROTECTED]
--- powerpc.git/arch/powerpc/platforms/pseries/eeh.c2008-07-18
08:51
Linas Vepstas wrote:
2008/7/7 Mike Mason [EMAIL PROTECTED]:
The following patch restores the PERR and SERR bits in the PCI
command register during an EEH device recovery.
We have found at least one case (an Agilent test card) where the
PERR/SERR bits are set to 1 by firmware at boot time
18 matches
Mail list logo