Re: [PATCH] repair: warn if running in low memory mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 04, 2011 at 05:12:43PM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> When checking large filesystems, xfs_repair makes an estimate of how
> much RAM it will need to execute effectively. If the amount of RAM
> is less than this, it reduces the bhash size and turns of
> prefetching, which will substantially slow down the repair process.
> Add a warning that indicates this is happening, along with a
> recommendation of how much RAM repair calculates it needs to run
> with prefetching enabled.
> 
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

> +				do_warn(_("Not enough RAM available for repair "
> +					"to enable prefetching. This will be "
> +					"_slow_.\n"
> +					"You need at least %luMB RAM to run "
> +					"with prefetching enabled."),
> +					mem_used * 1280 / (1024 * 1024));

Maybe but the strings on a separate line to be easier readable and
greppable, e.g.:

				do_warn(
	_("Not enough RAM available for repair to enable prefetching. "
	  "This will be _slow_.\n"
	  "You need at least %luMB RAM to run with prefetching enabled."),

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux