On Wed, Jul 29, 2015 at 12:53 PM, Joe Stringer <[email protected]> wrote: > Signed-off-by: Joe Stringer <[email protected]> > --- > 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 <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
