To: hch@infradead.org
Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org,
       linux-security-module@vger.kernel.org, takedakn@nttdata.co.jp
Subject: Re: [TOMOYO #6 02/21] Add struct vfsmount to struct task_struct.
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
References: <20080108094903.484613061@nttdata.co.jp>
	<20080108094956.610620186@nttdata.co.jp>
	<20080108140902.GA12238@infradead.org>
In-Reply-To: <20080108140902.GA12238@infradead.org>
Message-Id: <200801082339.CGC57333.OJFVHMSFOLFtOQ@I-love.SAKURA.ne.jp>
Date: Tue, 8 Jan 2008 23:39:03 +0900
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1078
Lines: 29

Hello.

Christoph Hellwig wrote:
> Just FYI:  A NACK to such an addition doesn't simply go away by
> ignoring it.
Excuse me. What NACK is remaining?

About the below comments?
> > NACK to this. Passing function parameters through the task_struct is
> > definitely not an acceptable hack
> 
> Exactly.  Having a vfsmount other than the current processes root or
> current working directory in task_struct doesn't make any sense.

I explained that this approach pushes the parameter on the stack memory
and behaves as if the parameter was passed by function calls.
http://lkml.org/lkml/2007/11/19/80

But if this approach is still unacceptable,
please discuss and adopt AppArmor's patches as soon as possible.
Both AppArmor and TOMOYO needs vfsmount parameters, but
I'm sad to see no comments on AppArmor's vfsmount patches.

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