Messages in this thread | | | Date | Wed, 3 Dec 2008 12:08:29 -0500 | From | Christoph Hellwig <> | Subject | Re: [PATCH 3/6] integrity: IMA as an integrity service provider |
| |
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..
| |