jail getfsstat patches.

Bjoern A. Zeeb bzeeb-lists at lists.zabbadoz.net
Sun Jun 27 08:56:13 PDT 2004

On Sun, 27 Jun 2004, Pawel Jakub Dawidek wrote:


first of all a BIG THANK YOU to Robert for the very detailed

> On Sun, Jun 27, 2004 at 08:59:32AM +0200, Pawel Jakub Dawidek wrote:
> +> If you give me a few days (maybe I'll be ready today) I'll try to prepare
> +> patch to commit so we can review it together.
> Ok, here it goes:
> 	http://people.freebsd.org/~pjd/patches/jail_enforce_statfs.patch

many thanks for the work; fetch says 17kb; looks like a very good
abstraction and also includes patches for the emulations; great.

I will look at this in detail.

One thing that I have seen while skipping through the first time:

could we avoid the function calls for non-jails or with
jail_enforce_statfs=0 ? This would make the code somewhat longer
as this part would be copied over multiple functions

	if (jailed(cred) && jail_enforce_statfs) {
		/* call of the two functions */

(perhaps use a macro ?) but save people outside jails, w/o jails
or with jail_enforce_statfs=0 the function calls.

To answer another question: though I maybe thought/said s.th. else in
the past I would like to keep the sysctl global and not have it per
jail (if we start doing per-jail things we might really consider
vimages (perhaps in 6-CURRENT) but that's out of the scope of
this discussion).


Bjoern A. Zeeb				bzeeb at Zabbadoz dot NeT

More information about the freebsd-current mailing list