Message ID | 20211207132525.26458-1-mateusz.palczewski@intel.com |
---|---|
State | Superseded |
Delegated to: | Anthony Nguyen |
Headers | show |
Series | [net,v3] i40e: Fix for displaying message regarding NVM version | expand |
Dear Mateusz, Am 07.12.21 um 14:25 schrieb Mateusz Palczewski: > When loading the i40e driver, it prints a message like: 'The driver for the > device detected a newer version of the NVM image v1.x than expected v1.y. > Please install the most recent version of the network driver.' This is > misleading as the driver is working as expected. > > Fix that by removing the second part of message and changing it from > dev_info to dev_dgb. > > Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") > Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> > --- > v3: Changed the patch after community review > v2: Fixed the fixes tag to be 12 digits long > --- > drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c > index 4664d07..c07746b 100644 > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c > @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) > > if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && > hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) > - dev_info(&pdev->dev, > - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", > + dev_dbg(&pdev->dev, > + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", > hw->aq.api_maj_ver, > hw->aq.api_min_ver, > I40E_FW_API_VERSION_MAJOR, > It’s a debug message now, so only a few people are going to notice, but a note on what to do to get rid of the message, would be still be nice. > To disable this informational message, update to a newer driver version. Kind regards, Paul
Dear Paul, >Dear Mateusz, > > >Am 07.12.21 um 14:25 schrieb Mateusz Palczewski: >> When loading the i40e driver, it prints a message like: 'The driver for the >> device detected a newer version of the NVM image v1.x than expected v1.y. >> Please install the most recent version of the network driver.' This is >> misleading as the driver is working as expected. >> >> Fix that by removing the second part of message and changing it from >> dev_info to dev_dgb. >> >> Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") >> Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> >> --- >> v3: Changed the patch after community review >> v2: Fixed the fixes tag to be 12 digits long >> --- >> drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c >> index 4664d07..c07746b 100644 >> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c >> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c >> @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) >> >> if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && >> hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) >> - dev_info(&pdev->dev, >> - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", >> + dev_dbg(&pdev->dev, >> + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", >> hw->aq.api_maj_ver, >> hw->aq.api_min_ver, >> I40E_FW_API_VERSION_MAJOR, >> > >It’s a debug message now, so only a few people are going to notice, but >a note on what to do to get rid of the message, would be still be nice. > >> To disable this informational message, update to a newer driver version. > I believe that, since the driver is working correctly, we shouldn't request any action from the user but just to inform about a NVM being newer version that expected. > >Kind regards, > >Paul Regards, Mateusz
Dear Mateusz, Am 07.12.21 um 15:23 schrieb Palczewski, Mateusz: >> Am 07.12.21 um 14:25 schrieb Mateusz Palczewski: >>> When loading the i40e driver, it prints a message like: 'The driver for the >>> device detected a newer version of the NVM image v1.x than expected v1.y. >>> Please install the most recent version of the network driver.' This is >>> misleading as the driver is working as expected. >>> >>> Fix that by removing the second part of message and changing it from >>> dev_info to dev_dgb. >>> >>> Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") >>> Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> >>> --- >>> v3: Changed the patch after community review >>> v2: Fixed the fixes tag to be 12 digits long >>> --- >>> drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c >>> index 4664d07..c07746b 100644 >>> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c >>> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c >>> @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) >>> >>> if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && >>> hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) >>> - dev_info(&pdev->dev, >>> - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", >>> + dev_dbg(&pdev->dev, >>> + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", >>> hw->aq.api_maj_ver, >>> hw->aq.api_min_ver, >>> I40E_FW_API_VERSION_MAJOR, >>> >> >> It’s a debug message now, so only a few people are going to notice, but >> a note on what to do to get rid of the message, would be still be nice. >> >>> To disable this informational message, update to a newer driver version. > > I believe that, since the driver is working correctly, we shouldn't > request any action from the user but just to inform about a NVM being > newer version that expected. Then “expected” should also be removed from the message. Kind regards, Paul
Dear Paul, >>> Am 07.12.21 um 14:25 schrieb Mateusz Palczewski: >>>> When loading the i40e driver, it prints a message like: 'The driver for the >>>> device detected a newer version of the NVM image v1.x than expected v1.y. >>>> Please install the most recent version of the network driver.' This is >>>> misleading as the driver is working as expected. >>>> >>>> Fix that by removing the second part of message and changing it from >>>> dev_info to dev_dgb. >>>> >>>> Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") >>>> Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> >>>> --- >>>> v3: Changed the patch after community review >>>> v2: Fixed the fixes tag to be 12 digits long >>>> --- >>>> drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- >>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c >>>> index 4664d07..c07746b 100644 >>>> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c >>>> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c >>>> @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) >>>> >>>> if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && >>>> hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) >>>> - dev_info(&pdev->dev, >>>> - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", >>>> + dev_dbg(&pdev->dev, >>>> + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", >>>> hw->aq.api_maj_ver, >>>> hw->aq.api_min_ver, >>>> I40E_FW_API_VERSION_MAJOR, >>>> >>> >>> It’s a debug message now, so only a few people are going to notice, but >>> a note on what to do to get rid of the message, would be still be nice. >>> >>>> To disable this informational message, update to a newer driver version. >> >> I believe that, since the driver is working correctly, we shouldn't >> request any action from the user but just to inform about a NVM being >> newer version that expected. > >Then “expected” should also be removed from the message. I don't agree. I believe that message without the information about expected NVM version will be more confusing than the on with it. Kind regards, Mateusz
Dear Mateusz, Am 08.12.21 um 11:13 schrieb Palczewski, Mateusz: >>>> Am 07.12.21 um 14:25 schrieb Mateusz Palczewski: >>>>> When loading the i40e driver, it prints a message like: 'The driver for the >>>>> device detected a newer version of the NVM image v1.x than expected v1.y. >>>>> Please install the most recent version of the network driver.' This is >>>>> misleading as the driver is working as expected. >>>>> >>>>> Fix that by removing the second part of message and changing it from >>>>> dev_info to dev_dgb. >>>>> >>>>> Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") >>>>> Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> >>>>> --- >>>>> v3: Changed the patch after community review >>>>> v2: Fixed the fixes tag to be 12 digits long >>>>> --- >>>>> drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- >>>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>>> >>>>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c >>>>> index 4664d07..c07746b 100644 >>>>> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c >>>>> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c >>>>> @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) >>>>> >>>>> if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && >>>>> hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) >>>>> - dev_info(&pdev->dev, >>>>> - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", >>>>> + dev_dbg(&pdev->dev, >>>>> + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", >>>>> hw->aq.api_maj_ver, >>>>> hw->aq.api_min_ver, >>>>> I40E_FW_API_VERSION_MAJOR, >>>>> >>>> >>>> It’s a debug message now, so only a few people are going to notice, but >>>> a note on what to do to get rid of the message, would be still be nice. >>>> >>>>> To disable this informational message, update to a newer driver version. >>> >>> I believe that, since the driver is working correctly, we shouldn't >>> request any action from the user but just to inform about a NVM being >>> newer version that expected. >> >> Then “expected” should also be removed from the message. > I don't agree. I believe that message without the information about > expected NVM version will be more confusing than the on with it. I just meant to remove the word “expected”. If something expected fails to happen, it’s unexpected, which suggests a problem. In this case it seems expected, that newer NVM versions can be present. (I do not want to bikeshed, and just voiced my user point of view.) Kind regards, Paul
diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index 4664d07..c07746b 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -15469,8 +15469,8 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent) if (hw->aq.api_maj_ver == I40E_FW_API_VERSION_MAJOR && hw->aq.api_min_ver > I40E_FW_MINOR_VERSION(hw)) - dev_info(&pdev->dev, - "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u. Please install the most recent version of the network driver.\n", + dev_dbg(&pdev->dev, + "The driver for the device detected a newer version of the NVM image v%u.%u than expected v%u.%u.\n", hw->aq.api_maj_ver, hw->aq.api_min_ver, I40E_FW_API_VERSION_MAJOR,
When loading the i40e driver, it prints a message like: 'The driver for the device detected a newer version of the NVM image v1.x than expected v1.y. Please install the most recent version of the network driver.' This is misleading as the driver is working as expected. Fix that by removing the second part of message and changing it from dev_info to dev_dgb. Fixes: 4fb29bddb ("i40e: The driver now prints the API version in error message") Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com> --- v3: Changed the patch after community review v2: Fixed the fixes tag to be 12 digits long --- drivers/net/ethernet/intel/i40e/i40e_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)