Page MenuHomeFreeBSD

Mechanically convert if_hn(4) to DrvAPI
ClosedPublic

Authored by jhibbits on Dec 22 2022, 3:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 12:29 PM
Unknown Object (File)
Tue, Nov 5, 2:57 PM
Unknown Object (File)
Fri, Oct 18, 3:33 AM
Unknown Object (File)
Thu, Oct 17, 6:07 PM
Unknown Object (File)
Wed, Oct 16, 5:08 PM
Unknown Object (File)
Tue, Oct 15, 1:46 PM
Unknown Object (File)
Oct 14 2024, 11:20 AM
Unknown Object (File)
Oct 12 2024, 9:49 AM
Subscribers

Diff Detail

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

Event Timeline

zlei added inline comments.
sys/dev/hyperv/netvsc/if_hn.c
956

in if_llmaddr_count() there's check against AF_LINK, is this line change identical ?

u_int   
if_llmaddr_count(if_t ifp)
{       
...
        CK_STAILQ_FOREACH(ifma, &ifp->if_multiaddrs, ifma_link)
                 if (ifma->ifma_addr->sa_family == AF_LINK)
                         count++;
...
        return (count);
}
1359

Set flags to wrong if. Should be VF.

1925

"in transparent VF mode.\n" is truncated.

1948

Should use if_name()

1967

if_getinputfn()

2070
2101
SYSCTL_ADD_UINT(ctx, child, OID_AUTO, "tso_max",
	    CTLFLAG_RD, &ifp->if_hw_tsomax, 0, "max TSO size");

tso_max sysctl node is readonly, is if_sethwtsomax() really needed ?

2120

Ditto

2139

Ditto

4109

CC @whu
Originally use atomic_clear_int(), is the change sufficient ?

5878

Atomics @whu

jhibbits added inline comments.
sys/dev/hyperv/netvsc/if_hn.c
956

You're right. Maybe I need a new API, if_has_multi(), for this.

2101

Oops, I misread the sysctl declaration.

jhibbits marked an inline comment as done.

Update diff after (several) rebases. Intended to do this 2 months ago. I hope I've addressed everything.

Generally looks good to me, once D39493 is committed.

sys/dev/hyperv/netvsc/if_hn.c
956
2107
2119
This revision is now accepted and ready to land.Apr 26 2023, 2:34 AM
This revision was automatically updated to reflect the committed changes.