Differential D56138

ure: lock MII bus operations
Needs ReviewPublic
Actions

Authored by rkitover_gmail.com on Sat, Mar 28, 10:50 PM.

Tags

Referenced Files

	Unknown Object (File)
	Mon, Mar 30, 7:38 PM

	Unknown Object (File)
	Mon, Mar 30, 7:38 PM

	Unknown Object (File)
	Mon, Mar 30, 5:42 PM

	Unknown Object (File)
	Sat, Mar 28, 11:52 PM

Subscribers

Details

Reviewers

adrian
sanpei
pouria
zlei

Summary

A race condition in the MII layer causes spurious events, see:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=252165

. Use a lock to serialize access to MII operations and drop ticks when
MII is locked.

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71788
Build 68671: arc lint + arc unit

Event Timeline

rkitover_gmail.com created this revision.Sat, Mar 28, 10:50 PM

Herald added a subscriber: imp. · View Herald TranscriptSat, Mar 28, 10:50 PM

rkitover_gmail.com requested review of this revision.Sat, Mar 28, 10:50 PM

Harbormaster completed remote builds in B71787: Diff 174463.Sat, Mar 28, 10:50 PM

Add Differential Revision

Harbormaster completed remote builds in B71788: Diff 174464.Sat, Mar 28, 10:52 PM

rkitover_gmail.com added projects: USB, desktop.Sat, Mar 28, 10:53 PM

rkitover_gmail.com added reviewers: adrian, sanpei, pouria, zlei.

rkitover_gmail.com mentioned this in D55682: ure: fix spurious link flaps from MII.Sat, Mar 28, 10:55 PM

I'd be happy to review this, but I need reproduction steps.

I can reproduce below steps

Device: Raspberry Pi Zero 2W(arm64)
OS: 15.0-RELEASE + (D55682) (maybe also -current)
Setup : CARP configured on the ure interface.
Dongle: Lenovo USB-C to LAN

Steps:
Machine A (Remote): Run ping [Unassigned Local IP].
Machine B (Target): Run the route -n monitor command in four separate instances and more.

Result: NG (Failed)

and this change is fixed(I'm still testing some devices. After finish, I will report it)

Does anyone know how I can get an adapter or hub device with the C variant chip (latest revision?)

Revision Contents
Changeset List

Path

Size

sys/

dev/

usb/

net/

25 lines

4 lines

Diff 174464

sys/dev/usb/net/if_ure.c

Loading...

sys/dev/usb/net/if_urereg.h

Loading...