#
292417 |
|
17-Dec-2015 |
jhb |
MFC 290429: When dumping an rman in DDB, include the RID of each resource.
|
#
269383 |
|
01-Aug-2014 |
truckman |
MFC r268780
Nuke the never-used RF_TIMESHARE feature, reducing the complexity of the code. The consensus on arch@ is that this feature might have been useful in the distant past, but is now just unnecessary bloat.
The int_rman_activate_resource() and int_rman_deactivate_resource() functions become trivial, so manually inline them.
The special deferred handling of RF_ACTIVE is no longer needed in reserve_resource_bound(), so eliminate the associated code at the end of the function.
These changes reduce the object file size by more than 500 bytes on i386.
Update the rman.9 man page to reflect the removal of the RF_TIMESHARE feature.
|
#
268821 |
|
17-Jul-2014 |
truckman |
MFC r268373
Declaration whitespace changes for style(9).
|
#
268167 |
|
02-Jul-2014 |
truckman |
MFC r266814
Initialize r_flags the same way in all cases using a sanitized copy of flags that has several bits cleared. The RF_WANTED and RF_FIRSTSHARE bits are invalid in this context, and we want to defer setting RF_ACTIVE in r_flags until later. This should make rman_get_flags() return the correct answer in all cases.
Add a KASSERT() to catch callers which incorrectly pass the RF_WANTED or RF_FIRSTSHARE flags.
Do a strict equality check on the share type bits of flags. In particular, do an equality check on RF_PREFETCHABLE. The previous code would allow one type of mismatch of RF_PREFETCHABLE but disallow the other type of mismatch. Also, ignore the the RF_ALIGNMENT_MASK bits since alignment validity should be handled by the amask check. This field contains an integer value, but previous code did a strange bitwise comparison on it.
Leave the original value of flags unmolested as a minor debug aid.
Change the start+amask overflow check to a KASSERT() since it is just meant to catch a highly unlikely programming error in the caller.
Reviewed by: jhb
|
#
266521 |
|
21-May-2014 |
truckman |
MFC r266426
Slightly restructure the final loop in rman_reserve_resource_bound(). Replace with the existing loop termination test with a similar condition from the nested "if" that may terminate the loop a bit sooner, but still not too early. This condition can then be removed from the nested "if". Relocate an operator to be style(9) compliant.
|
#
266428 |
|
19-May-2014 |
truckman |
Be even more paranoid about overflow.
Requested by: ache
|
#
266427 |
|
19-May-2014 |
truckman |
Nuke a couple of unnecessary assigments. Nothing uses the values of rstart and rend after this point.
|
#
265901 |
|
12-May-2014 |
truckman |
MFC r265363
Avoid unsigned integer overflow which can cause rman_reserve_resource_bound() to return incorrect results.
Continue the initial search until the first viable region is found.
Add a comment to explain the search termination test.
PR: kern/188534 Reviewed by: jhb (previous version)
|
#
269383 |
|
01-Aug-2014 |
truckman |
MFC r268780
Nuke the never-used RF_TIMESHARE feature, reducing the complexity of the code. The consensus on arch@ is that this feature might have been useful in the distant past, but is now just unnecessary bloat.
The int_rman_activate_resource() and int_rman_deactivate_resource() functions become trivial, so manually inline them.
The special deferred handling of RF_ACTIVE is no longer needed in reserve_resource_bound(), so eliminate the associated code at the end of the function.
These changes reduce the object file size by more than 500 bytes on i386.
Update the rman.9 man page to reflect the removal of the RF_TIMESHARE feature.
|
#
268821 |
|
17-Jul-2014 |
truckman |
MFC r268373
Declaration whitespace changes for style(9).
|
#
268167 |
|
02-Jul-2014 |
truckman |
MFC r266814
Initialize r_flags the same way in all cases using a sanitized copy of flags that has several bits cleared. The RF_WANTED and RF_FIRSTSHARE bits are invalid in this context, and we want to defer setting RF_ACTIVE in r_flags until later. This should make rman_get_flags() return the correct answer in all cases.
Add a KASSERT() to catch callers which incorrectly pass the RF_WANTED or RF_FIRSTSHARE flags.
Do a strict equality check on the share type bits of flags. In particular, do an equality check on RF_PREFETCHABLE. The previous code would allow one type of mismatch of RF_PREFETCHABLE but disallow the other type of mismatch. Also, ignore the the RF_ALIGNMENT_MASK bits since alignment validity should be handled by the amask check. This field contains an integer value, but previous code did a strange bitwise comparison on it.
Leave the original value of flags unmolested as a minor debug aid.
Change the start+amask overflow check to a KASSERT() since it is just meant to catch a highly unlikely programming error in the caller.
Reviewed by: jhb
|
#
266521 |
|
21-May-2014 |
truckman |
MFC r266426
Slightly restructure the final loop in rman_reserve_resource_bound(). Replace with the existing loop termination test with a similar condition from the nested "if" that may terminate the loop a bit sooner, but still not too early. This condition can then be removed from the nested "if". Relocate an operator to be style(9) compliant.
|
#
266428 |
|
19-May-2014 |
truckman |
Be even more paranoid about overflow.
Requested by: ache
|
#
266427 |
|
19-May-2014 |
truckman |
Nuke a couple of unnecessary assigments. Nothing uses the values of rstart and rend after this point.
|
#
265901 |
|
12-May-2014 |
truckman |
MFC r265363
Avoid unsigned integer overflow which can cause rman_reserve_resource_bound() to return incorrect results.
Continue the initial search until the first viable region is found.
Add a comment to explain the search termination test.
PR: kern/188534 Reviewed by: jhb (previous version)
|