Re: [PATCH 3/6] integrity: IMA as an integrity service provider

From: Christoph Hellwig
Date: Wed Dec 03 2008 - 12:08:41 EST


On Wed, Dec 03, 2008 at 08:55:54AM -0800, Dave Hansen wrote:
> On Wed, 2008-12-03 at 08:03 -0500, Christoph Hellwig wrote:
> > On Tue, Dec 02, 2008 at 03:35:25PM -0800, Dave Hansen wrote:
> > > If you're looking for another way to split your patch out, you could
> > > just stick use inode->integrity for the first patch, then introduce your
> > > radix-tree in a subsequent patch. It would be functionally fine, and
> > > more clearly separate out the ideas.
> >
> > This was done earlier and is a really bad thing for review. Just as the
> > breakout happening in this round.
>
> If Christoph and I tell you conflicting things, it goes without saying
> to listen to Christoph. :)

Or listen to what makes more sense. It's not the most important comment
of the review anyway..

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/