]> CyberLeo.Net >> Repos - FreeBSD/stable/10.git/commit
Redo MFC r300220,r300223:
authorngie <ngie@ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f>
Sat, 11 Jun 2016 11:28:29 +0000 (11:28 +0000)
committerngie <ngie@ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f>
Sat, 11 Jun 2016 11:28:29 +0000 (11:28 +0000)
commit97038629572932dd817517a6a77ffdd4b25c8812
tree2f9f99ce0b5d311b1247500d1c53d5d17acb37aa
parent13ca842e2269c3f09140b2bffd54aa8ac770fe8b
Redo MFC r300220,r300223:

Differential Revision: https://reviews.freebsd.org/D6803
Reviewed by: alc, kib
Sponsored by: EMC / Isilon Storage Division

r300220 (by cem):

sys/vmmeter.h: Fix trivial '-Wsign-compare' warning in common header

Frankly, it doesn't make sense for vm_pageout_wakeup_thresh to have a negative
value (it is only ever set to a fraction of v_free_min, which is unsigned and
also obviously non-negative). But I'm not going to try and convert every
non-negative scalar in the VM to unsigned today, so just cast it for the
comparison.

r300223 (by cem):

vm/vm_page.h: Fix trivial '-Wpointer-sign' warning

pq_vcnt, as a count of real things, has no business being negative. It is only
ever initialized by a u_int counter.

The warning came from the atomic_add_int() in vm_pagequeue_cnt_add().

Rectify the warning by changing the variable to u_int. No functional change.

Suggested by: Clang 3.3

git-svn-id: svn://svn.freebsd.org/base/stable/10@301833 ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f
sys/sys/vmmeter.h
sys/vm/vm_page.c
sys/vm/vm_page.h