From: Florian Fainelli <f.fainelli@gmail.com>
To: Maxim Uvarov <muvarov@gmail.com>
Cc: netdev@vger.kernel.org
Subject: Re: port mirror on dsa switches?
Date: Thu, 16 Mar 2017 12:48:58 -0700 [thread overview]
Message-ID: <6d50e527-a2e1-5d63-1cb4-bea893fbeb9b@gmail.com> (raw)
In-Reply-To: <CAJGZr0+qyKppYr5mfAM5TOtD9RMKnrZTeCOhaBK1wV_sHdPL3A@mail.gmail.com>
On 03/16/2017 12:43 PM, Maxim Uvarov wrote:
> 2017-03-16 19:47 GMT+03:00 Florian Fainelli <f.fainelli@gmail.com>:
>> Hi,
>>
>> On 03/16/2017 03:32 AM, Maxim Uvarov wrote:
>>> Hello,
>>>
>>> Some dsa switches can support port mirror in hardware. Does somebody
>>> have any idea how to
>>> work with it from linux side in generic way?
>>
>> It has been implemented with commit
>> f50f212749e8a28803af3628acbeb85ee0458ed5 ("net: dsa: Add plumbing for
>> port mirroring") and you can see an implementation example with the b53
>> driver.
>>
>
> thanks, see that in newer kernel. It will be good to update doc with
> commands to use.
> I see from cover latter you mirrored eth1 to eth2. I assume it's dsa
> ports names for b53?
Yes, DSA will refuse mirroring to network devices that are not ports of
the switch (arguable we could allow mirroring to master network device,
but that's TODO).
> Do I also need updated tc?
Yes you do need a reasonably recent iproute2, like 4.11 or something close.
>
>> For a DSA driver you should be implementing port_mirror_add and
>> port_mirror_del operations which provide you with the necessary
>> information. Hopefully the switch you are working with (mv88e6xxx?) is
>> also supportable using that API, if not, please submit changes to extend it.
>>
>> Thanks!
>> --
>> Florian
>
> Yes, I think it will match mv88e6xxx. Will try to play with it. Thanks a lot!
>
--
Florian
prev parent reply other threads:[~2017-03-16 19:49 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-16 10:32 port mirror on dsa switches? Maxim Uvarov
2017-03-16 16:47 ` Florian Fainelli
2017-03-16 19:43 ` Maxim Uvarov
2017-03-16 19:48 ` Florian Fainelli [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6d50e527-a2e1-5d63-1cb4-bea893fbeb9b@gmail.com \
--to=f.fainelli@gmail.com \
--cc=muvarov@gmail.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).