> -----Original Message----- > From: Justin Pettit [mailto:[email protected]] > Sent: Thursday, January 11, 2018 6:58 PM > To: Stokes, Ian <[email protected]> > Cc: [email protected] > Subject: Re: [PATCH 1/1] odp-util: Fix Sparse warning. > > > > > On Jan 11, 2018, at 9:58 AM, Ian Stokes <[email protected]> wrote: > > > > Sparse complains with warning: incorrect type in argument 1 (different > > base types) in function parse_odp_userspace_action due to a call to > > htonll(rule_cookie). Rule_cookie variable is already ovs_be64 so fix > > this by removing the call to htonll as there is no need to convert for > > use with put_32aligned_be64(). > > > > CC: Justin Pettit <[email protected]> > > Fixes: d39ec23de384 ("ofproto-dpif: Don't slow-path controller > > actions.") > > Signed-off-by: Ian Stokes <[email protected]> > > --- > > lib/odp-util.c | 3 +-- > > 1 files changed, 1 insertions(+), 2 deletions(-) > > > > diff --git a/lib/odp-util.c b/lib/odp-util.c index f8c84e1..fa1a5c9 > > 100644 > > --- a/lib/odp-util.c > > +++ b/lib/odp-util.c > > @@ -1245,8 +1245,7 @@ parse_odp_userspace_action(const char *s, struct > ofpbuf *actions) > > cookie.controller.continuation = continuation ? true : > false; > > cookie.controller.reason = reason; > > cookie.controller.recirc_id = recirc_id; > > - put_32aligned_be64(&cookie.controller.rule_cookie, > > - htonll(rule_cookie)); > > + put_32aligned_be64(&cookie.controller.rule_cookie, > > + rule_cookie); > > cookie.controller.controller_id = controller_id; > > cookie.controller.max_len = max_len; > > } else if (ovs_scan(&s[n], ",userdata(%n", &n1)) { > > Thanks for pointing out the issue, Ian. I don't think this is the > solution, though, since it causes unit test failure. I'll send a follow- > up patch that should address the issue.
Ah, I kicked travis off before leaving the office and after seeing the initial gcc compilation test was now passing, I thought it was good, lesson learned :(. Ian > > --Justin > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
