Linux PCI subsystem development
 help / color / mirror / Atom feed
* [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check
@ 2015-01-30 19:45 Richardson, Charlotte
  2015-02-02  3:37 ` Myron Stowe
  2015-02-02 15:37 ` Bjorn Helgaas
  0 siblings, 2 replies; 4+ messages in thread
From: Richardson, Charlotte @ 2015-01-30 19:45 UTC (permalink / raw)
  To: 'linux-pci@vger.kernel.org'
  Cc: 'stable@vger.kernel.org', Lawrence, Joe,
	'myron.stowe@redhat.com'

Hi, Myron -

commit 1278998f8ff6 missed the NEC variants of the ftServers.
We'll need this in RHEL7/6, but can you shepherd this patch upstream first?
Thanks!

/Charlotte

-->8--

>From 64a19253432e87b2293d3b8ee323e8b8ac63395d Mon Sep 17 00:00:00 2001
From: Charlotte Richardson <charlotte.richardson@stratus.com>
Date: Fri, 30 Jan 2015 12:58:31 -0500
Subject: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check.

NEC OEMs the same platforms as Stratus does, which have multiple
devices on some PCIe buses under downstream ports.

Signed-off-by: Charlotte Richardson <charlotte.richardson@stratus.com>
Fixes: 1278998f8ff6 ("PCI: Work around Stratus ftServer broken PCIe hierarchy (fix DMI check)")
Cc: <stable@vger.kernel.org>
Cc: Myron Stowe <myron.stowe@redhat.com>
---
 arch/x86/pci/common.c | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c
index 7b20bcc..2fb3847 100644
--- a/arch/x86/pci/common.c
+++ b/arch/x86/pci/common.c
@@ -448,6 +448,22 @@ static const struct dmi_system_id pciprobe_dmi_table[] __initconst = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "ftServer"),
 		},
 	},
+        {
+                .callback = set_scan_all,
+                .ident = "Stratus/NEC ftServer",
+                .matches = {
+                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
+                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R32"),
+                },
+        },
+        {
+                .callback = set_scan_all,
+                .ident = "Stratus/NEC ftServer",
+                .matches = {
+                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
+                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R31"),
+                },
+        },
 	{}
 };
 
-- 
1.8.3.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check
  2015-01-30 19:45 [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check Richardson, Charlotte
@ 2015-02-02  3:37 ` Myron Stowe
  2015-02-02 15:37 ` Bjorn Helgaas
  1 sibling, 0 replies; 4+ messages in thread
From: Myron Stowe @ 2015-02-02  3:37 UTC (permalink / raw)
  To: Richardson, Charlotte
  Cc: 'linux-pci@vger.kernel.org',
	'stable@vger.kernel.org', Lawrence, Joe,
	'myron.stowe@redhat.com'

On Fri, 2015-01-30 at 19:45 +0000, Richardson, Charlotte wrote:
> Hi, Myron -
> 
> commit 1278998f8ff6 missed the NEC variants of the ftServers.
> We'll need this in RHEL7/6, but can you shepherd this patch upstream first?

Hi Charlotte:

Sure, looks like you've already done the work so I keep tabs on Bjorn's
bringing it in and then handle the back-ports to RHEL.

Please open RHEL BZs for both 6 and 7 if this has not already been done
and let me know what the BZ #s for each are so I can assign them to
myself (just send me the info to my redhat address, no need to pollute
the upstream lists with RHEL specifics).

Myron
> Thanks!
> 
> /Charlotte
> 
> -->8--
> 
> From 64a19253432e87b2293d3b8ee323e8b8ac63395d Mon Sep 17 00:00:00 2001
> From: Charlotte Richardson <charlotte.richardson@stratus.com>
> Date: Fri, 30 Jan 2015 12:58:31 -0500
> Subject: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check.
> 
> NEC OEMs the same platforms as Stratus does, which have multiple
> devices on some PCIe buses under downstream ports.
> 
> Signed-off-by: Charlotte Richardson <charlotte.richardson@stratus.com>
> Fixes: 1278998f8ff6 ("PCI: Work around Stratus ftServer broken PCIe hierarchy (fix DMI check)")
> Cc: <stable@vger.kernel.org>
> Cc: Myron Stowe <myron.stowe@redhat.com>
> ---
>  arch/x86/pci/common.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c
> index 7b20bcc..2fb3847 100644
> --- a/arch/x86/pci/common.c
> +++ b/arch/x86/pci/common.c
> @@ -448,6 +448,22 @@ static const struct dmi_system_id pciprobe_dmi_table[] __initconst = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "ftServer"),
>  		},
>  	},
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R32"),
> +                },
> +        },
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R31"),
> +                },
> +        },
>  	{}
>  };
>  



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check
  2015-01-30 19:45 [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check Richardson, Charlotte
  2015-02-02  3:37 ` Myron Stowe
@ 2015-02-02 15:37 ` Bjorn Helgaas
  2015-02-02 15:38   ` Richardson, Charlotte
  1 sibling, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2015-02-02 15:37 UTC (permalink / raw)
  To: Richardson, Charlotte
  Cc: 'linux-pci@vger.kernel.org',
	'stable@vger.kernel.org', Lawrence, Joe,
	'myron.stowe@redhat.com'

On Fri, Jan 30, 2015 at 07:45:02PM +0000, Richardson, Charlotte wrote:
> Hi, Myron -
> 
> commit 1278998f8ff6 missed the NEC variants of the ftServers.
> We'll need this in RHEL7/6, but can you shepherd this patch upstream first?
> Thanks!
> 
> /Charlotte
> 
> -->8--
> 
> From 64a19253432e87b2293d3b8ee323e8b8ac63395d Mon Sep 17 00:00:00 2001
> From: Charlotte Richardson <charlotte.richardson@stratus.com>
> Date: Fri, 30 Jan 2015 12:58:31 -0500
> Subject: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check.
> 
> NEC OEMs the same platforms as Stratus does, which have multiple
> devices on some PCIe buses under downstream ports.
> 
> Signed-off-by: Charlotte Richardson <charlotte.richardson@stratus.com>
> Fixes: 1278998f8ff6 ("PCI: Work around Stratus ftServer broken PCIe hierarchy (fix DMI check)")
> Cc: <stable@vger.kernel.org>
> Cc: Myron Stowe <myron.stowe@redhat.com>

Applied to for-linus for v3.19, thanks!

> ---
>  arch/x86/pci/common.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c
> index 7b20bcc..2fb3847 100644
> --- a/arch/x86/pci/common.c
> +++ b/arch/x86/pci/common.c
> @@ -448,6 +448,22 @@ static const struct dmi_system_id pciprobe_dmi_table[] __initconst = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "ftServer"),
>  		},
>  	},
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R32"),
> +                },
> +        },
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R31"),
> +                },
> +        },
>  	{}
>  };
>  
> -- 
> 1.8.3.1
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check
  2015-02-02 15:37 ` Bjorn Helgaas
@ 2015-02-02 15:38   ` Richardson, Charlotte
  0 siblings, 0 replies; 4+ messages in thread
From: Richardson, Charlotte @ 2015-02-02 15:38 UTC (permalink / raw)
  To: 'Bjorn Helgaas'
  Cc: 'linux-pci@vger.kernel.org',
	'stable@vger.kernel.org', Lawrence, Joe,
	'myron.stowe@redhat.com'

Thanks!

-----Original Message-----
From: Bjorn Helgaas [mailto:bhelgaas@google.com] 
Sent: Monday, February 02, 2015 10:37 AM
To: Richardson, Charlotte
Cc: 'linux-pci@vger.kernel.org'; 'stable@vger.kernel.org'; Lawrence, Joe; 'myron.stowe@redhat.com'
Subject: Re: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check

On Fri, Jan 30, 2015 at 07:45:02PM +0000, Richardson, Charlotte wrote:
> Hi, Myron -
> 
> commit 1278998f8ff6 missed the NEC variants of the ftServers.
> We'll need this in RHEL7/6, but can you shepherd this patch upstream first?
> Thanks!
> 
> /Charlotte
> 
> -->8--
> 
> From 64a19253432e87b2293d3b8ee323e8b8ac63395d Mon Sep 17 00:00:00 2001
> From: Charlotte Richardson <charlotte.richardson@stratus.com>
> Date: Fri, 30 Jan 2015 12:58:31 -0500
> Subject: [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check.
> 
> NEC OEMs the same platforms as Stratus does, which have multiple 
> devices on some PCIe buses under downstream ports.
> 
> Signed-off-by: Charlotte Richardson <charlotte.richardson@stratus.com>
> Fixes: 1278998f8ff6 ("PCI: Work around Stratus ftServer broken PCIe 
> hierarchy (fix DMI check)")
> Cc: <stable@vger.kernel.org>
> Cc: Myron Stowe <myron.stowe@redhat.com>

Applied to for-linus for v3.19, thanks!

> ---
>  arch/x86/pci/common.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c index 
> 7b20bcc..2fb3847 100644
> --- a/arch/x86/pci/common.c
> +++ b/arch/x86/pci/common.c
> @@ -448,6 +448,22 @@ static const struct dmi_system_id pciprobe_dmi_table[] __initconst = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "ftServer"),
>  		},
>  	},
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R32"),
> +                },
> +        },
> +        {
> +                .callback = set_scan_all,
> +                .ident = "Stratus/NEC ftServer",
> +                .matches = {
> +                        DMI_MATCH(DMI_SYS_VENDOR, "NEC"),
> +                        DMI_MATCH(DMI_PRODUCT_NAME, "Express5800/R31"),
> +                },
> +        },
>  	{}
>  };
>  
> --
> 1.8.3.1
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" 
> in the body of a message to majordomo@vger.kernel.org More majordomo 
> info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-02-02 15:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-30 19:45 [PATCH] PCI: Add NEC ftServer variants to PCIe DMI check Richardson, Charlotte
2015-02-02  3:37 ` Myron Stowe
2015-02-02 15:37 ` Bjorn Helgaas
2015-02-02 15:38   ` Richardson, Charlotte

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox