Commit 209e2367 authored by Mikko Rapeli's avatar Mikko Rapeli Committed by Mike Snitzer

uapi dm-log-userspace.h: use __u32, __s32, __u64 and __s64 from linux/types.h

Fixes userspace compilation errors like:

linux/dm-log-userspace.h:416:2: error: unknown type name ‘uint64_t’
Signed-off-by: default avatarMikko Rapeli <mikko.rapeli@iki.fi>
Reviewed-by: default avatarBart Van Assche <Bart.VanAssche@sandisk.com>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent 21ffe552
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
#ifndef __DM_LOG_USERSPACE_H__ #ifndef __DM_LOG_USERSPACE_H__
#define __DM_LOG_USERSPACE_H__ #define __DM_LOG_USERSPACE_H__
#include <linux/types.h>
#include <linux/dm-ioctl.h> /* For DM_UUID_LEN */ #include <linux/dm-ioctl.h> /* For DM_UUID_LEN */
/* /*
...@@ -147,12 +148,12 @@ ...@@ -147,12 +148,12 @@
/* /*
* DM_ULOG_GET_REGION_SIZE corresponds to (found in dm-dirty-log.h): * 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: * Payload-to-userspace:
* None. * None.
* Payload-to-kernel: * 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. * The region size is something that was determined at constructor time.
* It is returned in the payload area and 'data_size' is set to * It is returned in the payload area and 'data_size' is set to
...@@ -168,11 +169,11 @@ ...@@ -168,11 +169,11 @@
* int (*is_clean)(struct dm_dirty_log *log, region_t region); * int (*is_clean)(struct dm_dirty_log *log, region_t region);
* *
* Payload-to-userspace: * Payload-to-userspace:
* uint64_t - the region to get clean status on * __u64 - the region to get clean status on
* Payload-to-kernel: * 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. * status is being made.
* *
* When the request has been processed, user-space must return the * When the request has been processed, user-space must return the
...@@ -187,9 +188,9 @@ ...@@ -187,9 +188,9 @@
* int can_block); * int can_block);
* *
* Payload-to-userspace: * Payload-to-userspace:
* uint64_t - the region to get sync status on * __u64 - the region to get sync status on
* Payload-to-kernel: * 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 * Exactly the same as 'is_clean' above, except this time asking "has the
* region been recovered?" vs. "is the region not being modified?" * region been recovered?" vs. "is the region not being modified?"
...@@ -203,7 +204,7 @@ ...@@ -203,7 +204,7 @@
* Payload-to-userspace: * Payload-to-userspace:
* If the 'integrated_flush' directive is present in the constructor * If the 'integrated_flush' directive is present in the constructor
* table, the payload is as same as DM_ULOG_MARK_REGION: * table, the payload is as same as DM_ULOG_MARK_REGION:
* uint64_t [] - region(s) to mark * __u64 [] - region(s) to mark
* else * else
* None * None
* Payload-to-kernel: * Payload-to-kernel:
...@@ -225,13 +226,13 @@ ...@@ -225,13 +226,13 @@
* void (*mark_region)(struct dm_dirty_log *log, region_t region); * void (*mark_region)(struct dm_dirty_log *log, region_t region);
* *
* Payload-to-userspace: * Payload-to-userspace:
* uint64_t [] - region(s) to mark * __u64 [] - region(s) to mark
* Payload-to-kernel: * Payload-to-kernel:
* None. * None.
* *
* Incoming payload contains the one or more regions to mark dirty. * Incoming payload contains the one or more regions to mark dirty.
* The number of regions contained in the payload can be determined from * 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 * When the request has been processed, user-space must return the
* dm_ulog_request to the kernel - setting the 'error' field and clearing * dm_ulog_request to the kernel - setting the 'error' field and clearing
...@@ -244,13 +245,13 @@ ...@@ -244,13 +245,13 @@
* void (*clear_region)(struct dm_dirty_log *log, region_t region); * void (*clear_region)(struct dm_dirty_log *log, region_t region);
* *
* Payload-to-userspace: * Payload-to-userspace:
* uint64_t [] - region(s) to clear * __u64 [] - region(s) to clear
* Payload-to-kernel: * Payload-to-kernel:
* None. * None.
* *
* Incoming payload contains the one or more regions to mark clean. * Incoming payload contains the one or more regions to mark clean.
* The number of regions contained in the payload can be determined from * 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 * When the request has been processed, user-space must return the
* dm_ulog_request to the kernel - setting the 'error' field and clearing * dm_ulog_request to the kernel - setting the 'error' field and clearing
...@@ -266,8 +267,8 @@ ...@@ -266,8 +267,8 @@
* None. * None.
* Payload-to-kernel: * Payload-to-kernel:
* { * {
* int64_t i; -- 1 if recovery necessary, 0 otherwise * __s64 i; -- 1 if recovery necessary, 0 otherwise
* uint64_t r; -- The region to recover if i=1 * __u64 r; -- The region to recover if i=1
* } * }
* 'data_size' should be set appropriately. * 'data_size' should be set appropriately.
* *
...@@ -283,8 +284,8 @@ ...@@ -283,8 +284,8 @@
* *
* Payload-to-userspace: * Payload-to-userspace:
* { * {
* uint64_t - region to set sync state on * __u64 - region to set sync state on
* int64_t - 0 if not-in-sync, 1 if in-sync * __s64 - 0 if not-in-sync, 1 if in-sync
* } * }
* Payload-to-kernel: * Payload-to-kernel:
* None. * None.
...@@ -302,7 +303,7 @@ ...@@ -302,7 +303,7 @@
* Payload-to-userspace: * Payload-to-userspace:
* None. * None.
* Payload-to-kernel: * 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 * No incoming payload. Kernel-bound payload contains the number of
* regions that are in-sync (in a size_t). * regions that are in-sync (in a size_t).
...@@ -350,11 +351,11 @@ ...@@ -350,11 +351,11 @@
* int (*is_remote_recovering)(struct dm_dirty_log *log, region_t region); * int (*is_remote_recovering)(struct dm_dirty_log *log, region_t region);
* *
* Payload-to-userspace: * Payload-to-userspace:
* uint64_t - region to determine recovery status on * __u64 - region to determine recovery status on
* Payload-to-kernel: * Payload-to-kernel:
* { * {
* int64_t is_recovering; -- 0 if no, 1 if yes * __s64 is_recovering; -- 0 if no, 1 if yes
* uint64_t in_sync_hint; -- lowest region still needing resync * __u64 in_sync_hint; -- lowest region still needing resync
* } * }
* *
* When the request has been processed, user-space must return the * When the request has been processed, user-space must return the
...@@ -413,16 +414,16 @@ struct dm_ulog_request { ...@@ -413,16 +414,16 @@ struct dm_ulog_request {
* differentiate between logs that are being swapped and have the * differentiate between logs that are being swapped and have the
* same 'uuid'. (Think "live" and "inactive" device-mapper tables.) * same 'uuid'. (Think "live" and "inactive" device-mapper tables.)
*/ */
uint64_t luid; __u64 luid;
char uuid[DM_UUID_LEN]; char uuid[DM_UUID_LEN];
char padding[3]; /* Padding because DM_UUID_LEN = 129 */ char padding[3]; /* Padding because DM_UUID_LEN = 129 */
uint32_t version; /* See DM_ULOG_REQUEST_VERSION */ __u32 version; /* See DM_ULOG_REQUEST_VERSION */
int32_t error; /* Used to report back processing errors */ __s32 error; /* Used to report back processing errors */
uint32_t seq; /* Sequence number for request */ __u32 seq; /* Sequence number for request */
uint32_t request_type; /* DM_ULOG_* defined above */ __u32 request_type; /* DM_ULOG_* defined above */
uint32_t data_size; /* How much data (not including this struct) */ __u32 data_size; /* How much data (not including this struct) */
char data[0]; char data[0];
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment