futex: add missing header guards

[Qemu-devel] [PATCH] futex: add missing header guards

Emilio G. Cota posted 1 patch 6 years, 6 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1507930220-17461-1-git-send-email-cota@braap.org

Test checkpatch passed

Test docker passed

Test s390x passed

include/qemu/futex.h | 5 +++++
1 file changed, 5 insertions(+)

Expand all Fold all

[Qemu-devel] [PATCH] futex: add missing header guards

Posted by Emilio G. Cota 6 years, 6 months ago

The header file was introduced by fbcc3e5 ("qemu-thread: optimize QemuLockCnt
with futexes on Linux", 2017-01-16) without header guards. Add them.

Signed-off-by: Emilio G. Cota <cota@braap.org>
---
 include/qemu/futex.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/include/qemu/futex.h b/include/qemu/futex.h
index bb7dc9e..91ae889 100644
--- a/include/qemu/futex.h
+++ b/include/qemu/futex.h
@@ -11,6 +11,9 @@
  *
  */
 
+#ifndef QEMU_FUTEX_H
+#define QEMU_FUTEX_H
+
 #include <sys/syscall.h>
 #include <linux/futex.h>
 
@@ -34,3 +37,5 @@ static inline void qemu_futex_wait(void *f, unsigned val)
         }
     }
 }
+
+#endif /* QEMU_FUTEX_H */
-- 
2.7.4

Re: [Qemu-devel] [PATCH] futex: add missing header guards

Posted by Philippe Mathieu-Daudé 6 years, 6 months ago

Cc: trivial

On 10/13/2017 06:30 PM, Emilio G. Cota wrote:
> The header file was introduced by fbcc3e5 ("qemu-thread: optimize QemuLockCnt
> with futexes on Linux", 2017-01-16) without header guards. Add them.
> 
> Signed-off-by: Emilio G. Cota <cota@braap.org>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> ---
>  include/qemu/futex.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/include/qemu/futex.h b/include/qemu/futex.h
> index bb7dc9e..91ae889 100644
> --- a/include/qemu/futex.h
> +++ b/include/qemu/futex.h
> @@ -11,6 +11,9 @@
>   *
>   */
>  
> +#ifndef QEMU_FUTEX_H
> +#define QEMU_FUTEX_H
> +
>  #include <sys/syscall.h>
>  #include <linux/futex.h>
>  
> @@ -34,3 +37,5 @@ static inline void qemu_futex_wait(void *f, unsigned val)
>          }
>      }
>  }
> +
> +#endif /* QEMU_FUTEX_H */
>