qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [Bug 1382477] Re: hw/i386/intel_iommu.c:902: wrong logical


From: T. Huth
Subject: [Qemu-devel] [Bug 1382477] Re: hw/i386/intel_iommu.c:902: wrong logical operator ?
Date: Thu, 23 Jun 2016 08:02:42 -0000

Fix has been included here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=1e06f131fd9a44dd4af

** Changed in: qemu
       Status: New => Fix Released

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1382477

Title:
  hw/i386/intel_iommu.c:902: wrong logical operator ?

Status in QEMU:
  Fix Released

Bug description:
  /home/dcb/qemu/trunk/qemu/hw/i386/intel_iommu.c:902:5: error: logical ‘and’ 
applied to non-boolean constant [-Werror=logical-op]
       pvtd_as = s->address_spaces[VTD_SID_TO_BUS(source_id)];
       ^

  $ fgrep VTD_SID_TO_BUS `find . -name \*.h -print`
  ./include/hw/i386/intel_iommu.h:#define VTD_SID_TO_BUS(sid)         (((sid) 
>> 8) && 0xff)
  $ 

  Sounds to me like

  #define VTD_SID_TO_BUS(sid)         (((sid) >> 8) & 0xff)

  would be better.

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1382477/+subscriptions



reply via email to

[Prev in Thread] Current Thread [Next in Thread]