[Feature Request] egress qdisc support

[robimarko]

Hi,
Does the driver support use of egress qdiscs?

We are trying to drop all egress traffic and unlike ingress rules, it does not appear to work, we can still see traffic out of the switch ports.

https://github.com/Marvell-switching/Switchdev-prestera/wiki/acl-configuration
Also does not mention egress at all.

Regards,
Robert

[Mickey201]

Hi Robert, that's right, Marvell Switchdev drivers currently support only ingress qdiscs.
Egress ACL support is planned for one of the future releases.

[jmpolom]

@Mickey201 can you please leave this open to track progress? This feature is important to our use case.

Egress ACL support is planned for one of the future releases.

Can you provide any more concrete of a timeline than "future releases"?

cc: @storrgie

[Mickey201]

Ok, I'll reopen it and let's implement a practice of adding "[Feature Request]" in the subject field for all similar issues.
For this one - I'll add it.
As for the schedule, we currently reopening the current DENT 3 content list and we will examine also this request.
Currently, no commitment can be set.

[jmpolom]

I'll reopen it and let's implement a practice of adding "[Feature Request]" in the subject field for all similar issues

Good practice. I see you also added a label. Also a good practice.

As for the schedule, we currently reopening the current DENT 3 content list and we will examine also this request.

Is this a Marvell managed DENT 3 content list? Could we discuss this tomorrow during the FWG call and reconcile against the DENT Project proposed feature list we've been working on?

[Mickey201]

Is this a Marvell managed DENT 3 content list? Could we discuss this tomorrow during the FWG call and reconcile against the DENT Project proposed feature list we've been working on?

Sorry but I think It's too early, more information will be shared once a stable draft will be ready.

[jmpolom]

once a stable draft will be ready

Stable draft of this Marvell feature list or the DENT project list? I think we can say the DENT project feature list is stable enough, if that is the issue here. Otherwise, when do you think that this list will be stable enough on the Marvell side for a discussion?

[taraschornyiplv]

Support for egress has been added in v4 driver
Please consult wiki
https://github.com/Marvell-switching/switchdev-prestera/wiki/ACL#acl-overview