All of lore.kernel.org
 help / color / mirror / Atom feed
From: Casey Leedom <leedom@chelsio.com>
To: David Miller <davem@davemloft.net>
Cc: vipul@chelsio.com, linux-pci@vger.kernel.org,
	bhelgaas@google.com, tomreu@chelsio.com, divy@chelsio.com,
	dm@chelsio.com, nirranjan@chelsio.com, eilong@broadcom.com,
	netdev@vger.kernel.org
Subject: Re: [PATCH V2 0/4] Enable bus master till the FLR completes for Chelsio T4
Date: Wed, 07 Aug 2013 17:11:26 -0700	[thread overview]
Message-ID: <5202E22E.4070408@chelsio.com> (raw)
In-Reply-To: <20130807.170836.1214269360365516393.davem@davemloft.net>


On 08/07/13 17:08, David Miller wrote:
> From: Vipul Pandya <vipul@chelsio.com>
> Date: Tue,  6 Aug 2013 15:48:35 +0530
>
>> This patch series adds device specific reset method for Chelsio T4 adapters.
>>
>> The series also adds new routine, pci_wait_for_pending_transaction, to wait for
>> pending pci transaction. There are few places where waiting for pending pci
>> transaction code was duplicated. This patch series also replaces occurances of
>> duplicate code.
>>
>> This patch series is built against Bjorn Helgaas pci tree next branch. We would
>> like to merge this series via the same.
>>
>> I have Cced respective maintainers of the modified drivers. Kindly review and
>> let us know in case of any review comments.
> Please push this via the PCI tree, and for the networking bits you can add:
>
> Acked-by: David S. Miller <davem@davemloft.net>
>
> Thanks.

Thanks David.  I ~think~ that's covered and we've already gone through 
one round of reviews that Bjorn Helgaas graciously provided.

Note that this patch should also obviate the need for the patch which we 
attempted to get into the Chelsio T4 network driver cxgb4 several months 
ago which monitored the chip for a hang caused by KVM detaching a T4 
Virtual Function from a Virtual Machine.  You were unhappy with that 
patch and requested that we approach the problem differently.  Thanks 
for your push.

Casey

  reply	other threads:[~2013-08-08  0:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-06 10:18 [PATCH V2 0/4] Enable bus master till the FLR completes for Chelsio T4 Vipul Pandya
2013-08-06 10:18 ` [PATCH V2 1/4] pci: Add pci_wait_for_pending_transaction to wait for pending pci transaction Vipul Pandya
2013-08-06 10:18 ` [PATCH V2 2/4] pci: Enable bus master till the FLR completes Vipul Pandya
2013-08-06 10:18 ` [PATCH V2 3/4] bnx2x: Use pci_wait_for_pending_transaction instead of for loop Vipul Pandya
2013-08-06 11:12   ` Eilon Greenstein
2013-08-06 10:18 ` [PATCH V2 4/4] pci: " Vipul Pandya
2013-08-08  0:08 ` [PATCH V2 0/4] Enable bus master till the FLR completes for Chelsio T4 David Miller
2013-08-08  0:11   ` Casey Leedom [this message]
2013-08-12 21:17 ` Bjorn Helgaas
2013-08-13 21:53   ` Casey Leedom
2013-08-13 23:00     ` Bjorn Helgaas

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=5202E22E.4070408@chelsio.com \
    --to=leedom@chelsio.com \
    --cc=bhelgaas@google.com \
    --cc=davem@davemloft.net \
    --cc=divy@chelsio.com \
    --cc=dm@chelsio.com \
    --cc=eilong@broadcom.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=nirranjan@chelsio.com \
    --cc=tomreu@chelsio.com \
    --cc=vipul@chelsio.com \
    /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.