Page MenuHomeFreeBSD

ice: Use IfAPI accessor in place of if_capenable reference
AbandonedPublic

Authored by jhibbits on Mar 6 2023, 4:56 PM.
Tags
None
Referenced Files
F102734173: D38929.diff
Sat, Nov 16, 12:27 PM
Unknown Object (File)
Sat, Oct 19, 1:21 AM
Unknown Object (File)
Sep 23 2024, 1:35 AM
Unknown Object (File)
Sep 21 2024, 3:25 PM
Unknown Object (File)
Sep 8 2024, 3:14 PM
Unknown Object (File)
Sep 3 2024, 6:05 AM
Unknown Object (File)
Aug 22 2024, 5:16 AM
Unknown Object (File)
Aug 15 2024, 6:47 AM
Subscribers

Details

Reviewers
erj
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

8923de5905 (Update ice to 1.37.7-k) moved to using the ifnet's
capenable instead of one in its own softc in one location. Use the
IfAPI accessor instead.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 50169
Build 47061: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Mar 6 2023, 4:56 PM

@erj was the original change to this intended? Before the change it was:

if ((scctx->isc_capenable & IFCAP_RXCSUM) != 0)

From the rest of the file it looks unintended, but I can't tell.

@erj was the original change to this intended? Before the change it was:

if ((scctx->isc_capenable & IFCAP_RXCSUM) != 0)

From the rest of the file it looks unintended, but I can't tell.

Yeah, I think that change was unintentional.

This comment was removed by erj.

@erj can you fix this the right way given what you also wrote in your now deleted comment? I could just revert the original change to the line if that's all that's needed.

@erj can you fix this the right way given what you also wrote in your now deleted comment? I could just revert the original change to the line if that's all that's needed.

I just reverted the change; we don't need to do something different with it just yet.

In D38929#893449, @erj wrote:

I just reverted the change; we don't need to do something different with it just yet.

Thanks! I'll abandon this now.