d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

When allocating swap blocks, if the available number of free blocks in a

Browse Source 
subtree is already zero, then setting the "largest contiguous free block"
hint for that subtree to anything other than zero makes no sense.  To be
clear, assigning a value to the hint that is too large is not a correctness
problem, only a pessimization.

Dragonfly BSD has applied the same change to blst_meta_alloc() but not
blst_meta_fill().

MFC after:	6 weeks
This commit is contained in:
Alan Cox 2017-06-08 15:48:54 +00:00
parent f2b7bf8afc
commit 86dd278f03
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/kern/subr_blist.c
View File

@ -419,7 +419,7 @@ blst_meta_alloc(
/*
* ALL-ALLOCATED special case
*/
scan->bm_bighint = count;
scan->bm_bighint = 0;
return(SWAPBLK_NONE);
}
@ -774,7 +774,7 @@ blst_meta_fill(
*/
nblks = scan->u.bmu_avail;
scan->u.bmu_avail = 0;
scan->bm_bighint = count;
scan->bm_bighint = 0;
return nblks;
}