Re: [ovs-dev] [PATCH 3/3] netdev-dummy: Add Clang lock annotations.

2013-08-16 Thread Ben Pfaff
Thank you for the reviews. I applied 2 and 3 to master. (Patch 1 was applied some time ago.) On Fri, Aug 16, 2013 at 01:29:41PM -0700, Andy Zhou wrote: > Looks good. > Acked-by: Andy Zhou > > > On Mon, Aug 12, 2013 at 12:54 PM, Ben Pfaff wrote: > > > The locks and unlocks added in this comm

Re: [ovs-dev] [PATCH 3/3] netdev-dummy: Add Clang lock annotations.

2013-08-16 Thread Andy Zhou
Looks good. Acked-by: Andy Zhou On Mon, Aug 12, 2013 at 12:54 PM, Ben Pfaff wrote: > The locks and unlocks added in this commit are superfluous but should not > hurt performance and make Clang happy. > > Signed-off-by: Ben Pfaff > --- > lib/netdev-dummy.c | 31 +++--

[ovs-dev] [PATCH 3/3] netdev-dummy: Add Clang lock annotations.

2013-08-12 Thread Ben Pfaff
The locks and unlocks added in this commit are superfluous but should not hurt performance and make Clang happy. Signed-off-by: Ben Pfaff --- lib/netdev-dummy.c | 31 +++ 1 file changed, 19 insertions(+), 12 deletions(-) diff --git a/lib/netdev-dummy.c b/lib/netdev