Re: [ovs-dev] [PATCH ovs v2 0/4] expand the meter table and fix bug

2020-05-17 Thread Tonghao Zhang
On Sat, May 16, 2020 at 1:14 AM Ilya Maximets  wrote:
>
> On 5/13/20 3:31 PM, xiangxia.m@gmail.com wrote:
> > From: Tonghao Zhang 
> >
> > The patch set expand or shrink the meter table when necessary.
> > and other patches fix bug or improve codes.
> >
> > Tonghao Zhang (4):
> >   dpif-netdev: Expand the meters supported number
> >   dpif-netdev: Add burst size to buckets
> >   dpif-netdev: Use the u64 instead of u32 for buckets
> >   Revert "dpif-netdev: includes microsecond delta in meter bucket
> > calculation"
> >
> >  include/openvswitch/ofp-meter.h |   2 +-
> >  lib/dpif-netdev.c   | 336 
> >  lib/ofp-meter.c |   4 +-
> >  3 files changed, 257 insertions(+), 85 deletions(-)
> >
>
> Thanks for working on this!
>
> General note about sending patches:
> Please, don't send patches in reply to the previous version.  This messes up
> mailboxes.  Send new versions separately.
> And it's better to have a dot at the end of a patch subject.
Ok, I got it, thanks.
> Best regards, Ilya Maximets.



-- 
Best regards, Tonghao
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovs v2 0/4] expand the meter table and fix bug

2020-05-15 Thread Ilya Maximets
On 5/13/20 3:31 PM, xiangxia.m@gmail.com wrote:
> From: Tonghao Zhang 
> 
> The patch set expand or shrink the meter table when necessary.
> and other patches fix bug or improve codes.
> 
> Tonghao Zhang (4):
>   dpif-netdev: Expand the meters supported number
>   dpif-netdev: Add burst size to buckets
>   dpif-netdev: Use the u64 instead of u32 for buckets
>   Revert "dpif-netdev: includes microsecond delta in meter bucket
> calculation"
> 
>  include/openvswitch/ofp-meter.h |   2 +-
>  lib/dpif-netdev.c   | 336 
>  lib/ofp-meter.c |   4 +-
>  3 files changed, 257 insertions(+), 85 deletions(-)
> 

Thanks for working on this!

General note about sending patches:
Please, don't send patches in reply to the previous version.  This messes up
mailboxes.  Send new versions separately.
And it's better to have a dot at the end of a patch subject.

Best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovs v2 0/4] expand the meter table and fix bug

2020-05-14 Thread William Tu
On Wed, May 13, 2020 at 09:31:31PM +0800, xiangxia.m@gmail.com wrote:
> From: Tonghao Zhang 
> 
> The patch set expand or shrink the meter table when necessary.
> and other patches fix bug or improve codes.
> 
> Tonghao Zhang (4):
>   dpif-netdev: Expand the meters supported number
>   dpif-netdev: Add burst size to buckets
>   dpif-netdev: Use the u64 instead of u32 for buckets
>   Revert "dpif-netdev: includes microsecond delta in meter bucket
> calculation"
> 
>  include/openvswitch/ofp-meter.h |   2 +-
>  lib/dpif-netdev.c   | 336 
>  lib/ofp-meter.c |   4 +-
>  3 files changed, 257 insertions(+), 85 deletions(-)
> 
Thanks!
The series looks good to me. See if others have more comments.
William
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH ovs v2 0/4] expand the meter table and fix bug

2020-05-13 Thread xiangxia . m . yue
From: Tonghao Zhang 

The patch set expand or shrink the meter table when necessary.
and other patches fix bug or improve codes.

Tonghao Zhang (4):
  dpif-netdev: Expand the meters supported number
  dpif-netdev: Add burst size to buckets
  dpif-netdev: Use the u64 instead of u32 for buckets
  Revert "dpif-netdev: includes microsecond delta in meter bucket
calculation"

 include/openvswitch/ofp-meter.h |   2 +-
 lib/dpif-netdev.c   | 336 
 lib/ofp-meter.c |   4 +-
 3 files changed, 257 insertions(+), 85 deletions(-)

-- 
2.26.1

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev