* [net-next] pci_regs.h: Add PCI bus link speed and width defines @ 2013-12-28 12:36 Jeff Kirsher 2014-01-03 0:21 ` David Miller 0 siblings, 1 reply; 10+ messages in thread From: Jeff Kirsher @ 2013-12-28 12:36 UTC (permalink / raw) To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann, linux-kernel Add missing PCI bus link speed 8.0 GT/s and bus link widths of x1, x2, x4 and x8. CC: <linux-kernel@vger.kernel.org> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- include/uapi/linux/pci_regs.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h index 4a98e85..c870c2a 100644 --- a/include/uapi/linux/pci_regs.h +++ b/include/uapi/linux/pci_regs.h @@ -489,7 +489,12 @@ #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */ #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */ +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */ +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */ +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */ +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */ #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */ #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */ #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */ -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2013-12-28 12:36 [net-next] pci_regs.h: Add PCI bus link speed and width defines Jeff Kirsher @ 2014-01-03 0:21 ` David Miller 2014-01-03 21:53 ` Jesse Brandeburg 0 siblings, 1 reply; 10+ messages in thread From: David Miller @ 2014-01-03 0:21 UTC (permalink / raw) To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann, linux-kernel From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Sat, 28 Dec 2013 04:36:13 -0800 > Add missing PCI bus link speed 8.0 GT/s and bus link widths of > x1, x2, x4 and x8. > > CC: <linux-kernel@vger.kernel.org> > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Can a PCI person please ACK this? This is a prerequisite for some networking driver changes Jeff would like to push to me. Thanks. > --- > include/uapi/linux/pci_regs.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h > index 4a98e85..c870c2a 100644 > --- a/include/uapi/linux/pci_regs.h > +++ b/include/uapi/linux/pci_regs.h > @@ -489,7 +489,12 @@ > #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ > #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ > #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ > +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */ > #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */ > +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */ > +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */ > +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */ > +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */ > #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */ > #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */ > #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */ > -- > 1.8.3.1 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-03 0:21 ` David Miller @ 2014-01-03 21:53 ` Jesse Brandeburg 2014-01-03 22:15 ` Bjorn Helgaas 0 siblings, 1 reply; 10+ messages in thread From: Jesse Brandeburg @ 2014-01-03 21:53 UTC (permalink / raw) To: David Miller, linux-pci@vger.kernel.org Cc: Jeff Kirsher, NetDEV list, gospo, sassmann, linux-kernel@vger.kernel.org, Bjorn Helgaas +linux-pci <and see note below> On Thu, Jan 2, 2014 at 4:21 PM, David Miller <davem@davemloft.net> wrote: > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > Date: Sat, 28 Dec 2013 04:36:13 -0800 > >> Add missing PCI bus link speed 8.0 GT/s and bus link widths of >> x1, x2, x4 and x8. >> >> CC: <linux-kernel@vger.kernel.org> >> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > Can a PCI person please ACK this? This is a prerequisite for some > networking driver changes Jeff would like to push to me. > > Thanks. > >> --- >> include/uapi/linux/pci_regs.h | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h >> index 4a98e85..c870c2a 100644 >> --- a/include/uapi/linux/pci_regs.h >> +++ b/include/uapi/linux/pci_regs.h >> @@ -489,7 +489,12 @@ >> #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ >> #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ >> #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ >> +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */ >> #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */ >> +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */ >> +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */ >> +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */ >> +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */ >> #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */ >> #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */ >> #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */ also note that these defines from this patch are already partially defined in drivers/pci/probe.c but likely need to move into a header file and be consolidated throughout the kernel. That said I don't see any issue with adding these as a separate set of defines in uapi/linux/pci_regs.h ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-03 21:53 ` Jesse Brandeburg @ 2014-01-03 22:15 ` Bjorn Helgaas 2014-01-03 23:56 ` David Miller 0 siblings, 1 reply; 10+ messages in thread From: Bjorn Helgaas @ 2014-01-03 22:15 UTC (permalink / raw) To: Jesse Brandeburg Cc: David Miller, linux-pci@vger.kernel.org, Jeff Kirsher, NetDEV list, gospo@redhat.com, sassmann@redhat.com, linux-kernel@vger.kernel.org, Bjorn Helgaas On Fri, Jan 3, 2014 at 2:53 PM, Jesse Brandeburg <jesse.brandeburg@gmail.com> wrote: > +linux-pci Thanks, Jesse. > <and see note below> > > On Thu, Jan 2, 2014 at 4:21 PM, David Miller <davem@davemloft.net> wrote: >> From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >> Date: Sat, 28 Dec 2013 04:36:13 -0800 >> >>> Add missing PCI bus link speed 8.0 GT/s and bus link widths of >>> x1, x2, x4 and x8. >>> >>> CC: <linux-kernel@vger.kernel.org> >>> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >> >> Can a PCI person please ACK this? This is a prerequisite for some >> networking driver changes Jeff would like to push to me. The additions look correct to me, and I don't object to them per se, so: Acked-by: Bjorn Helgaas <bhelgaas@google.com> However, I do raise my eyebrows a bit at drivers that poke around in the PCIe capability. I would prefer to have PCI core interfaces that handle that instead. But I haven't seen Jeff's changes yet. >>> --- >>> include/uapi/linux/pci_regs.h | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h >>> index 4a98e85..c870c2a 100644 >>> --- a/include/uapi/linux/pci_regs.h >>> +++ b/include/uapi/linux/pci_regs.h >>> @@ -489,7 +489,12 @@ >>> #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ >>> #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ >>> #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ >>> +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */ >>> #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */ >>> +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */ >>> +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */ >>> +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */ >>> +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */ >>> #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */ >>> #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */ >>> #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */ > > also note that these defines from this patch are already partially > defined in drivers/pci/probe.c but likely need to move into a header > file and be consolidated throughout the kernel. That said I don't see > any issue with adding these as a separate set of defines in > uapi/linux/pci_regs.h > -- > 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] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-03 22:15 ` Bjorn Helgaas @ 2014-01-03 23:56 ` David Miller 2014-01-04 0:00 ` Bjorn Helgaas 0 siblings, 1 reply; 10+ messages in thread From: David Miller @ 2014-01-03 23:56 UTC (permalink / raw) To: bhelgaas Cc: jesse.brandeburg, linux-pci, jeffrey.t.kirsher, netdev, gospo, sassmann, linux-kernel, bjorn.helgaas From: Bjorn Helgaas <bhelgaas@google.com> Date: Fri, 3 Jan 2014 15:15:57 -0700 > However, I do raise my eyebrows a bit at drivers that poke around in > the PCIe capability. I would prefer to have PCI core interfaces that > handle that instead. But I haven't seen Jeff's changes yet. The changes just read the link status to interpret the speed at which the PCI-E link is running at. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-03 23:56 ` David Miller @ 2014-01-04 0:00 ` Bjorn Helgaas 2014-01-04 0:30 ` David Miller 0 siblings, 1 reply; 10+ messages in thread From: Bjorn Helgaas @ 2014-01-04 0:00 UTC (permalink / raw) To: David Miller Cc: jesse.brandeburg, linux-pci@vger.kernel.org, Kirsher, Jeffrey T, netdev, gospo@redhat.com, sassmann@redhat.com, linux-kernel@vger.kernel.org, bjorn.helgaas On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 3 Jan 2014 15:15:57 -0700 > >> However, I do raise my eyebrows a bit at drivers that poke around in >> the PCIe capability. I would prefer to have PCI core interfaces that >> handle that instead. But I haven't seen Jeff's changes yet. > > The changes just read the link status to interpret the speed at which > the PCI-E link is running at. Several drivers want to do that. It'd be nice if somebody abstracted that out somehow. Jacob added pcie_get_minimum_link() which is similar. But maybe Jeff needs something more in this case. In any case, it's not a blocker for this patch. Bjorn ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-04 0:00 ` Bjorn Helgaas @ 2014-01-04 0:30 ` David Miller 2014-01-04 1:27 ` Jeff Kirsher 0 siblings, 1 reply; 10+ messages in thread From: David Miller @ 2014-01-04 0:30 UTC (permalink / raw) To: bhelgaas Cc: jesse.brandeburg, linux-pci, jeffrey.t.kirsher, netdev, gospo, sassmann, linux-kernel, bjorn.helgaas From: Bjorn Helgaas <bhelgaas@google.com> Date: Fri, 3 Jan 2014 17:00:42 -0700 > On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote: >> From: Bjorn Helgaas <bhelgaas@google.com> >> Date: Fri, 3 Jan 2014 15:15:57 -0700 >> >>> However, I do raise my eyebrows a bit at drivers that poke around in >>> the PCIe capability. I would prefer to have PCI core interfaces that >>> handle that instead. But I haven't seen Jeff's changes yet. >> >> The changes just read the link status to interpret the speed at which >> the PCI-E link is running at. > > Several drivers want to do that. It'd be nice if somebody abstracted > that out somehow. Jacob added pcie_get_minimum_link() which is > similar. But maybe Jeff needs something more in this case. > > In any case, it's not a blocker for this patch. Ok. Jeff, please merge this via the Intel submission process and don't forget to add Bjorn's ACK. Thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-04 0:30 ` David Miller @ 2014-01-04 1:27 ` Jeff Kirsher 2014-01-04 1:48 ` David Miller 0 siblings, 1 reply; 10+ messages in thread From: Jeff Kirsher @ 2014-01-04 1:27 UTC (permalink / raw) To: David Miller Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann, linux-kernel, bjorn.helgaas [-- Attachment #1: Type: text/plain, Size: 1495 bytes --] On Fri, 2014-01-03 at 19:30 -0500, David Miller wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri, 3 Jan 2014 17:00:42 -0700 > > > On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote: > >> From: Bjorn Helgaas <bhelgaas@google.com> > >> Date: Fri, 3 Jan 2014 15:15:57 -0700 > >> > >>> However, I do raise my eyebrows a bit at drivers that poke around in > >>> the PCIe capability. I would prefer to have PCI core interfaces that > >>> handle that instead. But I haven't seen Jeff's changes yet. > >> > >> The changes just read the link status to interpret the speed at which > >> the PCI-E link is running at. > > > > Several drivers want to do that. It'd be nice if somebody abstracted > > that out somehow. Jacob added pcie_get_minimum_link() which is > > similar. But maybe Jeff needs something more in this case. > > > > In any case, it's not a blocker for this patch. > > Ok. > > Jeff, please merge this via the Intel submission process and don't forget > to add Bjorn's ACK. > > Thanks. Ok, will do. What do you want to do with the current series of patches (mainly the ixgbe LER patches) on my tree? I know that there is a thread going on between Joe Perches and Mark, but it sounded like Mark is working on a follow-on patch to address Joe's concerns. Guess I want to know if I leave my tree alone for you to pull or toss the current pull request series and get the i40e series applied. Cheers, Jeff [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-04 1:27 ` Jeff Kirsher @ 2014-01-04 1:48 ` David Miller 2014-01-04 1:51 ` Jeff Kirsher 0 siblings, 1 reply; 10+ messages in thread From: David Miller @ 2014-01-04 1:48 UTC (permalink / raw) To: jeffrey.t.kirsher Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann, linux-kernel, bjorn.helgaas From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Date: Fri, 03 Jan 2014 17:27:39 -0800 > What do you want to do with the current series of patches > (mainly the ixgbe LER patches) on my tree? I know that there is a > thread going on between Joe Perches and Mark, but it sounded like Mark > is working on a follow-on patch to address Joe's concerns. I want the LER config option removed. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines 2014-01-04 1:48 ` David Miller @ 2014-01-04 1:51 ` Jeff Kirsher 0 siblings, 0 replies; 10+ messages in thread From: Jeff Kirsher @ 2014-01-04 1:51 UTC (permalink / raw) To: David Miller Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann, linux-kernel, bjorn.helgaas [-- Attachment #1: Type: text/plain, Size: 480 bytes --] On Fri, 2014-01-03 at 20:48 -0500, David Miller wrote: > From: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > Date: Fri, 03 Jan 2014 17:27:39 -0800 > > > What do you want to do with the current series of patches > > (mainly the ixgbe LER patches) on my tree? I know that there is a > > thread going on between Joe Perches and Mark, but it sounded like Mark > > is working on a follow-on patch to address Joe's concerns. > > I want the LER config option removed. Ok. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-01-04 1:51 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-12-28 12:36 [net-next] pci_regs.h: Add PCI bus link speed and width defines Jeff Kirsher 2014-01-03 0:21 ` David Miller 2014-01-03 21:53 ` Jesse Brandeburg 2014-01-03 22:15 ` Bjorn Helgaas 2014-01-03 23:56 ` David Miller 2014-01-04 0:00 ` Bjorn Helgaas 2014-01-04 0:30 ` David Miller 2014-01-04 1:27 ` Jeff Kirsher 2014-01-04 1:48 ` David Miller 2014-01-04 1:51 ` Jeff Kirsher
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox