Re: [ovs-dev] question about mirror code

2016-02-05 Thread Ben Pfaff
On Tue, Jan 26, 2016 at 12:13:08AM +0800, Huanle Han wrote: > 2016-01-25 1:00 GMT+08:00 Ben Pfaff : > > > On Sun, Jan 24, 2016 at 11:59:57PM +0800, Huanle Han wrote: > > > I think a more line "mirrors &= ~ctx->mirrors;" is needed at the end of > > the > > > "while" loop. > > > Otherwise, duplicate

Re: [ovs-dev] question about mirror code

2016-01-25 Thread Huanle Han
2016-01-25 1:00 GMT+08:00 Ben Pfaff : > On Sun, Jan 24, 2016 at 11:59:57PM +0800, Huanle Han wrote: > > I think a more line "mirrors &= ~ctx->mirrors;" is needed at the end of > the > > "while" loop. > > Otherwise, duplicated mirror action may be applied in recursive > > mirror_packet called by ou

Re: [ovs-dev] question about mirror code

2016-01-24 Thread Ben Pfaff
On Sun, Jan 24, 2016 at 11:59:57PM +0800, Huanle Han wrote: > I think a more line "mirrors &= ~ctx->mirrors;" is needed at the end of the > "while" loop. > Otherwise, duplicated mirror action may be applied in recursive > mirror_packet called by output_normal. Did you arrive at that conclusion fro

Re: [ovs-dev] question about mirror code

2016-01-24 Thread Huanle Han
Hi, Ben, I think a more line "mirrors &= ~ctx->mirrors;" is needed at the end of the "while" loop. Otherwise, duplicated mirror action may be applied in recursive mirror_packet called by output_normal. Here are the test cases: Test case for the bug I mentioned in previous mail: 1. setup port: v

Re: [ovs-dev] question about mirror code

2016-01-19 Thread Ben Pfaff
On Sat, Jan 16, 2016 at 02:21:03PM +0800, hanxueluo wrote: > Hi, guys I have a question about the code below in function > "mirror_packet". Some mirrors in "mirrors" may be skipped because the > select_vlan doesn't match ("if (vlans && !bitmap_is_set(vlans, > vlan)"), but they may be useful for nex

[ovs-dev] question about mirror code

2016-01-15 Thread hanxueluo
Hi, guys I have a question about the code below in function "mirror_packet". Some mirrors in "mirrors" may be skipped because the select_vlan doesn't match ("if (vlans && !bitmap_is_set(vlans, vlan)"), but they may be useful for next xbundle. So I think we should record only the mirrors which ar