On 2/1/21 2:45 PM, Jakub Kicinski wrote:
Matt, would you mind clarifying if this is indeed a regression for i211?


Admittedly, I didn't go all the way back to test that it is, indeed, a regression. The Fixes commit that it was tagged with on Tony's tree was something more recent than initial igb landing (its a refactor of RSS) and there are numerous posts online indicating common I211 hardware (eg PCEngines APU2) support RSS and properly load multiple cores, so I naively assumed that it is a regression of some form.

Did you test kernels odler than c883de9fd787, Nick?

Given that, and the non-invasive nature of the patch, I figured it was worth trying to land on LTS trees and going ahead with it for 5.11, but I don't feel strongly on how it ends up on LTS, it just seems a waste to not land it there.

Matt

Reply via email to