On Tue, Jan 12, 2010 at 05:00:31AM -0500, Christoph Hellwig wrote:
> On Tue, Jan 12, 2010 at 03:41:07PM +1100, Dave Chinner wrote:
> > Loop devices are now created and destroyed on demand, and
> > the existing hacks to do this no longer work. Remove them.
>
> Just noticed that I had not been running this test at all for a long
> time as it doesn't work with external logs.
>
> Seems like the current code indeed fails reliably for me and this
> patch fixes it.
>
> The only problem is that the loop device autodestruct features is only
> available since Linux 2.6.25 and also needs a recent util-linux. Is
> this old enough to simply go with the auto destruct version, or do
> we need to support either one?
>
> What about the patch below? This adds the -d argument to the umount
> calls for the loop images, which destroys the loop images in older
> kernels and is a no-op for modern kernels that have the loop
> auto-destruct feature:
Yes, that looks like a better way to solve the problem.
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
|