Page MenuHomeFreeBSD
Differential D5784

Bump bio_cmd and bio_*flags from 8 bits to 16.
ClosedPublic
Actions

Authored by imp on Mar 30 2016, 7:55 PM.
Tags
None
Referenced Files
F81976847: D5784.id15181.diff
Wed, Apr 24, 12:41 AM
F81969404: D5784.id15181.diff
Tue, Apr 23, 9:49 PM
Unknown Object (File)
Fri, Apr 19, 12:20 PM
Unknown Object (File)
Fri, Mar 29, 12:38 AM
Unknown Object (File)
Mar 23 2024, 1:41 AM
Unknown Object (File)
Feb 3 2024, 10:09 AM
Unknown Object (File)
Dec 20 2023, 12:01 AM
Unknown Object (File)
Dec 13 2023, 6:58 PM
View All 43 Files
Subscribers
None

Details

Reviewers
kib
scottl
ken
mckusick
Commits
rS297955: Bump bio_cmd and bio_*flags from 8 bits to 16.
Summary

Ken Merry needs some additional commands for his SMR work
I need at least one additional command and some flags for my io backpressure work

So I'm bumping these from 8 bits to 16 to keep alignments sane, etc,

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp updated this revision to Diff 14748.Mar 30 2016, 7:55 PM
imp retitled this revision from to Bump bio_cmd and bio_*flags from 8 bits to 16..
imp updated this object.
imp edited the test plan for this revision. (Show Details)
imp updated this object.Mar 30 2016, 7:57 PM
imp added reviewers: ken, scottl, kib, mckusick.
ken accepted this revision.Mar 30 2016, 8:17 PM
ken edited edge metadata.

Looks good to me!

This revision is now accepted and ready to land.Mar 30 2016, 8:17 PM
scottl accepted this revision.Mar 30 2016, 8:27 PM
scottl edited edge metadata.
imp updated this object.Mar 30 2016, 10:59 PM
imp edited edge metadata.
kib accepted this revision.Mar 31 2016, 5:30 AM
kib edited edge metadata.

Note that 32bit accesses are slightly faster than 16bit, on x86. I would go with uint if touching this.

imp added a comment.Apr 11 2016, 7:54 PM
In D5784#123709, @kib wrote:

Note that 32bit accesses are slightly faster than 16bit, on x86. I would go with uint if touching this.

I've thought about this for a few days now. I think I'm going to pass on the suggestion. I don't want to push the end of the cached bit of the bio too far out.
Plus I don't see the need to have them so big.

Closed by commit rS297955: Bump bio_cmd and bio_*flags from 8 bits to 16. (authored by imp). · Explain WhyApr 14 2016, 5:10 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
geom/
4 lines
4 lines
mirror/
2 lines
raid3/
2 lines
sys/
8 lines
4 lines

Diff 15181

View Options

head/sys/geom/geom_io.c

Loading...
View Options

head/sys/geom/geom_subr.c

Loading...
View Options

head/sys/geom/mirror/g_mirror.c

Loading...
View Options

head/sys/geom/raid3/g_raid3.c

Loading...
View Options

head/sys/sys/bio.h

Loading...
View Options

head/sys/sys/buf.h

Loading...