svn commit: r232183 - head/sys/kern

Ed Schouten ed at 80386.nl
Sun Feb 26 21:12:51 UTC 2012


Hi Jilles,

* Jilles Tjoelker <jilles at FreeBSD.org>, 20120226 16:14:
> +static int
> +pipe_chmod(fp, mode, active_cred, td)
> +	struct file *fp;
> + 	mode_t mode;
> +	struct ucred *active_cred;
> +	struct thread *td;
> +{
> +	struct pipe *cpipe;
> +	int error;
> +
> +	cpipe = fp->f_data;
> +	if (cpipe->pipe_state & PIPE_NAMED)
> +		error = vn_chmod(fp, mode, active_cred, td);
> +	else
> +		error = invfo_chmod(fp, mode, active_cred, td);
> +	return (error);
> +}

Maybe this would be a useless optimisation, but wouldn't it be better to
just use two separate struct fileops here?

-- 
 Ed Schouten <ed at 80386.nl>
 WWW: http://80386.nl/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20120226/d7f7dcdc/attachment.pgp


More information about the svn-src-all mailing list