On Fri, Apr 21, 2023 at 05:16:50PM +0200, Adrian Moreno wrote: > In preparation for supporting 64-bit rates in tc policies, move the > allocation and initialization of struct tc_police object inside > nl_msg_put_act_police(). That way, the function is now called with the > actual rates. > > Signed-off-by: Adrian Moreno <[email protected]>
Tested-by: Simon Horman <[email protected]> > --- > lib/netdev-linux.c | 37 ++++++++++++++++++------------------- > 1 file changed, 18 insertions(+), 19 deletions(-) > > diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c > index 8ee75981b..a2bae300c 100644 > --- a/lib/netdev-linux.c > +++ b/lib/netdev-linux.c > @@ -2653,21 +2653,26 @@ nl_msg_act_police_end_nest(struct ofpbuf *request, > size_t offset, > } > > static void > -nl_msg_put_act_police(struct ofpbuf *request, struct tc_police *police, > +nl_msg_put_act_police(struct ofpbuf *request, uint32_t index, > + uint64_t kbits_rate, uint64_t kbits_burst, > uint64_t pkts_rate, uint64_t pkts_burst, > uint32_t notexceed_act, bool single_action) nit: not that I have a good idea of a better approach, but this has a lot of parameters now :( _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
