linux-kernel - Re: [PULL REQUEST] Ext2, ext3, and quota updates for 2.6.35-rc1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 21 May 2010 21:24:56 +0200
From:	Jan Kara <jack@...e.cz>
To:	Linus Torvalds <torvalds@...ux-foundation.org>
Cc:	Jan Kara <jack@...e.cz>, LKML <linux-kernel@...r.kernel.org>,
	linux-fsdevel@...r.kernel.org
Subject: Re: [PULL REQUEST] Ext2, ext3, and quota updates for 2.6.35-rc1

On Fri 21-05-10 11:50:15, Linus Torvalds wrote:
> 
> 
> On Fri, 21 May 2010, Jan Kara wrote:
> >
> >   Oops, I've just found a bug in one of my ocfs2 fixes. I've fixed that up
> > so the new diffstat is here:
> 
> I already pulled your previous version.
  OK, I'll push the fix separately.

> And btw, Jan, I'm getting pretty upset with your apparently cavalier 
> attitude to filesystems. We've had several bugs lately for some quota or 
> filesystem "cleanups", which in at least one case caused corrupted 
> filesystems etc.
  That ext3 bug was nasty I agree :( I'm now pushing some quota tests to
XFSQA and setup a machine that runs them. So that should improve a test
coverage...

> You need to be a hell of a lot more careful, or I will have to stop 
> pulling from you. Filesystem bugs are _major_ no-no's. 
> 
> How long had that buggy commit been tested? By who? Was it acked or 
> reviewed by anybody else?
  The buggy commit was "ocfs2: Fix lock inversion in quotas during umount"
and it was reviewed by Joel Becker. Neither of us realized that we have to
stop quota syncing thread also on remount read-only...

								Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/