Re: [PATCH] raid5: fix build error, sector_t usage

[NeilBrown <neilb@suse.de>]

On Thu, 21 Apr 2011 09:07:26 -0700 Randy Dunlap <randy.dunlap@oracle.com>
wrote:

> From: Randy Dunlap <randy.dunlap@oracle.com>
> 
> Change <sectors> from unsigned long long to sector_t.
> This matches its source field.
> 
> ERROR: "__udivdi3" [drivers/md/raid456.ko] undefined!
> 
> Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
> ---
>  drivers/md/raid5.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- linux-next-20110421.orig/drivers/md/raid5.c
> +++ linux-next-20110421/drivers/md/raid5.c
> @@ -5678,7 +5678,7 @@ static void raid5_quiesce(mddev_t *mddev
>  static void *raid45_takeover_raid0(mddev_t *mddev, int level)
>  {
>  	struct raid0_private_data *raid0_priv = mddev->private;
> -	unsigned long long sectors;
> +	sector_t sectors;
>  
>  	/* for raid0 takeover only one zone is supported */
>  	if (raid0_priv->nr_strip_zones > 1) {

Thanks Randy!

Linus:  You can either apply this patch directly or pull again from my
for-linus branch.

The following changes since commit 28a8397852011e323d16a1eecd4787d72b7b9a9e:

  md: Update documentation for sync_min and sync_max entries (2011-04-20 15:40:01 +1000)

are available in the git repository at:
  git://neil.brown.name/md/ for-linus

Randy Dunlap (1):
      raid5: fix build error, sector_t usage

 drivers/md/raid5.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


Thanks, and apologies,

NeilBrown