[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openvz-criu
Subject:    [CRIU] [PATCH v3 12/15] criu/files: Fix wait_fds_event() for 64 bit platforms
From:       Michael Holzheu <holzheu () linux ! vnet ! ibm ! com>
Date:       2017-06-30 18:31:47
Message-ID: 20170630183150.76021-13-holzheu () linux ! vnet ! ibm ! com
[Download RAW message or body]

The htole64() function returns a 64 bit value. This is stored in
a 32 bit integer in wait_fds_event() which results in 0 for big-endian
64 bit platforms. The fuxtex() system call also requires 32 bit values.

Fix this and use htole32() also for 64 bit platforms.

Signed-off-by: Michael Holzheu <holzheu@linux.vnet.ibm.com>
---
 criu/files.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/criu/files.c b/criu/files.c
index a36eb36..bad6069 100644
--- a/criu/files.c
+++ b/criu/files.c
@@ -193,11 +193,8 @@ void wait_fds_event(void)
 {
 	futex_t *f = &current->task_st;
 	int value;
-#if BITS_PER_LONG == 64
-	value = htole64(FDS_EVENT);
-#else
+
 	value = htole32(FDS_EVENT);
-#endif
 	futex_wait_if_cond(f, value, &);
 	clear_fds_event();
 }
-- 
2.7.4

_______________________________________________
CRIU mailing list
CRIU@openvz.org
https://lists.openvz.org/mailman/listinfo/criu
[prev in list] [next in list] [prev in thread] [next in thread]