uapi dm-log-userspace.h: use __u32, __s32, __u64 and __s64 from linux/types.h
authorMikko Rapeli <mikko.rapeli@iki.fi>
Sun, 28 Aug 2016 06:42:37 +0000 (08:42 +0200)
committerMike Snitzer <snitzer@redhat.com>
Mon, 21 Nov 2016 14:52:02 +0000 (09:52 -0500)
Fixes userspace compilation errors like:

linux/dm-log-userspace.h:416:2: error: unknown type name ‘uint64_t’

Signed-off-by: Mikko Rapeli <mikko.rapeli@iki.fi>
Reviewed-by: Bart Van Assche <Bart.VanAssche@sandisk.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
include/uapi/linux/dm-log-userspace.h

index 0fa0d9ef06a59b4e69cd63574519a58630c31774..05e91e14c501a3e3328b202270fc724993e9fb08 100644 (file)
@@ -7,6 +7,7 @@
 #ifndef __DM_LOG_USERSPACE_H__
 #define __DM_LOG_USERSPACE_H__
 
+#include <linux/types.h>
 #include <linux/dm-ioctl.h> /* For DM_UUID_LEN */
 
 /*
 
 /*
  * DM_ULOG_GET_REGION_SIZE corresponds to (found in dm-dirty-log.h):
- * uint32_t (*get_region_size)(struct dm_dirty_log *log);
+ * __u32 (*get_region_size)(struct dm_dirty_log *log);
  *
  * Payload-to-userspace:
  *     None.
  * Payload-to-kernel:
- *     uint64_t - contains the region size
+ *     __u64 - contains the region size
  *
  * The region size is something that was determined at constructor time.
  * It is returned in the payload area and 'data_size' is set to
  * int (*is_clean)(struct dm_dirty_log *log, region_t region);
  *
  * Payload-to-userspace:
- *     uint64_t - the region to get clean status on
+ *     __u64 - the region to get clean status on
  * Payload-to-kernel:
- *     int64_t  - 1 if clean, 0 otherwise
+ *     __s64  - 1 if clean, 0 otherwise
  *
- * Payload is sizeof(uint64_t) and contains the region for which the clean
+ * Payload is sizeof(__u64) and contains the region for which the clean
  * status is being made.
  *
  * When the request has been processed, user-space must return the
  *               int can_block);
  *
  * Payload-to-userspace:
- *     uint64_t - the region to get sync status on
+ *     __u64 - the region to get sync status on
  * Payload-to-kernel:
- *     int64_t - 1 if in-sync, 0 otherwise
+ *     __s64 - 1 if in-sync, 0 otherwise
  *
  * Exactly the same as 'is_clean' above, except this time asking "has the
  * region been recovered?" vs. "is the region not being modified?"
  * Payload-to-userspace:
  *     If the 'integrated_flush' directive is present in the constructor
  *     table, the payload is as same as DM_ULOG_MARK_REGION:
- *             uint64_t [] - region(s) to mark
+ *             __u64 [] - region(s) to mark
  *     else
  *             None
  * Payload-to-kernel:
  * void (*mark_region)(struct dm_dirty_log *log, region_t region);
  *
  * Payload-to-userspace:
- *     uint64_t [] - region(s) to mark
+ *     __u64 [] - region(s) to mark
  * Payload-to-kernel:
  *     None.
  *
  * Incoming payload contains the one or more regions to mark dirty.
  * The number of regions contained in the payload can be determined from
- * 'data_size/sizeof(uint64_t)'.
+ * 'data_size/sizeof(__u64)'.
  *
  * When the request has been processed, user-space must return the
  * dm_ulog_request to the kernel - setting the 'error' field and clearing
  * void (*clear_region)(struct dm_dirty_log *log, region_t region);
  *
  * Payload-to-userspace:
- *     uint64_t [] - region(s) to clear
+ *     __u64 [] - region(s) to clear
  * Payload-to-kernel:
  *     None.
  *
  * Incoming payload contains the one or more regions to mark clean.
  * The number of regions contained in the payload can be determined from
- * 'data_size/sizeof(uint64_t)'.
+ * 'data_size/sizeof(__u64)'.
  *
  * When the request has been processed, user-space must return the
  * dm_ulog_request to the kernel - setting the 'error' field and clearing
  *     None.
  * Payload-to-kernel:
  *     {
- *             int64_t i; -- 1 if recovery necessary, 0 otherwise
- *             uint64_t r; -- The region to recover if i=1
+ *             __s64 i; -- 1 if recovery necessary, 0 otherwise
+ *             __u64 r; -- The region to recover if i=1
  *     }
  * 'data_size' should be set appropriately.
  *
  *
  * Payload-to-userspace:
  *     {
- *             uint64_t - region to set sync state on
- *             int64_t  - 0 if not-in-sync, 1 if in-sync
+ *             __u64 - region to set sync state on
+ *             __s64  - 0 if not-in-sync, 1 if in-sync
  *     }
  * Payload-to-kernel:
  *     None.
  * Payload-to-userspace:
  *     None.
  * Payload-to-kernel:
- *     uint64_t - the number of in-sync regions
+ *     __u64 - the number of in-sync regions
  *
  * No incoming payload.  Kernel-bound payload contains the number of
  * regions that are in-sync (in a size_t).
  * int (*is_remote_recovering)(struct dm_dirty_log *log, region_t region);
  *
  * Payload-to-userspace:
- *     uint64_t - region to determine recovery status on
+ *     __u64 - region to determine recovery status on
  * Payload-to-kernel:
  *     {
- *             int64_t is_recovering;  -- 0 if no, 1 if yes
- *             uint64_t in_sync_hint;  -- lowest region still needing resync
+ *             __s64 is_recovering;  -- 0 if no, 1 if yes
+ *             __u64 in_sync_hint;  -- lowest region still needing resync
  *     }
  *
  * When the request has been processed, user-space must return the
@@ -413,16 +414,16 @@ struct dm_ulog_request {
         * differentiate between logs that are being swapped and have the
         * same 'uuid'.  (Think "live" and "inactive" device-mapper tables.)
         */
-       uint64_t luid;
+       __u64 luid;
        char uuid[DM_UUID_LEN];
        char padding[3];        /* Padding because DM_UUID_LEN = 129 */
 
-       uint32_t version;       /* See DM_ULOG_REQUEST_VERSION */
-       int32_t error;          /* Used to report back processing errors */
+       __u32 version;       /* See DM_ULOG_REQUEST_VERSION */
+       __s32 error;          /* Used to report back processing errors */
 
-       uint32_t seq;           /* Sequence number for request */
-       uint32_t request_type;  /* DM_ULOG_* defined above */
-       uint32_t data_size;     /* How much data (not including this struct) */
+       __u32 seq;           /* Sequence number for request */
+       __u32 request_type;  /* DM_ULOG_* defined above */
+       __u32 data_size;     /* How much data (not including this struct) */
 
        char data[0];
 };