Page MenuHomeFreeBSD
Differential D5922

usb: malloc with M_WAITOK never returns NULL
Needs ReviewPublic
Actions

Authored by howard0su_gmail.com on Apr 12 2016, 12:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 1, 2:36 AM
Unknown Object (File)
Sat, Nov 29, 2:44 AM
Unknown Object (File)
Fri, Nov 28, 4:34 PM
Unknown Object (File)
Thu, Nov 20, 5:26 PM
Unknown Object (File)
Nov 2 2025, 1:14 AM
Unknown Object (File)
Oct 24 2025, 2:09 AM
Unknown Object (File)
Oct 21 2025, 12:15 AM
Unknown Object (File)
Oct 20 2025, 12:50 AM
View All Files
Subscribers
imp
Contributor Reviews (src)
This revision needs review, but all specified reviewers are disabled or inactive.

Details

Reviewers
• hselasky

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3225
Build 3258: arc lint + arc unit

Event Timeline

howard0su_gmail.com updated this revision to Diff 15093.Apr 12 2016, 12:43 AM
howard0su_gmail.com retitled this revision from to usb: malloc with M_WAITOK never returns NULL.
howard0su_gmail.com updated this object.
howard0su_gmail.com edited the test plan for this revision. (Show Details)
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptApr 12 2016, 12:43 AM
howard0su_gmail.com added a reviewer: • hselasky.Apr 12 2016, 2:23 PM
• hselasky edited edge metadata.Apr 12 2016, 3:00 PM

Hi,

In the USB core M_NOWAIT should be used, and NULL checks should be preserved. The NULL checks are active for embedded products (non-FreeBSD) where the malloc implementation does not support M_WAITOK.

--HPS

howard0su_gmail.com added a comment.Apr 12 2016, 3:33 PM
In D5922#126253, @hselasky wrote:

Hi,

In the USB core M_NOWAIT should be used, and NULL checks should be preserved. The NULL checks are active for embedded products (non-FreeBSD) where the malloc implementation does not support M_WAITOK.

--HPS

So all M_WAITOK should be replaced with M_NOWAIT?

• hselasky added a comment.Apr 13 2016, 6:55 AM

Where there are NULL checks M_NOWAIT is the correct option for malloc(). Else use M_WAITOK.

--HPS

• hselasky added a comment.Jan 23 2020, 2:26 PM

Can you rebase this patch?

Revision Contents
Changeset List

PathSize
sys/
dev/
usb/
input/
8 lines
net/
4 lines
serial/
3 lines
storage/
4 lines
15 lines
11 lines
5 lines
3 lines
7 lines

Diff 15093

View Options

sys/dev/usb/input/uhid.c

Loading...
View Options

sys/dev/usb/net/if_kue.c

Loading...
View Options

sys/dev/usb/serial/ufoma.c

Loading...
View Options

sys/dev/usb/storage/ustorage_fs.c

Loading...
View Options

sys/dev/usb/usb_dev.c

Loading...
View Options

sys/dev/usb/usb_device.c

Loading...
View Options

sys/dev/usb/usb_generic.c

Loading...
View Options

sys/dev/usb/usb_mbuf.c

Loading...
View Options

sys/dev/usb/usb_transfer.c

Loading...