* [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic
@ 2021-06-15 17:20 David Christensen
2021-06-24 13:51 ` David Marchand
0 siblings, 1 reply; 4+ messages in thread
From: David Christensen @ 2021-06-15 17:20 UTC (permalink / raw)
To: dev; +Cc: David Christensen
Fix the IOMMU detection logic that looks for the "platform" field of
/proc/cpuinfo on POWER systems.
Fixes: 905215731833 ("bus/pci: support IOVA as VA on PowerNV systems")
Signed-off-by: David Christensen <drc@linux.vnet.ibm.com>
---
drivers/bus/pci/linux/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/bus/pci/linux/pci.c b/drivers/bus/pci/linux/pci.c
index 0dc99e9cb2..edfe430268 100644
--- a/drivers/bus/pci/linux/pci.c
+++ b/drivers/bus/pci/linux/pci.c
@@ -569,7 +569,7 @@ pci_device_iommu_support_va(__rte_unused const struct rte_pci_device *dev)
/* Check for a PowerNV platform */
while (getline(&line, &len, fp) != -1) {
- if (strstr(line, "platform") != NULL)
+ if (strstr(line, "platform") == NULL)
continue;
if (strstr(line, "PowerNV") != NULL) {
--
2.27.0
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic
2021-06-15 17:20 [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic David Christensen
@ 2021-06-24 13:51 ` David Marchand
2021-06-24 16:59 ` David Christensen
2021-07-05 8:58 ` David Marchand
0 siblings, 2 replies; 4+ messages in thread
From: David Marchand @ 2021-06-24 13:51 UTC (permalink / raw)
To: David Christensen; +Cc: dev
On Tue, Jun 15, 2021 at 7:21 PM David Christensen
<drc@linux.vnet.ibm.com> wrote:
>
> Fix the IOMMU detection logic that looks for the "platform" field of
> /proc/cpuinfo on POWER systems.
>
> Fixes: 905215731833 ("bus/pci: support IOVA as VA on PowerNV systems")
Cc: stable@dpdk.org
>
> Signed-off-by: David Christensen <drc@linux.vnet.ibm.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
Looking at http://patchwork.dpdk.org/project/dpdk/patch/20210623204355.74362-1-drc@linux.vnet.ibm.com/,
I understand it should be merged after this current patch.
Can you confirm?
--
David Marchand
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic
2021-06-24 13:51 ` David Marchand
@ 2021-06-24 16:59 ` David Christensen
2021-07-05 8:58 ` David Marchand
1 sibling, 0 replies; 4+ messages in thread
From: David Christensen @ 2021-06-24 16:59 UTC (permalink / raw)
To: David Marchand; +Cc: dev
On 6/24/21 6:51 AM, David Marchand wrote:
> On Tue, Jun 15, 2021 at 7:21 PM David Christensen
> <drc@linux.vnet.ibm.com> wrote:
>>
>> Fix the IOMMU detection logic that looks for the "platform" field of
>> /proc/cpuinfo on POWER systems.
>>
>> Fixes: 905215731833 ("bus/pci: support IOVA as VA on PowerNV systems")
> Cc: stable@dpdk.org
>
>>
>> Signed-off-by: David Christensen <drc@linux.vnet.ibm.com>
> Reviewed-by: David Marchand <david.marchand@redhat.com>
>
> Looking at http://patchwork.dpdk.org/project/dpdk/patch/20210623204355.74362-1-drc@linux.vnet.ibm.com/,
> I understand it should be merged after this current patch.
> Can you confirm?
Yes, that's correct.
Dave
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic
2021-06-24 13:51 ` David Marchand
2021-06-24 16:59 ` David Christensen
@ 2021-07-05 8:58 ` David Marchand
1 sibling, 0 replies; 4+ messages in thread
From: David Marchand @ 2021-07-05 8:58 UTC (permalink / raw)
To: David Christensen; +Cc: dev, Thinh Tran
On Thu, Jun 24, 2021 at 3:51 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> On Tue, Jun 15, 2021 at 7:21 PM David Christensen
> <drc@linux.vnet.ibm.com> wrote:
> >
> > Fix the IOMMU detection logic that looks for the "platform" field of
> > /proc/cpuinfo on POWER systems.
> >
> > Fixes: 905215731833 ("bus/pci: support IOVA as VA on PowerNV systems")
> Cc: stable@dpdk.org
>
> >
> > Signed-off-by: David Christensen <drc@linux.vnet.ibm.com>
> Reviewed-by: David Marchand <david.marchand@redhat.com>
Applied, thanks.
--
David Marchand
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-07-05 8:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-15 17:20 [dpdk-dev] [PATCH] bus/pci: Fix PowerNV platform detection logic David Christensen
2021-06-24 13:51 ` David Marchand
2021-06-24 16:59 ` David Christensen
2021-07-05 8:58 ` David Marchand
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).