Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 16 Mar 2013 22:53:05 +0000 (UTC)
From:      Neel Natu <neel@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r248392 - head/sys/amd64/vmm
Message-ID:  <201303162253.r2GMr5O0039599@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: neel
Date: Sat Mar 16 22:53:05 2013
New Revision: 248392
URL: http://svnweb.freebsd.org/changeset/base/248392

Log:
  Fix the '-Wtautological-compare' warning emitted by clang for comparing the
  unsigned enum type with a negative value.
  
  Obtained from:	NetApp

Modified:
  head/sys/amd64/vmm/vmm.c

Modified: head/sys/amd64/vmm/vmm.c
==============================================================================
--- head/sys/amd64/vmm/vmm.c	Sat Mar 16 22:44:14 2013	(r248391)
+++ head/sys/amd64/vmm/vmm.c	Sat Mar 16 22:53:05 2013	(r248392)
@@ -937,7 +937,7 @@ vm_set_x2apic_state(struct vm *vm, int v
 	if (vcpuid < 0 || vcpuid >= VM_MAXCPU)
 		return (EINVAL);
 
-	if (state < 0 || state >= X2APIC_STATE_LAST)
+	if (state >= X2APIC_STATE_LAST)
 		return (EINVAL);
 
 	vm->vcpu[vcpuid].x2apic_state = state;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201303162253.r2GMr5O0039599>