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)
Mon, Apr 8, 5:04 PM
Unknown Object (File)
Mar 19 2024, 3:38 PM
Unknown Object (File)
Mar 19 2024, 2:58 PM
Unknown Object (File)
Mar 5 2024, 11:43 AM
Unknown Object (File)
Feb 22 2024, 7:26 PM
Unknown Object (File)
Feb 22 2024, 2:07 AM
Unknown Object (File)
Feb 22 2024, 1:58 AM
Unknown Object (File)
Feb 22 2024, 1:58 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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

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);
}
1347

Set flags to wrong if. Should be VF.

1912–1913

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

1935–1936

Should use if_name()

1953–1954

if_getinputfn()

2057
2088
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 ?

2107

Ditto

2126

Ditto

4091

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

5786

Atomics @whu

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

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

2088

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
944
2094
2106
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.