linux-kernel - Re: nfsd changes for 3.6

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 for Android: free password hash cracker in your pocket

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

Date:	Tue, 31 Jul 2012 14:54:05 -0700
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	"J. Bruce Fields" <bfields@...ldses.org>
Cc:	linux-nfs@...r.kernel.org, linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org, viro@...iv.linux.org.uk,
	matthew@....cx
Subject: Re: nfsd changes for 3.6

On Tue, Jul 31, 2012 at 7:36 AM, J. Bruce Fields <bfields@...ldses.org> wrote:
>
> By the way, for a few years now I've been semi-regularly picking up
> locks.c changes for my tree.  I wonder if I should be.

I really wish you didn't. Or at least not this way.

For example, now you removed that lm_release_private case, but you did
that without apparently talking to any VFS maintainers.

And the commit seems to be pure shit.

Why is it pure shit? Look at what users are left. THERE ARE NO USERS
THAT SET THAT FIELD ANY MORE!

Maybe I missed some odd user that somehow avoids the grep, but I don't
think so. So there is nothing that ever sets it, why the hell does the
thing still remain at all?

So if this is a real bugfix, then I think it damn well should

 (a) at a minimum have been discussed with VFS maintainers

 (b) have been thought through a bit more (ie remove all remains of
that now useless lm_release_private field)

 (c) likely never have gone through the NFSD tree in any case.

I've pulled the changes for now, but I absolutely *detest* seeing
things like this. I care very little about the nfsd code, but I care
deeply when I see insanity happening in the VFS layer. And right now,
"lm_release_private" is insane as of this pull.

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