Date: Wed, 9 Jun 2004 15:06:58 -0700
From: Andrew Morton <akpm@osdl.org>
To: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
Cc: axboe@suse.de, edt@aei.ca, linux-kernel@vger.kernel.org
Subject: Re: ide errors in 7-rc1-mm1 and later
Message-Id: <20040609150658.5e5e6653.akpm@osdl.org>
In-Reply-To: <200406092352.18470.bzolnier@elka.pw.edu.pl>
References: <1085689455.7831.8.camel@localhost>
	<200406041814.35003.bzolnier@elka.pw.edu.pl>
	<20040605091853.GA13641@suse.de>
	<200406092352.18470.bzolnier@elka.pw.edu.pl>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1416
Lines: 35

Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote:
>
> Sure, you don't need my ACK, that's obvious - you need it from Linus/Andrew.

But your nack would almost certainly prevent a merge, pending resolution of
whatever the issues are.

> 
> ...
>
> BTW are you aware of two (minor?) corner cases of the current implementation?
> 
> - you can't have journal on a separate device
>   (pre and post flushes will only flush device storing journal not data)

External journals in ext3 aren't really supported - they just happen to
work as a plaything.  I haven't tested it in several years, but I believe
people do use it.

That being said, the bug you identify is an ext3 bug.  The easiest way for
me to fix it up within ext3 would be to issue some flush command to the
filesystem's disk, wait for that to complete, then write the buffer_ordered
commit block to the journal's disk.  That's blkdev_issue_flush(), yes?

> - if you more than > 1 filesystem on the disk (quite likely scenario) it
>   can happen that barrier (flush) will fail for sector for file from the
>   other fs and later barrier for this other fs will succeed

I don't understand this one.

-
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/