From: Oliver Hartkopp <socketcan@hartkopp.net>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Ram Pai <linuxram@us.ibm.com>,
torvalds@linux-foundation.org, linux-pci@vger.kernel.org,
linux-kernel@vger.kernel.org, yinghai@kernel.org,
bhutchings@solarflare.com, bhelgaas@google.com,
linux@dominikbrodowski.net
Subject: Re: [PATCH 3/5 v2] PCI: make SRIOV resources optional
Date: Thu, 07 Jul 2011 17:34:45 +0200 [thread overview]
Message-ID: <4E15D215.2010203@hartkopp.net> (raw)
In-Reply-To: <20110706104848.7e9d21fc@jbarnes-desktop>
On 06.07.2011 19:48, Jesse Barnes wrote:
> On Fri, 01 Jul 2011 08:01:33 +0200
> Oliver Hartkopp <socketcan@hartkopp.net> wrote:
>
>> On 01.07.2011 01:47, Ram Pai wrote:
>>> From: Yinghai Lu <yinghai@kernel.org>
>>>
>>> From: Yinghai Lu <yinghai@kernel.org>
>>>
>>> Allocate resources to SRIOV BARs only after all other required
>>> resource-requests are satisfied. Dont retry if resource allocation for SRIOV
>>> BARs fail.
>>>
>>> Signed-off-by: Ram Pai <linuxram@us.ibm.com>
>>> Signed-off-by: Yinghai Lu <yinghai@kernel.org>
>>
>> Tested-by: Oliver Hartkopp <socketcan@hartkopp.net>
>>
>> I applied the whole patchset and it looks very similar to the dmsg output
>> before the problematic commit that caused the regression.
>
> Ok Oliver, can I ask you to test one more tree? We'll shoot for
> including Ram's patches in -next, but I've just included his other
> patch to disable the new realloc code by default for the current
> kernel. Can you give my for-linus tree a try and make sure it gives
> you back the old behavior? Thanks a lot for your diligent testing so
> far.
I can confirm that it has the exact pci assignments as it had before.
I'll send you the dmesg output in a separate mail.
Best regards,
Oliver
>
> git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/pci-2.6 is the
> tree, then check out the "for-linus" branch.
>
> Thanks,
next prev parent reply other threads:[~2011-07-07 15:34 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-30 23:47 [PATCH 0/5 v2] PCI: fix cardbus and sriov regressions Ram Pai
2011-06-30 23:47 ` [PATCH 1/5 v2] PCI: honor child buses optional size in hot plug configuration Ram Pai
2011-06-30 23:47 ` [PATCH 2/5 v2] PCI : ability to relocate assigned pci-resources Ram Pai
2011-06-30 23:47 ` [PATCH 3/5 v2] PCI: make SRIOV resources optional Ram Pai
2011-07-01 6:01 ` Oliver Hartkopp
2011-07-06 17:48 ` Jesse Barnes
2011-07-07 15:34 ` Oliver Hartkopp [this message]
2011-06-30 23:47 ` [PATCH 4/5 v2] PCI: make cardbus-bridge " Ram Pai
2011-06-30 23:47 ` [PATCH 5/5 v2] PCI: code and terminology cleanup Ram Pai
2011-07-01 23:07 ` [PATCH 0/5 v2] PCI: fix cardbus and sriov regressions Ben Hutchings
2011-07-02 13:04 ` Ram Pai
2011-07-04 23:35 ` Ben Hutchings
2011-07-03 21:30 ` Linus Torvalds
2011-07-04 3:55 ` Harry Wei
2011-07-06 8:53 ` Ram Pai
2011-07-06 17:46 ` Jesse Barnes
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=4E15D215.2010203@hartkopp.net \
--to=socketcan@hartkopp.net \
--cc=bhelgaas@google.com \
--cc=bhutchings@solarflare.com \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux@dominikbrodowski.net \
--cc=linuxram@us.ibm.com \
--cc=torvalds@linux-foundation.org \
--cc=yinghai@kernel.org \
/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