From: Neil Brown <neilb@suse.de>
To: Jeff Garzik <jeff@garzik.org>
Date: Wed, 11 Oct 2006 09:53:11 +1000
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <17708.12903.712376.255113@cse.unsw.edu.au>
Cc: Andrew Morton <akpm@osdl.org>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] MD: conditionalize some code
In-Reply-To: message from Jeff Garzik on Tuesday October 10
References: <20061010231631.GA18222@havoc.gtf.org>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1228
Lines: 51

On Tuesday October 10, jeff@garzik.org wrote:
> 
> The autorun code is only used if this module is built into the static
> kernel image.  Adjust #ifdefs accordingly.
> 
> Signed-off-by: Jeff Garzik <jeff@garzik.org>

Acked-by: NeilBrown <neilb@suse.de>

Thanks,
NeilBrown

> 
> ---
> 
>  drivers/md/md.c               |    4 +++-
> 
> diff --git a/drivers/md/md.c b/drivers/md/md.c
> index 57fa64f..c75cdf9 100644
> --- a/drivers/md/md.c
> +++ b/drivers/md/md.c
> @@ -3368,6 +3368,7 @@ out:
>  	return err;
>  }
>  
> +#ifndef MODULE
>  static void autorun_array(mddev_t *mddev)
>  {
>  	mdk_rdev_t *rdev;
> @@ -3482,6 +3483,7 @@ static void autorun_devices(int part)
>  	}
>  	printk(KERN_INFO "md: ... autorun DONE.\n");
>  }
> +#endif /* !MODULE */
>  
>  static int get_version(void __user * arg)
>  {
> @@ -5592,7 +5594,7 @@ static void autostart_arrays(int part)
>  	autorun_devices(part);
>  }
>  
> -#endif
> +#endif /* !MODULE */
>  
>  static __exit void md_exit(void)
>  {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/