Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-11 Thread Wei Yang
On Thu, Jul 09, 2015 at 10:49:06PM -0700, Yinghai Lu wrote: >On Thu, Jul 9, 2015 at 7:48 PM, Yinghai Lu wrote: >> On Thu, Jul 9, 2015 at 7:30 PM, Wei Yang wrote: >>> If you could update your for-pci-v4.3-next branch, that would be more >>> convenient for me to do the test. >> >> Just updated that

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-11 Thread Yinghai Lu
On Fri, Jul 10, 2015 at 5:03 PM, Wei Yang wrote: > On Thu, Jul 09, 2015 at 10:49:06PM -0700, Yinghai Lu wrote: > I have tested you latest branch with this one as the last commit: > > ec94cc7 PCI: Don't set flags to 0 when assign resource fail > > My P8 machine boots up. Good. > > Another issue

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-10 Thread Wei Yang
On Fri, Jul 10, 2015 at 05:42:10PM -0700, Yinghai Lu wrote: >On Fri, Jul 10, 2015 at 5:03 PM, Wei Yang wrote: >> On Thu, Jul 09, 2015 at 10:49:06PM -0700, Yinghai Lu wrote: > >> I have tested you latest branch with this one as the last commit: >> >> ec94cc7 PCI: Don't set flags to 0 when assign re

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-09 Thread Yinghai Lu
On Thu, Jul 9, 2015 at 7:48 PM, Yinghai Lu wrote: > On Thu, Jul 9, 2015 at 7:30 PM, Wei Yang wrote: >> If you could update your for-pci-v4.3-next branch, that would be more >> convenient for me to do the test. > > Just updated that branch, please check it. > just updated the branch again. If yo

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-09 Thread Yinghai Lu
On Thu, Jul 9, 2015 at 7:30 PM, Wei Yang wrote: > If you could update your for-pci-v4.3-next branch, that would be more > convenient for me to do the test. Just updated that branch, please check it. Thanks Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-09 Thread Wei Yang
On Thu, Jul 09, 2015 at 09:20:08AM -0700, Yinghai Lu wrote: >On Wed, Jul 8, 2015 at 11:04 PM, Wei Yang wrote: >> This one is on top of the last one ? or replace the last one? > >should be just before last one. I can't apply this on top of b0b9229 PCI, x86: Add pci=assign_pref_bars to re-allocat

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-09 Thread Yinghai Lu
On Wed, Jul 8, 2015 at 11:04 PM, Wei Yang wrote: > This one is on top of the last one ? or replace the last one? should be just before last one. Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-08 Thread Wei Yang
This one is on top of the last one ? or replace the last one? On Wed, Jul 08, 2015 at 10:01:47PM -0700, Yinghai Lu wrote: >On Wed, Jul 8, 2015 at 8:30 PM, Wei Yang wrote: >> Hi, Yinghai >> >> This patch may introduce some problem. >> >> On my P8 machine, after applying this patch, I see following

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-08 Thread Yinghai Lu
On Wed, Jul 8, 2015 at 8:30 PM, Wei Yang wrote: > Hi, Yinghai > > This patch may introduce some problem. > > On my P8 machine, after applying this patch, I see following error: > > [0.589948] pnv_ioda_setup_pe_seg: trigger IO SEG 0 > [0.589992] pnv_ioda_setup_pe_seg: res[io 0x1000-0x3fff]

Re: [PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-08 Thread Wei Yang
Hi, Yinghai This patch may introduce some problem. On my P8 machine, after applying this patch, I see following error: [0.589948] pnv_ioda_setup_pe_seg: trigger IO SEG 0 [0.589992] pnv_ioda_setup_pe_seg: res[io 0x1000-0x3fff] 100 The last 0x100 is the res->flags, which indicates the UN

[PATCH 36/36] PCI: Don't set flags to 0 when assign resource fail

2015-07-06 Thread Yinghai Lu
make flags take IORESOURCE_UNSET | IORESOURCE_DISABLED instead. Signed-off-by: Yinghai Lu --- drivers/pci/setup-bus.c | 52 + drivers/pci/setup-res.c | 11 +++ 2 files changed, 38 insertions(+), 25 deletions(-) diff --git a/drivers/pci/set