Re: [PATCH v2 2/7] KVM: X86: Use _BITUL() macro in UAPI headers

From
Paolo Bonzini <pbonzini@redhat.com>
Date
2021-05-24 12:28:54
Also in
kvm, linux-media, lkml
Reply to
Joe Richey, 2021-05-21
On 21/05/21 10:58, Joe Richey wrote:
quoted hunk ↗ jump to hunk
From: Joe Richey <redacted>

Replace BIT() in KVM's UPAI header with _BITUL(). BIT() is not defined
in the UAPI headers and its usage may cause userspace build errors.

Fixes: fb04a1eddb1a ("KVM: X86: Implement ring-based dirty memory tracking")
Signed-off-by: Joe Richey <redacted>
---
  include/uapi/linux/kvm.h       | 5 +++--
  tools/include/uapi/linux/kvm.h | 5 +++--
  2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/include/uapi/linux/kvm.h b/include/uapi/linux/kvm.h
index 3fd9a7e9d90c..79d9c44d1ad7 100644
--- a/include/uapi/linux/kvm.h
+++ b/include/uapi/linux/kvm.h
@@ -8,6 +8,7 @@
   * Note: you must update KVM_API_VERSION if you change this interface.
   */
  
+#include <linux/const.h>
  #include <linux/types.h>
  #include <linux/compiler.h>
  #include <linux/ioctl.h>
@@ -1879,8 +1880,8 @@ struct kvm_hyperv_eventfd {
   * conversion after harvesting an entry.  Also, it must not skip any
   * dirty bits, so that dirty bits are always harvested in sequence.
   */
-#define KVM_DIRTY_GFN_F_DIRTY           BIT(0)
-#define KVM_DIRTY_GFN_F_RESET           BIT(1)
+#define KVM_DIRTY_GFN_F_DIRTY           _BITUL(0)
+#define KVM_DIRTY_GFN_F_RESET           _BITUL(1)
  #define KVM_DIRTY_GFN_F_MASK            0x3
  
  /*
diff --git a/tools/include/uapi/linux/kvm.h b/tools/include/uapi/linux/kvm.h
index 3fd9a7e9d90c..79d9c44d1ad7 100644
--- a/tools/include/uapi/linux/kvm.h
+++ b/tools/include/uapi/linux/kvm.h
@@ -8,6 +8,7 @@
   * Note: you must update KVM_API_VERSION if you change this interface.
   */
  
+#include <linux/const.h>
  #include <linux/types.h>
  #include <linux/compiler.h>
  #include <linux/ioctl.h>
@@ -1879,8 +1880,8 @@ struct kvm_hyperv_eventfd {
   * conversion after harvesting an entry.  Also, it must not skip any
   * dirty bits, so that dirty bits are always harvested in sequence.
   */
-#define KVM_DIRTY_GFN_F_DIRTY           BIT(0)
-#define KVM_DIRTY_GFN_F_RESET           BIT(1)
+#define KVM_DIRTY_GFN_F_DIRTY           _BITUL(0)
+#define KVM_DIRTY_GFN_F_RESET           _BITUL(1)
  #define KVM_DIRTY_GFN_F_MASK            0x3
  
  /*
Queued thi sone, thanks.

Paolo


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Thread (31 messages) 31 messages, 9 authors, last reply 2021-05-24
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help