On Fri, Feb 14, 2014 at 4:15 PM, Pravin Shelar <pshe...@nicira.com> wrote:
> I got following sparse warning.
>
> /home/pravin/ovs/w6/datapath/linux/flow_table.c:97:9: warning:
> incorrect type in argument 1 (different address spaces)
> /home/pravin/ovs/w6/datapath/linux/flow_table.c:97:9:    expected
> struct spinlock [usertype] *lock
> /home/pravin/ovs/w6/datapath/linux/flow_table.c:97:9:    got struct
> spinlock [noderef] <asn:4>*<noident>
>
>
> Otherwise looks good.

One other thing - can you make multiline comments be of the form:
 /* ds
 * sfd
 */

People sometimes complain about this. Otherwise it looks good to me as well.

Acked-by: Jesse Gross <je...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to