linux-kernel - Re: [patch 0/7] Linux Kernel Markers (redux)

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]

Message-ID: <20070924181148.GA3285@infradead.org>
Date:	Mon, 24 Sep 2007 19:11:48 +0100
From:	Christoph Hellwig <hch@...radead.org>
To:	Mathieu Desnoyers <mathieu.desnoyers@...ymtl.ca>
Cc:	akpm@...ux-foundation.org, linux-kernel@...r.kernel.org,
	Christoph Hellwig <hch@...radead.org>
Subject: Re: [patch 0/7] Linux Kernel Markers (redux)

On Mon, Sep 24, 2007 at 12:49:50PM -0400, Mathieu Desnoyers wrote:
> Hi Andrew,
> 
> Following Christoph Hellwig's suggestion, aiming at a Linux Kernel Markers
> inclusion for 2.6.24, I made a simplified version of the Linux Kernel Markers.
> There are no more dependencies on any other patchset.
> 
> The modification only involved turning the immediate values into static
> variables and adapting the documentation accordingly. It will have a little more
> data cache impact when disabled than the version based on the immediate values,
> but it is far less complex.
> 
> Since things have not moved much in the markers area recently (most of the
> concerns were about the immediate values), I expect it to be ready for 2.6.24.

I've done a quick review and except for my usual nitpicking it looks
very good to me.  I'll try my sputrace code with this version and will
report on how it works.

I'd really love to see markers in 2.6.24.  The impact to core code of
the actual markers is minimal with some more struct fields in functions
in the module code, although we'll want Mathieu's fix for the modules
list aswell which is a little more invasive.  But I think we want that
one anyway.

I'd say we probably don't want the blktrace conversion that late in the
game, though - but we can queue that once together with the optimizations
from Mathieu's earlier patches in -mm.

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