Hello. On 22-11-2013 0:52, Ben Pfaff wrote:
Following commit (''netlink: Do not enforce alignment of last Netlink attribute''), signal the ability to receive unaligned Netlink messages to the datapath to enable utilization of zerocopy optimizations.
Signed-off-by: Thomas Graf <tg...@redhat.com>[...]
diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c index 1dcf321..5c952bc 100644 --- a/lib/dpif-linux.c +++ b/lib/dpif-linux.c[...]@@ -1839,6 +1841,10 @@ dpif_linux_dp_to_ofpbuf(const struct dpif_linux_dp *dp, struct ofpbuf *buf) nl_msg_put_u32(buf, OVS_DP_ATTR_UPCALL_PID, *dp->upcall_pid); } + if (dp->user_features) { + nl_msg_put_u32(buf, OVS_DP_ATTR_USER_FEATURES, dp->user_features); + } +{} not needed here.
This is Open vSwitch code,
I didn't put much attention at the file name or subject but the issue seemed just too obvious...
so it follows Open vSwitch coding style,
which requires the {}.
Strange style. Oh well... :-) WBR, Sergei _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
