Page MenuHomeFreeBSD
Differential D13835

Make use of mallocarray(9) in drm drivers.
ClosedPublic
Actions

Authored by pfg on Jan 10 2018, 8:40 PM.
Tags
None
Referenced Files
F135502430: D13835.id37871.diff
Mon, Nov 10, 9:09 AM
F135447439: D13835.id37871.diff
Sun, Nov 9, 11:09 PM
F135445878: D13835.id38314.diff
Sun, Nov 9, 10:53 PM
F135436491: D13835.diff
Sun, Nov 9, 8:48 PM
F135435658: D13835.id37748.diff
Sun, Nov 9, 8:35 PM
F135435561: D13835.id.diff
Sun, Nov 9, 8:34 PM
F135419088: D13835.id37871.diff
Sun, Nov 9, 4:52 PM
Unknown Object (File)
Sun, Nov 9, 4:06 PM
View All 86 Files
Subscribers
imp

Details

Reviewers
dumbbell
pfg
Commits
rS328261: drm2: Basic use of mallocarray(9).
Summary

These seem like easy cases where we can use mallocarray(9), however I am
unsure if doing such changes will increase the difficulty for future
merges.

Diff Detail

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

Event Timeline

pfg created this revision.Jan 10 2018, 8:40 PM
Harbormaster completed remote builds in B14262: Diff 37748.Jan 10 2018, 8:40 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 10 2018, 8:40 PM
dumbbell added a comment.Jan 10 2018, 10:52 PM

Hi!

What is the benefit of using mallocarray() here?

pfg added a comment.Jan 10 2018, 11:52 PM
In D13835#290226, @dumbbell wrote:

Hi!

What is the benefit of using mallocarray() here?

TBH, I don't think there is any chance of an overflow with real hardware. Perhaps in a virtualized world there may be a chance to cause an overflow on anything or with specialized hardware (like the PS4 hackers did) but that's pretty extreme.
I am trying to keep the kernel consistent using mallocarray(9) where it can be used, but I am OK if you don't want this.

The code should be eaiser to read, but it will also make merging back the code to stable a nightmare.

pfg updated this revision to Diff 37871.Jan 12 2018, 4:44 PM

This should be a minimal and more straightforward replacement.

Note that we now panic if there is an overflow.

Harbormaster completed remote builds in B14335: Diff 37871.Jan 12 2018, 4:44 PM
pfg accepted this revision.Jan 22 2018, 2:35 AM

This is small enough that shouldn't affect upstream merging (also the code is very FreeBSD-specific).
It also respects the general objective of the code.

This revision is now accepted and ready to land.Jan 22 2018, 2:35 AM
dumbbell accepted this revision.Jan 22 2018, 7:39 AM

Yes, that's fine, the patch is small enough. Thank you!

Closed by commit rS328261: drm2: Basic use of mallocarray(9). (authored by pfg). · Explain WhyJan 22 2018, 3:56 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
dev/
drm2/
8 lines

Diff 37871

View Options

sys/dev/drm2/drm_mem_util.h

Loading...