| Message ID | 20150618031252.15654.56639.stgit@ahduyck-vm-fedora22 |
|---|---|
| State | Superseded |
| Headers | show |
On Wed, 2015-06-17 at 20:12 -0700, Alexander Duyck wrote: > This change adds the MAC address to the list of values recorded on > driver > load. The MAC address represents the serial number of the unit and > allows > us to track the value should a card be replaced in a system. > > Signed-off-by: Alexander Duyck <alexander.h.duyck@redhat.com> > --- > drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Thanks Alex, I will get this added to my queue.
On Wed, 2015-06-17 at 20:12 -0700, Alexander Duyck wrote: > This change adds the MAC address to the list of values recorded on > driver > load. The MAC address represents the serial number of the unit and > allows > us to track the value should a card be replaced in a system. > > Signed-off-by: Alexander Duyck <alexander.h.duyck@redhat.com> > --- > drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) With the recent fm10k patches that Jake submitted, this patch no longer applies cleanly. If you could re-spin your patch against my next-queue tree (dev-queue branch) that would be much appreciated.
On 06/18/2015 04:49 PM, Jeff Kirsher wrote: > On Wed, 2015-06-17 at 20:12 -0700, Alexander Duyck wrote: >> This change adds the MAC address to the list of values recorded on >> driver >> load. The MAC address represents the serial number of the unit and >> allows >> us to track the value should a card be replaced in a system. >> >> Signed-off-by: Alexander Duyck <alexander.h.duyck@redhat.com> >> --- >> drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > With the recent fm10k patches that Jake submitted, this patch no longer > applies cleanly. If you could re-spin your patch against my next-queue > tree (dev-queue branch) that would be much appreciated. I should have a new patch for you in 20 minutes or so. Just waiting on the build to finish and then I'll give it a quick test. - Alex
diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_pci.c b/drivers/net/ethernet/intel/fm10k/fm10k_pci.c index df9fda38bdd1..5db41ab3b762 100644 --- a/drivers/net/ethernet/intel/fm10k/fm10k_pci.c +++ b/drivers/net/ethernet/intel/fm10k/fm10k_pci.c @@ -1837,7 +1837,7 @@ static int fm10k_probe(struct pci_dev *pdev, fm10k_ptp_register(interface); /* print bus type/speed/width info */ - dev_info(&pdev->dev, "(PCI Express:%s Width: %s Payload: %s)\n", + dev_info(&pdev->dev, "(PCI Express:%s Width: %s Payload: %s) %pM\n", (hw->bus.speed == fm10k_bus_speed_8000 ? "8.0GT/s" : hw->bus.speed == fm10k_bus_speed_5000 ? "5.0GT/s" : hw->bus.speed == fm10k_bus_speed_2500 ? "2.5GT/s" : @@ -1849,7 +1849,7 @@ static int fm10k_probe(struct pci_dev *pdev, (hw->bus.payload == fm10k_bus_payload_128 ? "128B" : hw->bus.payload == fm10k_bus_payload_256 ? "256B" : hw->bus.payload == fm10k_bus_payload_512 ? "512B" : - "Unknown")); + "Unknown"), netdev->dev_addr); /* print warning for non-optimal configurations */ fm10k_slot_warn(interface);
This change adds the MAC address to the list of values recorded on driver load. The MAC address represents the serial number of the unit and allows us to track the value should a card be replaced in a system. Signed-off-by: Alexander Duyck <alexander.h.duyck@redhat.com> --- drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)