Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RTNETLINK answers: Operation not supported #2

Open
stephandave78 opened this issue Apr 11, 2019 · 3 comments
Open

RTNETLINK answers: Operation not supported #2

stephandave78 opened this issue Apr 11, 2019 · 3 comments

Comments

@stephandave78
Copy link

Hi @jeez @vcgomes ,

I was reading about the previously closed issue with respect to loading of ETF in dual core PCs. I was able to follow that and I got the mqprio qdisc loaded in my system successfully.

But when I try adding ETF as child to mqprio qdisc, system replies me " RTNETLINK answers: Operation not supported" error. Help me in solving this.

System details:
Linux x86 with 4.19 kernel and iproute2 4.19 version

Commands used:
sudo tc qdisc replace dev IFACE parent root handle 100 mqprio num_tc 2 map 1 1 1 0 1 1 1 1 1 1 1 1 1 1 1 1 queues 1@0 1@1 hw 0
sudo tc qdisc add dev IFACE parent 100:1 etf offload clockid CLOCK_TAI delta 150000

@vcgomes
Copy link
Contributor

vcgomes commented Apr 11, 2019

What are the network controllers that you are using? Right now, only Intel i210's support ETF offloading.

@stephandave78
Copy link
Author

stephandave78 commented Apr 12, 2019

@vcgomes
At our end, we have i211 network controller. Is there any possibility of patching up the igb driver to support ETF offload in i211 controller?

Thanks.

@vcgomes
Copy link
Contributor

vcgomes commented Apr 12, 2019

Reading the i211 datasheet, I get the impression that it doesn't support the LaunchTime feature really, as the datasheet doesn't have any references to the TQAVCTRL register, which enabled the mode that enables the LaunchTime transmit descriptor to be used.

But if you want to confirm it from your side in practice, something like this patch (not tested) might do the job:

https://gist.github.com/vcgomes/960af8f98424ba18ba531241fa22be61

But I don't have high hopes for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants