On 08/22/16 11:32, Mikko Rapeli wrote: > - * uint32_t (*get_region_size)(struct dm_dirty_log *log); > + * __u32 (*get_region_size)(struct dm_dirty_log *log); uint32_t is a type that is defined by ANSI C but __u32 not. So this change looks wrong to me. Would it have been sufficient to add "#include <linux/types.h>" and keep the uint32_t etc. type names? Bart. -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel