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 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.