On Wed, Jul 29, 2015 at 12:53 PM, Joe Stringer <joestrin...@nicira.com> wrote: > Signed-off-by: Joe Stringer <joestrin...@nicira.com> > --- > datapath/datapath.c | 25 +++++++++++++------------ > datapath/datapath.h | 2 +- > 2 files changed, 14 insertions(+), 13 deletions(-)
If I remember correctly the reason why these weren't marked const is because older kernels had functions that took these parameters as non-const and it provoked warnings. However, it seems to me that we have mostly redefined the places where these are passed, so perhaps it's not an issue anymore. Can you double check to see if that's the case? If so, this looks good to me. Acked-by: Jesse Gross <je...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev