Re: [uml-devel] [PATCH 1/1] um: ubd: Fix data corruption

2010-10-05 Thread Chris Frey
On Tue, Oct 05, 2010 at 10:23:19AM +0200, Tejun Heo wrote: > H, can you please give a shot at the following one? Thank you. I applied this patch on top of stock 2.6.35.5 as usual (no other patches) and tested on my maverick image as before. I ran a fsck.ext3 on the filesystem image from the

Re: [uml-devel] [PATCH 1/1] um: ubd: Fix data corruption

2010-10-05 Thread richard -rw- weinberger
On Wed, Sep 29, 2010 at 7:21 AM, Jens Axboe wrote: > I think we need to find the real fix here, just disabling merging > is not a fix (it's just a nasty work-around for the real bug). > Jens, Do you have an idea which parts of the code are buggy? -- Cheers, //richard -

Re: [uml-devel] [PATCH 1/1] um: ubd: Fix data corruption

2010-10-05 Thread Tejun Heo
Hello, sorry about chiming in later. I was off last week. On 09/29/2010 08:34 AM, Chris Frey wrote: > On Wed, Sep 29, 2010 at 02:21:07PM +0900, Jens Axboe wrote: >> This seems to imply that the original commit pin pointed is not >> the only issue we have in that code atm. >> >> I think we need to

Re: [uml-devel] [PATCH 1/1] um: ubd: Fix data corruption

2010-10-05 Thread Chris Frey
On Mon, Oct 04, 2010 at 06:37:36PM +0200, Tejun Heo wrote: > Hello, sorry about chiming in later. I was off last week. No problem, I'm eager to test patches to fix this. > I think we're on the right track. The problem with Jens' patch was > that it didn't consider the fact that blk_end_request

Re: [uml-devel] [PATCH 1/1] um: ubd: Fix data corruption

2010-10-05 Thread Tejun Heo
On 10/04/2010 09:51 PM, Chris Frey wrote: > On Mon, Oct 04, 2010 at 06:37:36PM +0200, Tejun Heo wrote: >> Hello, sorry about chiming in later. I was off last week. > > No problem, I'm eager to test patches to fix this. > >> I think we're on the right track. The problem with Jens' patch was >> t