netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kleber Sacilotto de Souza <klebers@linux.vnet.ibm.com>
To: David Miller <davem@davemloft.net>
Cc: ogerlitz@mellanox.com, netdev@vger.kernel.org,
	jackm@dev.mellanox.co.il, yevgenyp@mellanox.co.il,
	cascardo@linux.vnet.ibm.com, brking@linux.vnet.ibm.com,
	shlomop@mellanox.com
Subject: Re: [PATCH] mlx4: Add support for EEH error recovery
Date: Mon, 23 Jul 2012 10:18:41 -0300	[thread overview]
Message-ID: <500D4F31.9020408@linux.vnet.ibm.com> (raw)
In-Reply-To: <20120722.171553.2139258607165498367.davem@davemloft.net>

On 07/22/2012 09:15 PM, David Miller wrote:

> From: Or Gerlitz <ogerlitz@mellanox.com>
> Date: Sun, 22 Jul 2012 13:26:32 +0300
> 
>> is there anything in the code you added which maybe implicitly
>> assumes PPC arch?
> 
> He implemented support for a standard PCI API in the kernel, he
> happened to test it on a particular platform, and I think that's
> the long and short of it.
> 


Exactly. The callbacks implemented are from standard PCI error recovery
(Documentation/PCI/pci-error-recovery.txt) and the changes doesn't
assume any platform in specific. The code was tested only on powerpc
systems since I don't have any mlx4 card on other platforms, however,
these changes shouldn't make the error recover any worse than the
current state.

-- 
Kleber Sacilotto de Souza
IBM Linux Technology Center

  reply	other threads:[~2012-07-23 13:18 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-20 19:55 [PATCH] mlx4: Add support for EEH error recovery Kleber Sacilotto de Souza
2012-07-22 10:29 ` Or Gerlitz
     [not found] ` <500BD558.2060803@mellanox.com>
2012-07-23  0:15   ` David Miller
2012-07-23 13:18     ` Kleber Sacilotto de Souza [this message]
2012-07-23 13:45       ` Or Gerlitz
2012-07-23 18:12         ` Kleber Sacilotto de Souza
2012-07-23 20:53           ` Kleber Sacilotto de Souza
2012-07-23 21:26             ` Or Gerlitz
2012-07-23 21:34               ` David Miller
2012-07-23 21:42                 ` Or Gerlitz
2012-07-23 21:44                   ` David Miller
2012-07-23 22:02                     ` Or Gerlitz
2012-07-23 22:21                       ` David Miller
2012-07-24 13:12               ` Kleber Sacilotto de Souza
2012-07-24 17:09                 ` Shlomo Pongartz
2012-07-24 17:35                   ` Kleber Sacilotto de Souza
2012-07-24 18:08                     ` Thadeu Lima de Souza Cascardo
2012-07-24 18:35                       ` Shlomo Pongratz
2012-07-24 18:39                       ` Shlomo Pongratz
2012-07-24 21:03 ` David Miller
2012-07-24 22:30   ` Or Gerlitz
2012-07-25 14:38     ` Shlomo Pongartz
     [not found]     ` <5010070B.5040405@mellanox.com>
2012-07-25 15:02       ` Kleber Sacilotto de Souza
2012-07-25 22:19       ` David Miller

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=500D4F31.9020408@linux.vnet.ibm.com \
    --to=klebers@linux.vnet.ibm.com \
    --cc=brking@linux.vnet.ibm.com \
    --cc=cascardo@linux.vnet.ibm.com \
    --cc=davem@davemloft.net \
    --cc=jackm@dev.mellanox.co.il \
    --cc=netdev@vger.kernel.org \
    --cc=ogerlitz@mellanox.com \
    --cc=shlomop@mellanox.com \
    --cc=yevgenyp@mellanox.co.il \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).