Page MenuHomeFreeBSD

mlx4_en: Implement SIOCGIFRSS{KEY,HASH}

Authored by on Aug 31 2017, 5:31 AM.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

gallatin accepted this revision.Aug 31 2017, 1:14 PM
gallatin added inline comments.
1905 ↗(On Diff #32537)

Why the ifdefs?

This revision is now accepted and ready to land.Aug 31 2017, 1:14 PM
1905 ↗(On Diff #32537)

I was not so sure whether I could MFC D12174 to 10-stable/11-stable or not, but I definitely want to MFC driver bits to 10-stable/11-stable.

If D12174 goes well, I probably will MFC it to 10-stable/11-stable. I think adding IOCs does not break ABI.

hselasky added inline comments.Sep 6 2017, 2:12 PM
2041 ↗(On Diff #32537)


Please add a "length" argument to rss_key and optimise away the intermediate "key" storage.


821 ↗(On Diff #32537)

BSD-style: 4-space indent after wrap.

846 ↗(On Diff #32537)

Can you optimise away this temporary storage?

2041 ↗(On Diff #32537)

Toeplitz key size is pretty standard. But, sure, I could add a length parameter.

As about optimizing away the key, it's fine w/ me; though I will have to say both code you mentioned is not on hot path.

821 ↗(On Diff #32537)

Heh, OK. This is verbatim copy of the original code.

846 ↗(On Diff #32537)


Address Hans' comment

This revision now requires review to proceed.Sep 7 2017, 3:05 AM

Thank you. Currently reviewing this patch in Mellanox.

hselasky accepted this revision.Oct 2 2017, 12:02 PM

With some modifications.

This revision is now accepted and ready to land.Oct 2 2017, 12:02 PM
This revision was automatically updated to reflect the committed changes.