linux-kernel - Re: [GIT] Linux client patches against Linux 2.6.18

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] [day] [month] [year] [list]

Message-Id: <1159079875.5633.74.camel@lade.trondhjem.org>
Date:	Sun, 24 Sep 2006 02:37:55 -0400
From:	Trond Myklebust <trond.myklebust@....uio.no>
To:	Linus Torvalds <torvalds@...l.org>
Cc:	Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org
Subject: Re: [GIT] Linux client patches against Linux 2.6.18

On Sat, 2006-09-23 at 17:06 -0700, Linus Torvalds wrote:

> I'm not seeing a sign-off or ack for the dentry stuff from Al, for 
> example. Was it passed by him? You moved the dentry rehash function inside 
> the dcache_lock, and if that was a bug-fix, it should have been marked as 
> so and done separately etc.

The only function that was moved inside the spinlock was d_hash(), but
that is a fairly trivial change (d_revalidate already does the same).
Otherwise, the only effect of the changes was to create unlocked
versions of d_rehash() for reuse in the new d_materialise_dentry().

In any case, all of those patches have been presented by David on the
fs-devel and lkml lists several times. The last time would be on 27th
July:

 http://marc.theaimsgroup.com/?l=linux-kernel&m=115403487316197&w=2

Al has in addition been Cced on much of the correspondence between David
and myself during the preparation for merging (although that would be
almost half a year ago now).
Finally, Christoph was also pressed into duty as an extra reviewer.

So yes, afaik all the relevant people have been kept informed, even if
they were not labelled as having been Cced in the changelog.

So I apologize for not having sent this as a separate patch, but as I
said above, it was a trivial change as far as the rehash is concerned,
and the only user of the new function is NFS.

Cheers,
  Trond

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