On Mon, Jan 16, 2017 at 11:54 PM, Daniel Jurgens <dani...@mellanox.com> wrote: > On 1/16/2017 3:44 PM, Or Gerlitz wrote: >> On Mon, Jan 16, 2017 at 7:29 PM, Tariq Toukan <tar...@mellanox.com> wrote: >>> From: Daniel Jurgens <dani...@mellanox.com> >>> >>> Use CPUs on the close NUMA when setting the EQ affinity hints. >> Dan, are we sure there are no down-sides for always doing this? this >> code is probably there for many years and we're introducing here new >> behaviour to potentially to many Ms installs when they get distro >> update that includes this patch.
> I don't see a downside, this just favors using the node local CPUs before > others. OK, so this just favors before others and not limits (not in front of the code now)? would be good to improve the change log and make this clear. > I don't understand your 2nd sentence there. "Ms installs"? Millions of installs that run Linux driver.