Hi Iwan, >Hi Andrew, > >I only own a simple 5 ports switch (88E6176) which has no problem of mirroring >the other ports to a single port. Except for a bandwith shortage ofcourse. >While I thought I checked adding and removing ports, I seemed to forgot to >check removing ingress traffic as it will now >disable mirroring egress >traffic. Searching for how I can distinct ingress from egress mirroring in >port_mirror_del, I saw there is a variable in the mirror struct called >ingress. Which seems strange, because why is it a seperate argument to the >port_mirror_add function? > >Origally I planned to be able to set the egress and ingress mirror seperatly. >But in my laziness when I saw there already was a function to configure the >destination port this functionality was lost. > >Because the other drivers which implemented the port_mirror_add (b53 and >ksz9477) also lacks additional checks to prevent new mirror filters from >breaking previous ones I assumed they were not necessary. > >At least I will soon sent a new version with at least the issue of removing >mirror ingress traffic fixed and the ability to define a seperate ingress and >egress port. > >Regards, >Iwan
I have a similar patch set for port mirror from a few years ago. I'd also like to see this functionality in mainline. One issue I ran into is when doing port mirror in a cross-chip dsa configuration. If the ingress and egress ports are on different chips, the ingress chip needs to set the egress to the cross-chip dsa port and the cross-chip egress port needs to be set appropriately. I also had the functionality to mirror egress from a port to a destination port. Is it appropriate to send my patch to the mailing list for review or should we work on this off-line? Thanks, Jason