lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <e4f29cb9-fbc1-47db-b4fa-769d72d42642@molgen.mpg.de>
Date: Mon, 26 Aug 2024 10:12:29 +0200
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: Yu Kuai <yukuai1@...weicloud.com>
Cc: mariusz.tkaczyk@...ux.intel.com, xni@...hat.com, song@...nel.org,
 linux-kernel@...r.kernel.org, linux-raid@...r.kernel.org,
 yukuai3@...wei.com, yi.zhang@...wei.com, yangerkun@...wei.com
Subject: Re: [PATCH md-6.12 v2 42/42] md/md-bitmap: make in memory structure
 internal

Dear Kuai,


Am 26.08.24 um 09:44 schrieb Yu Kuai:
> From: Yu Kuai <yukuai3@...wei.com>
> 
> Now that struct btimap_page and bitmap is not used external anymore,

s/btimap/bitmap/

externally(?)

> move them from md-bitmap.h to md-bitmap.c.(expect that dm-raid is still

Please add space before (.

> using define marco 'COUNTER_MAX').

(Except that dm-raid is still using the macro `COUNTER_MAX`.)

> 
> Also fix some checkpatch warnings.

Which ones? Maybe make it separate patches.


Kind regards,

Paul


> Signed-off-by: Yu Kuai <yukuai3@...wei.com>
> ---
>   drivers/md/md-bitmap.c  | 247 ++++++++++++++++++++++++++++++++++++----
>   drivers/md/md-bitmap.h  | 189 +-----------------------------
>   drivers/md/md-cluster.c |   4 +-
>   drivers/md/md.h         |   2 +-
>   drivers/md/raid1.c      |   5 +-
>   5 files changed, 235 insertions(+), 212 deletions(-)
> 
> diff --git a/drivers/md/md-bitmap.c b/drivers/md/md-bitmap.c
> index 098bffac2167..864418c8c028 100644
> --- a/drivers/md/md-bitmap.c
> +++ b/drivers/md/md-bitmap.c
> @@ -32,6 +32,186 @@
>   #include "md.h"
>   #include "md-bitmap.h"
>   
> +#define BITMAP_MAJOR_LO 3
> +/* version 4 insists the bitmap is in little-endian order
> + * with version 3, it is host-endian which is non-portable
> + * Version 5 is currently set only for clustered devices
> + */
> +#define BITMAP_MAJOR_HI 4
> +#define BITMAP_MAJOR_CLUSTERED 5
> +#define	BITMAP_MAJOR_HOSTENDIAN 3
> +
> +/*
> + * in-memory bitmap:
> + *
> + * Use 16 bit block counters to track pending writes to each "chunk".
> + * The 2 high order bits are special-purpose, the first is a flag indicating
> + * whether a resync is needed.  The second is a flag indicating whether a
> + * resync is active.
> + * This means that the counter is actually 14 bits:
> + *
> + * +--------+--------+------------------------------------------------+
> + * | resync | resync |               counter                          |
> + * | needed | active |                                                |
> + * |  (0-1) |  (0-1) |              (0-16383)                         |
> + * +--------+--------+------------------------------------------------+
> + *
> + * The "resync needed" bit is set when:
> + *    a '1' bit is read from storage at startup.
> + *    a write request fails on some drives
> + *    a resync is aborted on a chunk with 'resync active' set
> + * It is cleared (and resync-active set) when a resync starts across all drives
> + * of the chunk.
> + *
> + *
> + * The "resync active" bit is set when:
> + *    a resync is started on all drives, and resync_needed is set.
> + *       resync_needed will be cleared (as long as resync_active wasn't already set).
> + * It is cleared when a resync completes.
> + *
> + * The counter counts pending write requests, plus the on-disk bit.
> + * When the counter is '1' and the resync bits are clear, the on-disk
> + * bit can be cleared as well, thus setting the counter to 0.
> + * When we set a bit, or in the counter (to start a write), if the fields is
> + * 0, we first set the disk bit and set the counter to 1.
> + *
> + * If the counter is 0, the on-disk bit is clear and the stripe is clean
> + * Anything that dirties the stripe pushes the counter to 2 (at least)
> + * and sets the on-disk bit (lazily).
> + * If a periodic sweep find the counter at 2, it is decremented to 1.
> + * If the sweep find the counter at 1, the on-disk bit is cleared and the
> + * counter goes to zero.
> + *
> + * Also, we'll hijack the "map" pointer itself and use it as two 16 bit block
> + * counters as a fallback when "page" memory cannot be allocated:
> + *
> + * Normal case (page memory allocated):
> + *
> + *     page pointer (32-bit)
> + *
> + *     [ ] ------+
> + *               |
> + *               +-------> [   ][   ]..[   ] (4096 byte page == 2048 counters)
> + *                          c1   c2    c2048
> + *
> + * Hijacked case (page memory allocation failed):
> + *
> + *     hijacked page pointer (32-bit)
> + *
> + *     [		  ][		  ] (no page memory allocated)
> + *      counter #1 (16-bit) counter #2 (16-bit)
> + *
> + */
> +
> +#define PAGE_BITS (PAGE_SIZE << 3)
> +#define PAGE_BIT_SHIFT (PAGE_SHIFT + 3)
> +
> +#define NEEDED(x) (((bitmap_counter_t) x) & NEEDED_MASK)
> +#define RESYNC(x) (((bitmap_counter_t) x) & RESYNC_MASK)
> +#define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
> +
> +/* how many counters per page? */
> +#define PAGE_COUNTER_RATIO (PAGE_BITS / COUNTER_BITS)
> +/* same, except a shift value for more efficient bitops */
> +#define PAGE_COUNTER_SHIFT (PAGE_BIT_SHIFT - COUNTER_BIT_SHIFT)
> +/* same, except a mask value for more efficient bitops */
> +#define PAGE_COUNTER_MASK  (PAGE_COUNTER_RATIO - 1)
> +
> +#define BITMAP_BLOCK_SHIFT 9
> +
> +/*
> + * bitmap structures:
> + */
> +
> +/* the in-memory bitmap is represented by bitmap_pages */
> +struct bitmap_page {
> +	/*
> +	 * map points to the actual memory page
> +	 */
> +	char *map;
> +	/*
> +	 * in emergencies (when map cannot be alloced), hijack the map
> +	 * pointer and use it as two counters itself
> +	 */
> +	unsigned int hijacked:1;
> +	/*
> +	 * If any counter in this page is '1' or '2' - and so could be
> +	 * cleared then that page is marked as 'pending'
> +	 */
> +	unsigned int pending:1;
> +	/*
> +	 * count of dirty bits on the page
> +	 */
> +	unsigned int  count:30;
> +};
> +
> +/* the main bitmap structure - one per mddev */
> +struct bitmap {
> +
> +	struct bitmap_counts {
> +		spinlock_t lock;
> +		struct bitmap_page *bp;
> +		/* total number of pages in the bitmap */
> +		unsigned long pages;
> +		/* number of pages not yet allocated */
> +		unsigned long missing_pages;
> +		/* chunksize = 2^chunkshift (for bitops) */
> +		unsigned long chunkshift;
> +		/* total number of data chunks for the array */
> +		unsigned long chunks;
> +	} counts;
> +
> +	struct mddev *mddev; /* the md device that the bitmap is for */
> +
> +	__u64	events_cleared;
> +	int need_sync;
> +
> +	struct bitmap_storage {
> +		/* backing disk file */
> +		struct file *file;
> +		/* cached copy of the bitmap file superblock */
> +		struct page *sb_page;
> +		unsigned long sb_index;
> +		/* list of cache pages for the file */
> +		struct page **filemap;
> +		/* attributes associated filemap pages */
> +		unsigned long *filemap_attr;
> +		/* number of pages in the file */
> +		unsigned long file_pages;
> +		/* total bytes in the bitmap */
> +		unsigned long bytes;
> +	} storage;
> +
> +	unsigned long flags;
> +
> +	int allclean;
> +
> +	atomic_t behind_writes;
> +	/* highest actual value at runtime */
> +	unsigned long behind_writes_used;
> +
> +	/*
> +	 * the bitmap daemon - periodically wakes up and sweeps the bitmap
> +	 * file, cleaning up bits and flushing out pages to disk as necessary
> +	 */
> +	unsigned long daemon_lastrun; /* jiffies of last run */
> +	/*
> +	 * when we lasted called end_sync to update bitmap with resync
> +	 * progress.
> +	 */
> +	unsigned long last_end_sync;
> +
> +	/* pending writes to the bitmap file */
> +	atomic_t pending_writes;
> +	wait_queue_head_t write_wait;
> +	wait_queue_head_t overflow_wait;
> +	wait_queue_head_t behind_wait;
> +
> +	struct kernfs_node *sysfs_can_clear;
> +	/* slot offset for clustered env */
> +	int cluster_slot;
> +};
> +
>   static int __bitmap_resize(struct bitmap *bitmap, sector_t blocks,
>   			   int chunksize, bool init);
>   
> @@ -491,9 +671,10 @@ static void md_bitmap_wait_writes(struct bitmap *bitmap)
>   
>   
>   /* update the event counter and sync the superblock to disk */
> -static void bitmap_update_sb(struct bitmap *bitmap)
> +static void bitmap_update_sb(void *data)
>   {
>   	bitmap_super_t *sb;
> +	struct bitmap *bitmap = data;
>   
>   	if (!bitmap || !bitmap->mddev) /* no bitmap for this array */
>   		return;
> @@ -1844,10 +2025,11 @@ static void bitmap_flush(struct mddev *mddev)
>   	bitmap_update_sb(bitmap);
>   }
>   
> -static void md_bitmap_free(struct bitmap *bitmap)
> +static void md_bitmap_free(void *data)
>   {
>   	unsigned long k, pages;
>   	struct bitmap_page *bp;
> +	struct bitmap *bitmap = data;
>   
>   	if (!bitmap) /* there was no bitmap */
>   		return;
> @@ -2076,7 +2258,7 @@ static int bitmap_load(struct mddev *mddev)
>   }
>   
>   /* caller need to free returned bitmap with md_bitmap_free() */
> -static struct bitmap *bitmap_get_from_slot(struct mddev *mddev, int slot)
> +static void *bitmap_get_from_slot(struct mddev *mddev, int slot)
>   {
>   	int rv = 0;
>   	struct bitmap *bitmap;
> @@ -2143,15 +2325,18 @@ static int bitmap_copy_from_slot(struct mddev *mddev, int slot, sector_t *low,
>   	return rv;
>   }
>   
> -static void bitmap_set_pages(struct bitmap *bitmap, unsigned long pages)
> +static void bitmap_set_pages(void *data, unsigned long pages)
>   {
> +	struct bitmap *bitmap = data;
> +
>   	bitmap->counts.pages = pages;
>   }
>   
> -static int bitmap_get_stats(struct bitmap *bitmap, struct md_bitmap_stats *stats)
> +static int bitmap_get_stats(void *data, struct md_bitmap_stats *stats)
>   {
>   	struct bitmap_storage *storage;
>   	struct bitmap_counts *counts;
> +	struct bitmap *bitmap = data;
>   	bitmap_super_t *sb;
>   
>   	if (!bitmap)
> @@ -2510,6 +2695,7 @@ space_show(struct mddev *mddev, char *page)
>   static ssize_t
>   space_store(struct mddev *mddev, const char *buf, size_t len)
>   {
> +	struct bitmap *bitmap;
>   	unsigned long sectors;
>   	int rv;
>   
> @@ -2520,8 +2706,8 @@ space_store(struct mddev *mddev, const char *buf, size_t len)
>   	if (sectors == 0)
>   		return -EINVAL;
>   
> -	if (mddev->bitmap &&
> -	    sectors < (mddev->bitmap->storage.bytes + 511) >> 9)
> +	bitmap = mddev->bitmap;
> +	if (bitmap && sectors < (bitmap->storage.bytes + 511) >> 9)
>   		return -EFBIG; /* Bitmap is too big for this small space */
>   
>   	/* could make sure it isn't too big, but that isn't really
> @@ -2698,10 +2884,13 @@ __ATTR(metadata, S_IRUGO|S_IWUSR, metadata_show, metadata_store);
>   static ssize_t can_clear_show(struct mddev *mddev, char *page)
>   {
>   	int len;
> +	struct bitmap *bitmap;
> +
>   	spin_lock(&mddev->lock);
> -	if (mddev->bitmap)
> -		len = sprintf(page, "%s\n", (mddev->bitmap->need_sync ?
> -					     "false" : "true"));
> +	bitmap = mddev->bitmap;
> +	if (bitmap)
> +		len = sprintf(page, "%s\n", (bitmap->need_sync ? "false" :
> +								 "true"));
>   	else
>   		len = sprintf(page, "\n");
>   	spin_unlock(&mddev->lock);
> @@ -2710,17 +2899,24 @@ static ssize_t can_clear_show(struct mddev *mddev, char *page)
>   
>   static ssize_t can_clear_store(struct mddev *mddev, const char *buf, size_t len)
>   {
> -	if (mddev->bitmap == NULL)
> +	struct bitmap *bitmap = mddev->bitmap;
> +
> +	if (!bitmap)
>   		return -ENOENT;
> -	if (strncmp(buf, "false", 5) == 0)
> -		mddev->bitmap->need_sync = 1;
> -	else if (strncmp(buf, "true", 4) == 0) {
> +
> +	if (strncmp(buf, "false", 5) == 0) {
> +		bitmap->need_sync = 1;
> +		return len;
> +	}
> +
> +	if (strncmp(buf, "true", 4) == 0) {
>   		if (mddev->degraded)
>   			return -EBUSY;
> -		mddev->bitmap->need_sync = 0;
> -	} else
> -		return -EINVAL;
> -	return len;
> +		bitmap->need_sync = 0;
> +		return len;
> +	}
> +
> +	return -EINVAL;
>   }
>   
>   static struct md_sysfs_entry bitmap_can_clear =
> @@ -2730,21 +2926,26 @@ static ssize_t
>   behind_writes_used_show(struct mddev *mddev, char *page)
>   {
>   	ssize_t ret;
> +	struct bitmap *bitmap;
> +
>   	spin_lock(&mddev->lock);
> -	if (mddev->bitmap == NULL)
> +	bitmap = mddev->bitmap;
> +	if (!bitmap)
>   		ret = sprintf(page, "0\n");
>   	else
> -		ret = sprintf(page, "%lu\n",
> -			      mddev->bitmap->behind_writes_used);
> +		ret = sprintf(page, "%lu\n", bitmap->behind_writes_used);
>   	spin_unlock(&mddev->lock);
> +
>   	return ret;
>   }
>   
>   static ssize_t
>   behind_writes_used_reset(struct mddev *mddev, const char *buf, size_t len)
>   {
> -	if (mddev->bitmap)
> -		mddev->bitmap->behind_writes_used = 0;
> +	struct bitmap *bitmap = mddev->bitmap;
> +
> +	if (bitmap)
> +		bitmap->behind_writes_used = 0;
>   	return len;
>   }
>   
> diff --git a/drivers/md/md-bitmap.h b/drivers/md/md-bitmap.h
> index c720729687e2..662e6fc141a7 100644
> --- a/drivers/md/md-bitmap.h
> +++ b/drivers/md/md-bitmap.h
> @@ -7,81 +7,7 @@
>   #ifndef BITMAP_H
>   #define BITMAP_H 1
>   
> -#define BITMAP_MAJOR_LO 3
> -/* version 4 insists the bitmap is in little-endian order
> - * with version 3, it is host-endian which is non-portable
> - * Version 5 is currently set only for clustered devices
> - */
> -#define BITMAP_MAJOR_HI 4
> -#define BITMAP_MAJOR_CLUSTERED 5
> -#define	BITMAP_MAJOR_HOSTENDIAN 3
> -
> -/*
> - * in-memory bitmap:
> - *
> - * Use 16 bit block counters to track pending writes to each "chunk".
> - * The 2 high order bits are special-purpose, the first is a flag indicating
> - * whether a resync is needed.  The second is a flag indicating whether a
> - * resync is active.
> - * This means that the counter is actually 14 bits:
> - *
> - * +--------+--------+------------------------------------------------+
> - * | resync | resync |               counter                          |
> - * | needed | active |                                                |
> - * |  (0-1) |  (0-1) |              (0-16383)                         |
> - * +--------+--------+------------------------------------------------+
> - *
> - * The "resync needed" bit is set when:
> - *    a '1' bit is read from storage at startup.
> - *    a write request fails on some drives
> - *    a resync is aborted on a chunk with 'resync active' set
> - * It is cleared (and resync-active set) when a resync starts across all drives
> - * of the chunk.
> - *
> - *
> - * The "resync active" bit is set when:
> - *    a resync is started on all drives, and resync_needed is set.
> - *       resync_needed will be cleared (as long as resync_active wasn't already set).
> - * It is cleared when a resync completes.
> - *
> - * The counter counts pending write requests, plus the on-disk bit.
> - * When the counter is '1' and the resync bits are clear, the on-disk
> - * bit can be cleared as well, thus setting the counter to 0.
> - * When we set a bit, or in the counter (to start a write), if the fields is
> - * 0, we first set the disk bit and set the counter to 1.
> - *
> - * If the counter is 0, the on-disk bit is clear and the stripe is clean
> - * Anything that dirties the stripe pushes the counter to 2 (at least)
> - * and sets the on-disk bit (lazily).
> - * If a periodic sweep find the counter at 2, it is decremented to 1.
> - * If the sweep find the counter at 1, the on-disk bit is cleared and the
> - * counter goes to zero.
> - *
> - * Also, we'll hijack the "map" pointer itself and use it as two 16 bit block
> - * counters as a fallback when "page" memory cannot be allocated:
> - *
> - * Normal case (page memory allocated):
> - *
> - *     page pointer (32-bit)
> - *
> - *     [ ] ------+
> - *               |
> - *               +-------> [   ][   ]..[   ] (4096 byte page == 2048 counters)
> - *                          c1   c2    c2048
> - *
> - * Hijacked case (page memory allocation failed):
> - *
> - *     hijacked page pointer (32-bit)
> - *Yu
> - *     [		  ][		  ] (no page memory allocated)
> - *      counter #1 (16-bit) counter #2 (16-bit)
> - *
> - */
> -
> -#ifdef __KERNEL__
> -
> -#define PAGE_BITS (PAGE_SIZE << 3)
> -#define PAGE_BIT_SHIFT (PAGE_SHIFT + 3)
> +#define BITMAP_MAGIC 0x6d746962
>   
>   typedef __u16 bitmap_counter_t;
>   #define COUNTER_BITS 16
> @@ -91,26 +17,6 @@ typedef __u16 bitmap_counter_t;
>   #define NEEDED_MASK ((bitmap_counter_t) (1 << (COUNTER_BITS - 1)))
>   #define RESYNC_MASK ((bitmap_counter_t) (1 << (COUNTER_BITS - 2)))
>   #define COUNTER_MAX ((bitmap_counter_t) RESYNC_MASK - 1)
> -#define NEEDED(x) (((bitmap_counter_t) x) & NEEDED_MASK)
> -#define RESYNC(x) (((bitmap_counter_t) x) & RESYNC_MASK)
> -#define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
> -
> -/* how many counters per page? */
> -#define PAGE_COUNTER_RATIO (PAGE_BITS / COUNTER_BITS)
> -/* same, except a shift value for more efficient bitops */
> -#define PAGE_COUNTER_SHIFT (PAGE_BIT_SHIFT - COUNTER_BIT_SHIFT)
> -/* same, except a mask value for more efficient bitops */
> -#define PAGE_COUNTER_MASK  (PAGE_COUNTER_RATIO - 1)
> -
> -#define BITMAP_BLOCK_SHIFT 9
> -
> -#endif
> -
> -/*
> - * bitmap structures:
> - */
> -
> -#define BITMAP_MAGIC 0x6d746962
>   
>   /* use these for bitmap->flags and bitmap->sb->state bit-fields */
>   enum bitmap_state {
> @@ -152,88 +58,6 @@ typedef struct bitmap_super_s {
>    *    devices.  For raid10 it is the size of the array.
>    */
>   
> -#ifdef __KERNEL__
> -
> -/* the in-memory bitmap is represented by bitmap_pages */
> -struct bitmap_page {
> -	/*
> -	 * map points to the actual memory page
> -	 */
> -	char *map;
> -	/*
> -	 * in emergencies (when map cannot be alloced), hijack the map
> -	 * pointer and use it as two counters itself
> -	 */
> -	unsigned int hijacked:1;
> -	/*
> -	 * If any counter in this page is '1' or '2' - and so could be
> -	 * cleared then that page is marked as 'pending'
> -	 */
> -	unsigned int pending:1;
> -	/*
> -	 * count of dirty bits on the page
> -	 */
> -	unsigned int  count:30;
> -};
> -
> -/* the main bitmap structure - one per mddev */
> -struct bitmap {
> -
> -	struct bitmap_counts {
> -		spinlock_t lock;
> -		struct bitmap_page *bp;
> -		unsigned long pages;		/* total number of pages
> -						 * in the bitmap */
> -		unsigned long missing_pages;	/* number of pages
> -						 * not yet allocated */
> -		unsigned long chunkshift;	/* chunksize = 2^chunkshift
> -						 * (for bitops) */
> -		unsigned long chunks;		/* Total number of data
> -						 * chunks for the array */
> -	} counts;
> -
> -	struct mddev *mddev; /* the md device that the bitmap is for */
> -
> -	__u64	events_cleared;
> -	int need_sync;
> -
> -	struct bitmap_storage {
> -		struct file *file;		/* backing disk file */
> -		struct page *sb_page;		/* cached copy of the bitmap
> -						 * file superblock */
> -		unsigned long sb_index;
> -		struct page **filemap;		/* list of cache pages for
> -						 * the file */
> -		unsigned long *filemap_attr;	/* attributes associated
> -						 * w/ filemap pages */
> -		unsigned long file_pages;	/* number of pages in the file*/
> -		unsigned long bytes;		/* total bytes in the bitmap */
> -	} storage;
> -
> -	unsigned long flags;
> -
> -	int allclean;
> -
> -	atomic_t behind_writes;
> -	unsigned long behind_writes_used; /* highest actual value at runtime */
> -
> -	/*
> -	 * the bitmap daemon - periodically wakes up and sweeps the bitmap
> -	 * file, cleaning up bits and flushing out pages to disk as necessary
> -	 */
> -	unsigned long daemon_lastrun; /* jiffies of last run */
> -	unsigned long last_end_sync; /* when we lasted called end_sync to
> -				      * update bitmap with resync progress */
> -
> -	atomic_t pending_writes; /* pending writes to the bitmap file */
> -	wait_queue_head_t write_wait;
> -	wait_queue_head_t overflow_wait;
> -	wait_queue_head_t behind_wait;
> -
> -	struct kernfs_node *sysfs_can_clear;
> -	int cluster_slot;		/* Slot offset for clustered env */
> -};
> -
>   struct md_bitmap_stats {
>   	u64		events_cleared;
>   	int		behind_writes;
> @@ -272,21 +96,20 @@ struct bitmap_operations {
>   	void (*cond_end_sync)(struct mddev *mddev, sector_t sector, bool force);
>   	void (*close_sync)(struct mddev *mddev);
>   
> -	void (*update_sb)(struct bitmap *bitmap);
> -	int (*get_stats)(struct bitmap *bitmap, struct md_bitmap_stats *stats);
> +	void (*update_sb)(void *data);
> +	int (*get_stats)(void *data, struct md_bitmap_stats *stats);
>   
>   	void (*sync_with_cluster)(struct mddev *mddev,
>   				  sector_t old_lo, sector_t old_hi,
>   				  sector_t new_lo, sector_t new_hi);
> -	struct bitmap *(*get_from_slot)(struct mddev *mddev, int slot);
> +	void *(*get_from_slot)(struct mddev *mddev, int slot);
>   	int (*copy_from_slot)(struct mddev *mddev, int slot, sector_t *lo,
>   			      sector_t *hi, bool clear_bits);
> -	void (*set_pages)(struct bitmap *bitmap, unsigned long pages);
> -	void (*free)(struct bitmap *bitmap);
> +	void (*set_pages)(void *data, unsigned long pages);
> +	void (*free)(void *data);
>   };
>   
>   /* the bitmap API */
>   void mddev_set_bitmap_ops(struct mddev *mddev);
>   
>   #endif
> -#endif
> diff --git a/drivers/md/md-cluster.c b/drivers/md/md-cluster.c
> index 7647ce4f76fa..6595f89becdb 100644
> --- a/drivers/md/md-cluster.c
> +++ b/drivers/md/md-cluster.c
> @@ -1144,7 +1144,7 @@ static int update_bitmap_size(struct mddev *mddev, sector_t size)
>   
>   static int resize_bitmaps(struct mddev *mddev, sector_t newsize, sector_t oldsize)
>   {
> -	struct bitmap *bitmap = mddev->bitmap;
> +	void *bitmap = mddev->bitmap;
>   	struct md_bitmap_stats stats;
>   	unsigned long my_pages;
>   	int i, rv;
> @@ -1218,9 +1218,9 @@ static int cluster_check_sync_size(struct mddev *mddev)
>   {
>   	int current_slot = md_cluster_ops->slot_number(mddev);
>   	int node_num = mddev->bitmap_info.nodes;
> -	struct bitmap *bitmap = mddev->bitmap;
>   	struct dlm_lock_resource *bm_lockres;
>   	struct md_bitmap_stats stats;
> +	void *bitmap = mddev->bitmap;
>   	unsigned long sync_size = 0;
>   	unsigned long my_sync_size;
>   	char str[64];
> diff --git a/drivers/md/md.h b/drivers/md/md.h
> index e56193f71ab4..1c6a5f41adca 100644
> --- a/drivers/md/md.h
> +++ b/drivers/md/md.h
> @@ -535,7 +535,7 @@ struct mddev {
>   	struct percpu_ref		writes_pending;
>   	int				sync_checkers;	/* # of threads checking writes_pending */
>   
> -	struct bitmap			*bitmap; /* the bitmap for the device */
> +	void				*bitmap; /* the bitmap for the device */
>   	struct bitmap_operations	*bitmap_ops;
>   	struct {
>   		struct file		*file; /* the bitmap file */
> diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c
> index 52a707e39a4d..6093df281eb1 100644
> --- a/drivers/md/raid1.c
> +++ b/drivers/md/raid1.c
> @@ -1425,7 +1425,6 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
>   	struct r1conf *conf = mddev->private;
>   	struct r1bio *r1_bio;
>   	int i, disks;
> -	struct bitmap *bitmap = mddev->bitmap;
>   	unsigned long flags;
>   	struct md_rdev *blocked_rdev;
>   	int first_clone;
> @@ -1578,7 +1577,7 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
>   	 * at a time and thus needs a new bio that can fit the whole payload
>   	 * this bio in page sized chunks.
>   	 */
> -	if (write_behind && bitmap)
> +	if (write_behind && mddev->bitmap)
>   		max_sectors = min_t(int, max_sectors,
>   				    BIO_MAX_VECS * (PAGE_SIZE >> 9));
>   	if (max_sectors < bio_sectors(bio)) {
> @@ -1614,7 +1613,7 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
>   			 * Not if there are too many, or cannot
>   			 * allocate memory, or a reader on WriteMostly
>   			 * is waiting for behind writes to flush */
> -			err = mddev->bitmap_ops->get_stats(bitmap, &stats);
> +			err = mddev->bitmap_ops->get_stats(mddev->bitmap, &stats);
>   			if (!err && write_behind && !stats.behind_wait &&
>   			    stats.behind_writes < max_write_behind)
>   				alloc_behind_master_bio(r1_bio, bio);


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ