Hi, > -----Original Message----- > From: Stephen Hemminger <[email protected]> > Sent: Sunday, August 14, 2022 10:58 PM > To: Jiang, Cheng1 <[email protected]> > Cc: [email protected]; Xia, Chenbo <[email protected]>; > [email protected]; Hu, Jiayu <[email protected]>; Ding, Xuan > <[email protected]>; Ma, WenwuX <[email protected]>; Wang, > YuanX <[email protected]>; Yang, YvonneX <[email protected]> > Subject: Re: [RFC 1/2] vhost: add ingress API for port mirroring datapath > > On Sun, 14 Aug 2022 12:49:19 +0000 > Cheng Jiang <[email protected]> wrote: > > > From: Wenwu Ma <[email protected]> > > > > Similar to the port mirroring function on the switch or router, this > > patch also implements an ingress function on the Vhost lib. When data > > is sent to a front-end, it will also send the data to its mirror > > front-end. > > > > Signed-off-by: Cheng Jiang <[email protected]> > > Signed-off-by: Wenwu Ma <[email protected]> > > We already have rte_flow, packet capture, and rx/tx callbacks. > This seems like re-invention.
Sorry that I didn't make it clear in the v1 commit message. This port mirror function is based on async vhost which is accelerated by DMA device. Compared with other mirror implements: 1. It's targeted for vhost. 2. The performance is really good. Its use scenario is to let one front-end(mirror-VM) monitor the traffic of another front-end(VM). It's different from the things you mentioned above. So, IMO I don't think it's re-invention. Thanks, Cheng
