Re: [PATCH] eata: Convert eata driver as normal PCI and platform device drivers

On 2016/3/2 5:36, Arthur Marsh wrote: > > > Arthur Marsh wrote on 02/03/16 03:57: >> >> >> Christoph Hellwig wrote on 01/03/16 17:22: >>> Hi Jiang. >>> >>> I'd love to see this patch in and abuse of the old PCI API gone. >>> >>> Did you resolve the problems Arthur saw with the previous iteratons

Re: [PATCH] eata: Convert eata driver as normal PCI and platform device drivers

On 2016/3/2 5:36, Arthur Marsh wrote: > > > Arthur Marsh wrote on 02/03/16 03:57: >> >> >> Christoph Hellwig wrote on 01/03/16 17:22: >>> Hi Jiang. >>> >>> I'd love to see this patch in and abuse of the old PCI API gone. >>> >>> Did you resolve the problems Arthur saw with the previous iteratons

[PATCH] eata: Convert eata driver as normal PCI and platform device drivers

o provides headroom for PCI hotplug with eata driver. It also represents non-PCI eata devices as platform devices, so it could be managed as normal devices. Signed-off-by: Jiang Liu <jiang@linux.intel.com> Cc: Hannes Reinecke <h...@suse.de> Cc: Ballabio, Dario <dario.balla...@emc.c

[PATCH] eata: Convert eata driver as normal PCI and platform device drivers

o provides headroom for PCI hotplug with eata driver. It also represents non-PCI eata devices as platform devices, so it could be managed as normal devices. Signed-off-by: Jiang Liu Cc: Hannes Reinecke Cc: Ballabio, Dario Cc: Christoph Hellwig --- drivers/scsi/eat

Re: [Bugfix v2 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

On 2015/12/24 13:15, Jeremiah Mahler wrote: > Jiang, > > On Wed, Dec 23, 2015 at 10:13:26PM +0800, Jiang Liu wrote: >> Function __assign_irq_vector() makes use of apic_chip_data.old_domain >> as a temporary buffer, which causes trouble to rollback logic in case of >>

Re: [Bugfix v2 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

On 2015/12/24 13:15, Jeremiah Mahler wrote: > Jiang, > > On Wed, Dec 23, 2015 at 10:13:26PM +0800, Jiang Liu wrote: >> Function __assign_irq_vector() makes use of apic_chip_data.old_domain >> as a temporary buffer, which causes trouble to rollback logic in case of >>

[Bugfix v2 2/5] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Enhance __assign_irq_vector() to rollback in case of failure so the caller doesn't need to explicitly rollback. Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/arch/x86/kernel/apic

[Bugfix v2 4/5] x86/irq: Fix a race condition between vector assigning and cleanup

helps to get rid of that atomic allocation in __send_cleanup_vector(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 76 ++--- 1 file changed, 34 insertions(+), 42 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic

[Bugfix v2 3/5] x86/irq: Fix a race window in x86_vector_free_irqs()

to guard all memory free code in x86_vector_free_irqs(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c index b32c6ef7b4b0..f648fce

[Bugfix v2 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Function __assign_irq_vector() makes use of apic_chip_data.old_domain as a temporary buffer, which causes trouble to rollback logic in case of failure. So use a dedicated temporary buffer for __assign_irq_vector(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c |9 + 1

[Bugfix v2 5/5] x86/irq: Trivial cleanups for x86 vector allocation code

Trivial cleanups for x86 vector allocation code: 1) reorganize apic_chip_data to optimize for size and cache efficiency 2) avoid redundant calling of irq_to_desc() 3) refine code comments Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 54

[Bugfix v2 3/5] x86/irq: Fix a race window in x86_vector_free_irqs()

to guard all memory free code in x86_vector_free_irqs(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c

[Bugfix v2 4/5] x86/irq: Fix a race condition between vector assigning and cleanup

domain This bugfix patch also helps to get rid of that atomic allocation in __send_cleanup_vector(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 76 ++--- 1 file changed, 34 insertions(+), 42 deletions(-)

[Bugfix v2 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Function __assign_irq_vector() makes use of apic_chip_data.old_domain as a temporary buffer, which causes trouble to rollback logic in case of failure. So use a dedicated temporary buffer for __assign_irq_vector(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kerne

[Bugfix v2 2/5] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Enhance __assign_irq_vector() to rollback in case of failure so the caller doesn't need to explicitly rollback. Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff

[Bugfix v2 5/5] x86/irq: Trivial cleanups for x86 vector allocation code

Trivial cleanups for x86 vector allocation code: 1) reorganize apic_chip_data to optimize for size and cache efficiency 2) avoid redundant calling of irq_to_desc() 3) refine code comments Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c

Re: Problems with x86/x86_64 qemu tests in linux-next due to 'Enhance __assign_irq_vector() to rollback ...'

On 2015/12/18 7:59, Guenter Roeck wrote: > Hi folks, > > several of my qemu tests of linux-next started failing a couple of days > ago. > Logs are available at http://server.roeck-us.net:8010/builders, in the > 'next' column. > > Bisect points to 'x86/irq: Enhance __assign_irq_vector() to

Re: Problems with x86/x86_64 qemu tests in linux-next due to 'Enhance __assign_irq_vector() to rollback ...'

On 2015/12/18 7:59, Guenter Roeck wrote: > Hi folks, > > several of my qemu tests of linux-next started failing a couple of days > ago. > Logs are available at http://server.roeck-us.net:8010/builders, in the > 'next' column. > > Bisect points to 'x86/irq: Enhance __assign_irq_vector() to

Re: [LKP] [lkp] [x86/irq] 4c24cee6b2: IP-Config: Auto-configuration of network failed

ata1.00: status: { DRDY } > [ 63.902649] ata1: soft resetting link > [ 64.062580] ata1.01: NODEV after polling detection > [ 64.073800] ata1.00: configured for MWDMA1 > [ 64.076813] ata1.00: device reported invalid CHS sector 0 > [ 64.096188] ata1: EH complete > >From c7c3cc3a048576fd1

Re: [LKP] [lkp] [x86/irq] 4c24cee6b2: IP-Config: Auto-configuration of network failed

ata1.00: status: { DRDY } > [ 63.902649] ata1: soft resetting link > [ 64.062580] ata1.01: NODEV after polling detection > [ 64.073800] ata1.00: configured for MWDMA1 > [ 64.076813] ata1.00: device reported invalid CHS sector 0 > [ 64.096188] ata1: EH complete > >From c7c3cc3a048576fd1

Re: [lkp] [x86/irq] 4c24cee6b2: IP-Config: Auto-configuration of network failed

Hi Ying, Thanks for reporting this issue. But I couldn't figure out what's wrong with this commit. And there's no error or warning messages in the attached dmesg file. Are there other systems reporting the same issue? Thanks, Gerry On 2015/12/11 15:49, kernel test robot wrote: > FYI, we

Re: [lkp] [x86/irq] 4c24cee6b2: IP-Config: Auto-configuration of network failed

Hi Ying, Thanks for reporting this issue. But I couldn't figure out what's wrong with this commit. And there's no error or warning messages in the attached dmesg file. Are there other systems reporting the same issue? Thanks, Gerry On 2015/12/11 15:49, kernel test robot wrote: > FYI, we

[tip:x86/apic] x86/irq: Trivial cleanups for x86 vector allocation code

Commit-ID: 27dd9e6098141a9ebaafe48d50277fcae6e09775 Gitweb: http://git.kernel.org/tip/27dd9e6098141a9ebaafe48d50277fcae6e09775 Author: Jiang Liu AuthorDate: Mon, 30 Nov 2015 16:09:30 +0800 Committer: Thomas Gleixner CommitDate: Thu, 10 Dec 2015 19:39:57 +0100 x86/irq: Trivial cleanups

[tip:x86/urgent] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Commit-ID: 6dd7cb991fcbfef55d8bf3d22b8a87f9d5007e20 Gitweb: http://git.kernel.org/tip/6dd7cb991fcbfef55d8bf3d22b8a87f9d5007e20 Author: Jiang Liu AuthorDate: Mon, 30 Nov 2015 16:09:26 +0800 Committer: Thomas Gleixner CommitDate: Thu, 10 Dec 2015 19:32:07 +0100 x86/irq: Do not reuse

[tip:x86/urgent] x86/irq: Fix a race condition between vector assigning and cleanup

Commit-ID: 41c7518a5d14543fa4aa1b5b9994ac26b38c0406 Gitweb: http://git.kernel.org/tip/41c7518a5d14543fa4aa1b5b9994ac26b38c0406 Author: Jiang Liu AuthorDate: Mon, 30 Nov 2015 16:09:29 +0800 Committer: Thomas Gleixner CommitDate: Thu, 10 Dec 2015 19:32:07 +0100 x86/irq: Fix a race

[tip:x86/urgent] x86/irq: Fix a race window in x86_vector_free_irqs()

Commit-ID: 21a1b3bf35018b446c943c15f0a6225e6f6497ae Gitweb: http://git.kernel.org/tip/21a1b3bf35018b446c943c15f0a6225e6f6497ae Author: Jiang Liu AuthorDate: Mon, 30 Nov 2015 16:09:28 +0800 Committer: Thomas Gleixner CommitDate: Thu, 10 Dec 2015 19:32:07 +0100 x86/irq: Fix a race

[tip:x86/urgent] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Commit-ID: 4c24cee6b2aeaee3dab896f76fef4fe79d9e4183 Gitweb: http://git.kernel.org/tip/4c24cee6b2aeaee3dab896f76fef4fe79d9e4183 Author: Jiang Liu AuthorDate: Mon, 30 Nov 2015 16:09:27 +0800 Committer: Thomas Gleixner CommitDate: Thu, 10 Dec 2015 19:32:07 +0100 x86/irq: Enhance

[tip:x86/urgent] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Commit-ID: 4c24cee6b2aeaee3dab896f76fef4fe79d9e4183 Gitweb: http://git.kernel.org/tip/4c24cee6b2aeaee3dab896f76fef4fe79d9e4183 Author: Jiang Liu <jiang@linux.intel.com> AuthorDate: Mon, 30 Nov 2015 16:09:27 +0800 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:x86/urgent] x86/irq: Fix a race window in x86_vector_free_irqs()

Commit-ID: 21a1b3bf35018b446c943c15f0a6225e6f6497ae Gitweb: http://git.kernel.org/tip/21a1b3bf35018b446c943c15f0a6225e6f6497ae Author: Jiang Liu <jiang@linux.intel.com> AuthorDate: Mon, 30 Nov 2015 16:09:28 +0800 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:x86/urgent] x86/irq: Fix a race condition between vector assigning and cleanup

Commit-ID: 41c7518a5d14543fa4aa1b5b9994ac26b38c0406 Gitweb: http://git.kernel.org/tip/41c7518a5d14543fa4aa1b5b9994ac26b38c0406 Author: Jiang Liu <jiang@linux.intel.com> AuthorDate: Mon, 30 Nov 2015 16:09:29 +0800 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:x86/apic] x86/irq: Trivial cleanups for x86 vector allocation code

Commit-ID: 27dd9e6098141a9ebaafe48d50277fcae6e09775 Gitweb: http://git.kernel.org/tip/27dd9e6098141a9ebaafe48d50277fcae6e09775 Author: Jiang Liu <jiang@linux.intel.com> AuthorDate: Mon, 30 Nov 2015 16:09:30 +0800 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:x86/urgent] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Commit-ID: 6dd7cb991fcbfef55d8bf3d22b8a87f9d5007e20 Gitweb: http://git.kernel.org/tip/6dd7cb991fcbfef55d8bf3d22b8a87f9d5007e20 Author: Jiang Liu <jiang@linux.intel.com> AuthorDate: Mon, 30 Nov 2015 16:09:26 +0800 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

Re: [PATCH v6 4/7] PCI: Add fwnode_handle to pci_sysdata

On 2015/11/3 5:33, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch adds an fwnode_handle to struct pci_sysdata, which is > used by the next patch in the series when trying to locate an > IRQ domain associated with a root PCI bus. > > Signed-off-by: Jake Oshins > --- >

Re: [PATCH v6 7/7] PCI: hv: New paravirtual PCI front-end for Hyper-V VMs

On 2015/11/3 5:33, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch introduces a new driver which exposes a root PCI bus whenever a PCI > Express device is passed through to a guest VM under Hyper-V. The device can > be single- or multi-function. The interrupts for the devices are

Re: [PATCH v6 4/7] PCI: Add fwnode_handle to pci_sysdata

On 2015/11/3 5:33, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch adds an fwnode_handle to struct pci_sysdata, which is > used by the next patch in the series when trying to locate an > IRQ domain associated with a root PCI bus. > > Signed-off-by: Jake Oshins

Re: [PATCH v6 7/7] PCI: hv: New paravirtual PCI front-end for Hyper-V VMs

On 2015/11/3 5:33, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch introduces a new driver which exposes a root PCI bus whenever a PCI > Express device is passed through to a guest VM under Hyper-V. The device can > be single- or multi-function. The interrupts

[Bugfix 4/5] x86/irq: Fix a race condition between vector assigning and cleanup

helps to get rid of that atomic allocation in __send_cleanup_vector(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 77 ++--- 1 file changed, 34 insertions(+), 43 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic

[Bugfix 2/5] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Enhance __assign_irq_vector() to rollback in case of failure so the caller doesn't need to explicitly rollback. Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/arch/x86/kernel/apic

[Bugfix 3/5] x86/irq: Fix a race window in x86_vector_free_irqs()

to guard all memory free code in x86_vector_free_irqs(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c index f03957e7c50d..57934ef

[Bugfix 5/5] x86/irq: Trivial cleanups for x86 vector allocation code

Trivial cleanups for x86 vector allocation code: 1) reorganize apic_chip_data to optimize for size and cache efficiency 2) avoid redundant calling of irq_to_desc() 3) refine code comments Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c | 54

[Bugfix 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Function __assign_irq_vector() makes use of apic_chip_data.old_domain as a temporary buffer, which causes trouble to rollback logic in case of failure. So use a dedicated temporary buffer for __assign_irq_vector(). Signed-off-by: Jiang Liu --- arch/x86/kernel/apic/vector.c |9 + 1

[Bugfix 4/5] x86/irq: Fix a race condition between vector assigning and cleanup

domain This bugfix patch also helps to get rid of that atomic allocation in __send_cleanup_vector(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 77 ++--- 1 file changed, 34 insertions(+), 43 deletions(-)

[Bugfix 2/5] x86/irq: Enhance __assign_irq_vector() to rollback in case of failure

Enhance __assign_irq_vector() to rollback in case of failure so the caller doesn't need to explicitly rollback. Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff

[Bugfix 3/5] x86/irq: Fix a race window in x86_vector_free_irqs()

to guard all memory free code in x86_vector_free_irqs(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c

[Bugfix 5/5] x86/irq: Trivial cleanups for x86 vector allocation code

Trivial cleanups for x86 vector allocation code: 1) reorganize apic_chip_data to optimize for size and cache efficiency 2) avoid redundant calling of irq_to_desc() 3) refine code comments Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kernel/apic/vector.c

[Bugfix 1/5] x86/irq: Do not reuse struct apic_chip_data.old_domain as temporary buffer

Function __assign_irq_vector() makes use of apic_chip_data.old_domain as a temporary buffer, which causes trouble to rollback logic in case of failure. So use a dedicated temporary buffer for __assign_irq_vector(). Signed-off-by: Jiang Liu <jiang@linux.intel.com> --- arch/x86/kerne

Re: [PATCH] x86/PCI: Fix regression caused by commit 4d6b4e69a245

On 2015/11/28 0:06, Rafael J. Wysocki wrote: > On Friday, November 27, 2015 11:12:33 AM Jiang Liu wrote: >> From: Liu Jiang >> >> Commit 4d6b4e69a245 ("x86/PCI/ACPI: Use common interface to support >> PCI host bridge") converted x86 to use the com

Re: [PATCH] x86/PCI: Fix regression caused by commit 4d6b4e69a245

On 2015/11/28 0:06, Rafael J. Wysocki wrote: > On Friday, November 27, 2015 11:12:33 AM Jiang Liu wrote: >> From: Liu Jiang <jiang@linux.intel.com> >> >> Commit 4d6b4e69a245 ("x86/PCI/ACPI: Use common interface to support >> PCI host bridge"

Re: irq_desc use-after-free in smp_irq_move_cleanup_interrupt

On 2015/11/26 5:12, Thomas Gleixner wrote: > On Wed, 25 Nov 2015, Thomas Gleixner wrote: >> So if CPU1 gets the IPI _BEFORE_ move_in_progress is set to 0, and >> does not get another IPI before the next move . That has been that >> way forever. >> >> Duh. Working on a real fix this time. >

Re: irq_desc use-after-free in smp_irq_move_cleanup_interrupt

On 2015/11/26 5:12, Thomas Gleixner wrote: > On Wed, 25 Nov 2015, Thomas Gleixner wrote: >> So if CPU1 gets the IPI _BEFORE_ move_in_progress is set to 0, and >> does not get another IPI before the next move . That has been that >> way forever. >> >> Duh. Working on a real fix this time. >

[PATCH] x86/PCI: Fix regression caused by commit 4d6b4e69a245

m_resource/ioport_resource. Fixes: 4d6b4e69a245 ("x86/PCI/ACPI: Use common interface to support PCI host bridge") Signed-off-by: Jiang Liu Reported-and-tested-by: Arthur Marsh Cc: Keith Busch Cc: Arthur Marsh Cc: Hans de Bruin --- arch/x86/pci/bus_numa.c | 13 ++--- dri

[PATCH] x86/PCI: Fix regression caused by commit 4d6b4e69a245

e resources. So change x86_pci_root_bus_resources() to not insert resources into iomem_resource/ioport_resource. Fixes: 4d6b4e69a245 ("x86/PCI/ACPI: Use common interface to support PCI host bridge") Signed-off-by: Jiang Liu <jiang@linux.intel.com> Reported-and-tested-by: Arthur Marsh <arthu

Re: [Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

il: > > 4d6b4e69a245e9df4b84dba387596086cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support PCI host bridge > > Use com

Re: [Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

On 2015/11/25 6:19, Rafael J. Wysocki wrote: > On Tue, Nov 24, 2015 at 5:49 PM, Bjorn Helgaas wrote: >> On Mon, Nov 16, 2015 at 12:27:37PM +0800, Jiang Liu wrote: >>> From: Liu Jiang >>> >>> Commit 4d6b4e69a245 ("x86/PCI/ACPI: Use common interface to su

Re: [Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

il: > > 4d6b4e69a245e9df4b84dba387596086cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu <jiang@linux.intel.com> > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support P

Re: [Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

On 2015/11/25 6:19, Rafael J. Wysocki wrote: > On Tue, Nov 24, 2015 at 5:49 PM, Bjorn Helgaas <helg...@kernel.org> wrote: >> On Mon, Nov 16, 2015 at 12:27:37PM +0800, Jiang Liu wrote: >>> From: Liu Jiang <jiang@linux.intel.com> >>> >>> Commit 4

[Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

m_resource/ioport_resource. Signed-off-by: Jiang Liu Reported-and-tested-by: Arthur Marsh Cc: Keith Busch Cc: Arthur Marsh --- arch/x86/pci/bus_numa.c | 16 +++- drivers/acpi/pci_root.c |7 +++ 2 files changed, 10 insertions(+), 13 deletions(-) diff --git a/arch/x86/pci/

[Bugfix] x86/PCI: Fix regression caused by commit 4d6b4e69a245

e resources. So change x86_pci_root_bus_resources() to not insert resources into iomem_resource/ioport_resource. Signed-off-by: Jiang Liu <jiang@linux.intel.com> Reported-and-tested-by: Arthur Marsh <arthur.ma...@internode.on.net> Cc: Keith Busch <keith.bu...@intel.com> Cc: Arthur Marsh <a

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/14 1:03, Lorenzo Pieralisi wrote: > Please trim your emails, thanks. > > On Fri, Nov 13, 2015 at 01:57:30PM +0100, Tomasz Nowicki wrote: >> On 12.11.2015 16:05, Jiang Liu wrote: > > [...] > >>>>> IA64 actually ignores the tra

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/14 1:03, Lorenzo Pieralisi wrote: > Please trim your emails, thanks. > > On Fri, Nov 13, 2015 at 01:57:30PM +0100, Tomasz Nowicki wrote: >> On 12.11.2015 16:05, Jiang Liu wrote: > > [...] > >>>>> IA64 actually ignores the tra

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 22:45, Tomasz Nowicki wrote: > On 12.11.2015 15:04, Jiang Liu wrote: >> On 2015/11/12 21:21, Tomasz Nowicki wrote: >>> On 12.11.2015 09:43, Jiang Liu wrote: >>>> On 2015/11/12 1:46, Lorenzo Pieralisi wrote: >>>>> On Tue, Nov

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 21:21, Tomasz Nowicki wrote: > On 12.11.2015 09:43, Jiang Liu wrote: >> On 2015/11/12 1:46, Lorenzo Pieralisi wrote: >>> On Tue, Nov 10, 2015 at 01:50:46PM +0800, Jiang Liu wrote: >>> >>> [...] >>> >>>>>> In particular,

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 1:46, Lorenzo Pieralisi wrote: > On Tue, Nov 10, 2015 at 01:50:46PM +0800, Jiang Liu wrote: > > [...] > >>>> In particular, I would like to understand, for an eg DWordIO descriptor, >>>> what Range Minimum, Range Maximum and Translation Of

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 1:46, Lorenzo Pieralisi wrote: > On Tue, Nov 10, 2015 at 01:50:46PM +0800, Jiang Liu wrote: > > [...] > >>>> In particular, I would like to understand, for an eg DWordIO descriptor, >>>> what Range Minimum, Range Maximum and Translation Of

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 21:21, Tomasz Nowicki wrote: > On 12.11.2015 09:43, Jiang Liu wrote: >> On 2015/11/12 1:46, Lorenzo Pieralisi wrote: >>> On Tue, Nov 10, 2015 at 01:50:46PM +0800, Jiang Liu wrote: >>> >>> [...] >>> >>>>>> In particular,

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/12 22:45, Tomasz Nowicki wrote: > On 12.11.2015 15:04, Jiang Liu wrote: >> On 2015/11/12 21:21, Tomasz Nowicki wrote: >>> On 12.11.2015 09:43, Jiang Liu wrote: >>>> On 2015/11/12 1:46, Lorenzo Pieralisi wrote: >>>>> On Tue, Nov

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/10 4:09, Arnd Bergmann wrote: > On Monday 09 November 2015 17:10:43 Lorenzo Pieralisi wrote: >> On Mon, Nov 09, 2015 at 03:07:38PM +0100, Tomasz Nowicki wrote: >>> On 06.11.2015 14:22, Jiang Liu wrote: >>>> On 2015/11/6 20:40, Tomasz Nowicki wrote: >&g

Re: [PATCH] acpi: add support for extended IRQ to PCI link

On 2015/11/9 13:45, Sinan Kaya wrote: > > > On 11/9/2015 12:24 AM, Jiang Liu wrote: >>> +u32 possible[ACPI_PCI_LINK_MAX_POSSIBLE]; >>> > u8 initialized:1; >>> > u8 reserved:7; >>> > }; >> Hi Sinan, >> This da

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/10 4:09, Arnd Bergmann wrote: > On Monday 09 November 2015 17:10:43 Lorenzo Pieralisi wrote: >> On Mon, Nov 09, 2015 at 03:07:38PM +0100, Tomasz Nowicki wrote: >>> On 06.11.2015 14:22, Jiang Liu wrote: >>>> On 2015/11/6 20:40, Tomasz Nowicki wrote: >&g

Re: [PATCH] acpi: add support for extended IRQ to PCI link

On 2015/11/9 13:45, Sinan Kaya wrote: > > > On 11/9/2015 12:24 AM, Jiang Liu wrote: >>> +u32 possible[ACPI_PCI_LINK_MAX_POSSIBLE]; >>> > u8 initialized:1; >>> > u8 reserved:7; >>> > }; >> Hi Sinan, >> This da

Re: lock-up on boot with x86/PCI/ACPI: Use common interface to support PCI host bridge

6cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support PCI host bridge > > Use common interface to simplify ACPI PCI host bri

Re: [PATCH] acpi: add support for extended IRQ to PCI link

On 2015/11/9 0:07, Sinan Kaya wrote: > The ACPI compiler uses the extended format when > used interrupt numbers are greater than 256. > The PCI link code currently only supports simple > interrupt format. The IRQ numbers are represented > using 32 bits when extended IRQ syntax. This patch >

Re: lock-up on boot with x86/PCI/ACPI: Use common interface to support PCI host bridge

6cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support PCI host bridge > > Use common interface to simplify ACPI PCI host bri

Re: lock-up on boot with x86/PCI/ACPI: Use common interface to support PCI host bridge

6cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu <jiang@linux.intel.com> > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support PCI host bridge > > Use comm

Re: [PATCH] acpi: add support for extended IRQ to PCI link

On 2015/11/9 0:07, Sinan Kaya wrote: > The ACPI compiler uses the extended format when > used interrupt numbers are greater than 256. > The PCI link code currently only supports simple > interrupt format. The IRQ numbers are represented > using 32 bits when extended IRQ syntax. This patch >

Re: lock-up on boot with x86/PCI/ACPI: Use common interface to support PCI host bridge

6cb66887d is the first bad commit > commit 4d6b4e69a245e9df4b84dba387596086cb66887d > Author: Jiang Liu <jiang@linux.intel.com> > Date: Wed Oct 14 14:29:41 2015 +0800 > > x86/PCI/ACPI: Use common interface to support PCI host bridge > > Use comm

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 23:32, Jiang Liu wrote: > On 2015/11/6 22:45, Lorenzo Pieralisi wrote: >> On Fri, Nov 06, 2015 at 09:22:46PM +0800, Jiang Liu wrote: >>> On 2015/11/6 20:40, Tomasz Nowicki wrote: >>>> On 06.11.2015 12:46, Jiang Liu wrote: >>>>

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 22:45, Lorenzo Pieralisi wrote: > On Fri, Nov 06, 2015 at 09:22:46PM +0800, Jiang Liu wrote: >> On 2015/11/6 20:40, Tomasz Nowicki wrote: >>> On 06.11.2015 12:46, Jiang Liu wrote: >>>> On 2015/11/6 18:37, Tomasz Nowicki wrote: >>>>> On

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 20:40, Tomasz Nowicki wrote: > On 06.11.2015 12:46, Jiang Liu wrote: >> On 2015/11/6 18:37, Tomasz Nowicki wrote: >>> On 06.11.2015 09:52, Jiang Liu wrote: >>> Sure, ARM64 (0-16M IO space) QEMU example: >>> DWordIO (ResourceProducer, MinFix

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 18:37, Tomasz Nowicki wrote: > On 06.11.2015 09:52, Jiang Liu wrote: >> On 2015/11/6 2:19, Lorenzo Pieralisi wrote: >>> On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >>>> On 14.10.2015 08:29, Jiang Liu wrote: >&g

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 2:19, Lorenzo Pieralisi wrote: > On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >> On 14.10.2015 08:29, Jiang Liu wrote: > > [...] > >>> +static void acpi_pci_root_validate_resources(struct device *dev, >>> +

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 2:19, Lorenzo Pieralisi wrote: > On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >> On 14.10.2015 08:29, Jiang Liu wrote: > > [...] > >>> +static void acpi_pci_root_validate_resources(struct device *dev, >>> +

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 20:40, Tomasz Nowicki wrote: > On 06.11.2015 12:46, Jiang Liu wrote: >> On 2015/11/6 18:37, Tomasz Nowicki wrote: >>> On 06.11.2015 09:52, Jiang Liu wrote: >>> Sure, ARM64 (0-16M IO space) QEMU example: >>> DWordIO (ResourceProducer, MinFix

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 18:37, Tomasz Nowicki wrote: > On 06.11.2015 09:52, Jiang Liu wrote: >> On 2015/11/6 2:19, Lorenzo Pieralisi wrote: >>> On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >>>> On 14.10.2015 08:29, Jiang Liu wrote: >&g

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 22:45, Lorenzo Pieralisi wrote: > On Fri, Nov 06, 2015 at 09:22:46PM +0800, Jiang Liu wrote: >> On 2015/11/6 20:40, Tomasz Nowicki wrote: >>> On 06.11.2015 12:46, Jiang Liu wrote: >>>> On 2015/11/6 18:37, Tomasz Nowicki wrote: >>>>> On

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 23:32, Jiang Liu wrote: > On 2015/11/6 22:45, Lorenzo Pieralisi wrote: >> On Fri, Nov 06, 2015 at 09:22:46PM +0800, Jiang Liu wrote: >>> On 2015/11/6 20:40, Tomasz Nowicki wrote: >>>> On 06.11.2015 12:46, Jiang Liu wrote: >>>>

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 2:19, Lorenzo Pieralisi wrote: > On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >> On 14.10.2015 08:29, Jiang Liu wrote: > > [...] > >>> +static void acpi_pci_root_validate_resources(struct device *dev, >>> +

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/5 22:21, Tomasz Nowicki wrote: > On 14.10.2015 08:29, Jiang Liu wrote: >> Introduce common interface acpi_pci_root_create() and related data >> structures to create PCI root bus for ACPI PCI host bridges. It will >> be used to kill duplicated arch specific code for

Re: [PATCH v2 02/11] fsl-mc: msi: Added FSL-MC-specific member to the msi_desc's union

On 2015/10/31 3:43, J. German Rivera wrote: > FSL-MC is a bus type different from PCI and platform, so it needs > its own member in the msi_desc's union. > > Signed-off-by: J. German Rivera > --- > Changes in v2: > - Addressed comment from Jiang Liu > * Added a dedica

Re: [PATCH v2 01/11] irqdomain: Added domain bus token DOMAIN_BUS_FSL_MC_MSI

rq_domain_bus_token { > DOMAIN_BUS_PCI_MSI, > DOMAIN_BUS_PLATFORM_MSI, > DOMAIN_BUS_NEXUS, > + DOMAIN_BUS_FSL_MC_MSI, > }; Reviewed-by: Jiang Liu > > /** > -- > 2.3.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel&

Re: [Bugfix v4] PCI, ACPI: Fix regressions caused by resource_size_t overflow with 32-bit kernel

On 2015/11/5 20:53, Tomasz Nowicki wrote: > On 02.11.2015 16:27, Tomasz Nowicki wrote: >> On 08.07.2015 09:26, Jiang Liu wrote: >>> Zoltan Boszormenyi reported this regression: >>>"There's a Realtek RTL8111/8168/8411 (PCI ID 10ec:8168, Subsystem ID

Re: [PATCH v2 02/11] fsl-mc: msi: Added FSL-MC-specific member to the msi_desc's union

On 2015/10/31 3:43, J. German Rivera wrote: > FSL-MC is a bus type different from PCI and platform, so it needs > its own member in the msi_desc's union. > > Signed-off-by: J. German Rivera <german.riv...@freescale.com> > --- > Changes in v2: > - Addressed comment

Re: [PATCH v2 01/11] irqdomain: Added domain bus token DOMAIN_BUS_FSL_MC_MSI

ain.h > @@ -73,6 +73,7 @@ enum irq_domain_bus_token { > DOMAIN_BUS_PCI_MSI, > DOMAIN_BUS_PLATFORM_MSI, > DOMAIN_BUS_NEXUS, > + DOMAIN_BUS_FSL_MC_MSI, > }; Reviewed-by: Jiang Liu <jiang@linux.intel.com> > > /** > -- > 2.3.3 > -- To uns

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/6 2:19, Lorenzo Pieralisi wrote: > On Thu, Nov 05, 2015 at 03:21:34PM +0100, Tomasz Nowicki wrote: >> On 14.10.2015 08:29, Jiang Liu wrote: > > [...] > >>> +static void acpi_pci_root_validate_resources(struct device *dev, >>> +

Re: [Patch v7 4/7] PCI/ACPI: Add interface acpi_pci_root_create()

On 2015/11/5 22:21, Tomasz Nowicki wrote: > On 14.10.2015 08:29, Jiang Liu wrote: >> Introduce common interface acpi_pci_root_create() and related data >> structures to create PCI root bus for ACPI PCI host bridges. It will >> be used to kill duplicated arch specific code for

Re: [Bugfix v4] PCI, ACPI: Fix regressions caused by resource_size_t overflow with 32-bit kernel

On 2015/11/5 20:53, Tomasz Nowicki wrote: > On 02.11.2015 16:27, Tomasz Nowicki wrote: >> On 08.07.2015 09:26, Jiang Liu wrote: >>> Zoltan Boszormenyi reported this regression: >>>"There's a Realtek RTL8111/8168/8411 (PCI ID 10ec:8168, Subsystem ID

Re: [RFC PATCHv3 3/4] x86/pci: Initial commit for new VMD device driver

22ed5502abc6eb Mon Sep 17 00:00:00 2001 From: Liu Jiang Date: Thu, 5 Nov 2015 11:25:07 +0800 Subject: [PATCH] msi: Relax msi_domain_alloc() to support parentless MSI irqdomains Previously msi_domain_alloc() assumes MSI irqdomains always have parent irqdomains, but that's not true for the ne

Re: [RFC PATCHv3 3/4] x86/pci: Initial commit for new VMD device driver

1c168b22ed5502abc6eb Mon Sep 17 00:00:00 2001 From: Liu Jiang <jiang@linux.intel.com> Date: Thu, 5 Nov 2015 11:25:07 +0800 Subject: [PATCH] msi: Relax msi_domain_alloc() to support parentless MSI irqdomains Previously msi_domain_alloc() assumes MSI irqdomains always have parent irqdomains, but th

Re: [PATCH v4 4/7] PCI: Add fwnode_handle to pci_sysdata

On 2015/10/30 7:46, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch adds an fwnode_handle to struct pci_sysdata, which is > used by the next patch in the series when trying to locate an > IRQ domain associated with a root PCI bus. > > Signed-off-by: Jake Oshins > --- >

Re: [PATCH v4 4/7] PCI: Add fwnode_handle to pci_sysdata

On 2015/10/30 7:46, ja...@microsoft.com wrote: > From: Jake Oshins > > This patch adds an fwnode_handle to struct pci_sysdata, which is > used by the next patch in the series when trying to locate an > IRQ domain associated with a root PCI bus. > > Signed-off-by: Jake

  1   2   3   4   5   6   7   8   9   10   >