mbox series

[0/2] fix defects of macro in VF

Message ID 20200508015913.48764-1-guinanx.sun@intel.com (mailing list archive)
Headers
Series fix defects of macro in VF |

Message

Guinan Sun May 8, 2020, 1:59 a.m. UTC
  The defects of UPDATE_VF_STAT and UPDATE_VF_STAT_36BIT exist.
If latest is less than last, we will get wrong result.
The issues exist only in ixgbe and e1000 NICs.

Guinan Sun (2):
  net/ixgbe: fix defects of macro in VF
  net/e1000: fix defects of macro in VF

 drivers/net/e1000/igb_ethdev.c   | 14 +++++++++++---
 drivers/net/ixgbe/ixgbe_ethdev.c | 20 ++++++++++++++++++--
 2 files changed, 29 insertions(+), 5 deletions(-)
  

Comments

Zhao1, Wei May 8, 2020, 2:18 a.m. UTC | #1
Hi,
	Please alaign " \" in each line.


> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Guinan Sun
> Sent: Friday, May 8, 2020 9:59 AM
> To: dev@dpdk.org
> Cc: Sun, GuinanX <guinanx.sun@intel.com>
> Subject: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> 
> The defects of UPDATE_VF_STAT and UPDATE_VF_STAT_36BIT exist.
> If latest is less than last, we will get wrong result.
> The issues exist only in ixgbe and e1000 NICs.
> 
> Guinan Sun (2):
>   net/ixgbe: fix defects of macro in VF
>   net/e1000: fix defects of macro in VF
> 
>  drivers/net/e1000/igb_ethdev.c   | 14 +++++++++++---
>  drivers/net/ixgbe/ixgbe_ethdev.c | 20 ++++++++++++++++++--
>  2 files changed, 29 insertions(+), 5 deletions(-)
> 
> --
> 2.17.1
  
Guinan Sun May 8, 2020, 5:07 a.m. UTC | #2
Hi ZhaoWei

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Friday, May 8, 2020 10:19 AM
> To: Sun, GuinanX <guinanx.sun@intel.com>; dev@dpdk.org
> Cc: Sun, GuinanX <guinanx.sun@intel.com>; Guo, Jia <jia.guo@intel.com>
> Subject: RE: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> 
> Hi,
> Please alaign " \" in each line.

It will be modified in V2 patch.
Remark: It can't be aligned in vim tool and email at the same time.
It will be aligned in vim tool in v2 patch.

> 
> 
> > -----Original Message-----
> > From: dev <dev-bounces@dpdk.org> On Behalf Of Guinan Sun
> > Sent: Friday, May 8, 2020 9:59 AM
> > To: dev@dpdk.org
> > Cc: Sun, GuinanX <guinanx.sun@intel.com>
> > Subject: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> >
> > The defects of UPDATE_VF_STAT and UPDATE_VF_STAT_36BIT exist.
> > If latest is less than last, we will get wrong result.
> > The issues exist only in ixgbe and e1000 NICs.
> >
> > Guinan Sun (2):
> >   net/ixgbe: fix defects of macro in VF
> >   net/e1000: fix defects of macro in VF
> >
> >  drivers/net/e1000/igb_ethdev.c   | 14 +++++++++++---
> >  drivers/net/ixgbe/ixgbe_ethdev.c | 20 ++++++++++++++++++--
> >  2 files changed, 29 insertions(+), 5 deletions(-)
> >
> > --
> > 2.17.1
>
  
Zhao1, Wei May 8, 2020, 5:41 a.m. UTC | #3
> -----Original Message-----
> From: Sun, GuinanX <guinanx.sun@intel.com>
> Sent: Friday, May 8, 2020 1:07 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> Cc: Guo, Jia <jia.guo@intel.com>
> Subject: RE: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> 
> Hi ZhaoWei
> 
> > -----Original Message-----
> > From: Zhao1, Wei
> > Sent: Friday, May 8, 2020 10:19 AM
> > To: Sun, GuinanX <guinanx.sun@intel.com>; dev@dpdk.org
> > Cc: Sun, GuinanX <guinanx.sun@intel.com>; Guo, Jia <jia.guo@intel.com>
> > Subject: RE: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> >
> > Hi,
> > Please alaign " \" in each line.
> 
> It will be modified in V2 patch.
> Remark: It can't be aligned in vim tool and email at the same time.
> It will be aligned in vim tool in v2 patch.
> 
> >

Ok.

> >
> > > -----Original Message-----
> > > From: dev <dev-bounces@dpdk.org> On Behalf Of Guinan Sun
> > > Sent: Friday, May 8, 2020 9:59 AM
> > > To: dev@dpdk.org
> > > Cc: Sun, GuinanX <guinanx.sun@intel.com>
> > > Subject: [dpdk-dev] [PATCH 0/2] fix defects of macro in VF
> > >
> > > The defects of UPDATE_VF_STAT and UPDATE_VF_STAT_36BIT exist.
> > > If latest is less than last, we will get wrong result.
> > > The issues exist only in ixgbe and e1000 NICs.
> > >
> > > Guinan Sun (2):
> > >   net/ixgbe: fix defects of macro in VF
> > >   net/e1000: fix defects of macro in VF
> > >
> > >  drivers/net/e1000/igb_ethdev.c   | 14 +++++++++++---
> > >  drivers/net/ixgbe/ixgbe_ethdev.c | 20 ++++++++++++++++++--
> > >  2 files changed, 29 insertions(+), 5 deletions(-)
> > >
> > > --
> > > 2.17.1
> >
>