svn commit: r301831 - stable/10

Garrett Cooper ngie at FreeBSD.org
Sat Jun 11 11:24:31 UTC 2016


Author: ngie
Date: Sat Jun 11 11:24:30 2016
New Revision: 301831
URL: https://svnweb.freebsd.org/changeset/base/301831

Log:
  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

Modified:
Directory Properties:
  stable/10/   (props changed)


More information about the svn-src-all mailing list