Differential D52182

umb: Fix setting the input routine
ClosedPublic
Actions

Authored by zlei on Aug 27 2025, 4:49 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jun 5, 8:37 PM

	Unknown Object (File)
	Wed, May 20, 9:30 PM

	Unknown Object (File)
	May 5 2026, 9:37 AM

	Unknown Object (File)
	Apr 27 2026, 5:00 AM

	Unknown Object (File)
	Apr 24 2026, 7:08 PM

	Unknown Object (File)
	Apr 22 2026, 7:45 AM

	Unknown Object (File)
	Apr 19 2026, 12:45 PM

	Unknown Object (File)
	Apr 16 2026, 7:53 AM

View All 57 Files

Subscribers

Details

Reviewers

khorben
philip

Group Reviewers

Commits

rGe921d2842ee8: umb: Fix setting the input routine

Summary

This driver does not depend on netmap, and umb_input() works greatly
without netmap.

Remove the #ifdef DEV_NETMAP so that when "device netmap" is not
configured this driver can still correctly pass the inbound packets to
the net stack. Otherwise the input routine will be if_input_default()
which will silently drop all inbound packets.

PR: 263783
Reported by: Andre Albsmeier <mail@fbsd2.e4m.org>
Tested by: Andre Albsmeier <mail@fbsd2.e4m.org>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Aug 27 2025, 4:49 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 27 2025, 4:49 PM

zlei requested review of this revision.Aug 27 2025, 4:49 PM

I'd like to have the fix in main before making the stable/15 branch. Please do not hesitate to ping me.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 3 2025, 5:20 PM

Closed by commit rGe921d2842ee8: umb: Fix setting the input routine (authored by zlei). · Explain Why

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rGe921d2842ee8: umb: Fix setting the input routine.

Revision Contents
Changeset List

Path

Size

sys/

dev/

usb/

net/

4 lines

Diff 161441

sys/dev/usb/net/if_umb.c

Loading...