Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 31 Oct 2009 19:03:53 -0600 (MDT)
From:      "M. Warner Losh" <imp@bsdimp.com>
To:        ed@FreeBSD.org
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r198706 - head/sys/sys
Message-ID:  <20091031.190353.-332187386.imp@bsdimp.com>
In-Reply-To: <200910311035.n9VAZfIb082932@svn.freebsd.org>
References:  <200910311035.n9VAZfIb082932@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In message: <200910311035.n9VAZfIb082932@svn.freebsd.org>
            Ed Schouten <ed@FreeBSD.org> writes:
: Author: ed
: Date: Sat Oct 31 10:35:41 2009
: New Revision: 198706
: URL: http://svn.freebsd.org/changeset/base/198706
: 
: Log:
:   Turn unused structure fields of cdevsw into spares.
:   
:   d_uid, d_gid and d_mode are unused, because permissions are stored in
:   cdevpriv nowadays. d_kind doesn't seem to be used at all. We no longer
:   keep a list of cdevsw's, so d_list is also unused.
:   
:   uid_t and gid_t are 32 bits, but mode_t is 16 bits, Because of alignment
:   constraints of d_kind, we can safely turn it into three 32-bit integers.
:   d_kind and d_list is equal in size to three pointers.
:   
:   Discussed with:	kib

Don't we need a D_VERSION bump for this?

Warner


: Modified:
:   head/sys/sys/conf.h
: 
: Modified: head/sys/sys/conf.h
: ==============================================================================
: --- head/sys/sys/conf.h	Sat Oct 31 09:03:48 2009	(r198705)
: +++ head/sys/sys/conf.h	Sat Oct 31 10:35:41 2009	(r198706)
: @@ -210,15 +210,13 @@ struct cdevsw {
:  	d_kqfilter_t		*d_kqfilter;
:  	d_purge_t		*d_purge;
:  	d_mmap_single_t		*d_mmap_single;
: -	uid_t			d_uid;
: -	gid_t			d_gid;
: -	mode_t			d_mode;
: -	const char		*d_kind;
: +
: +	int32_t			d_spare0[3];
: +	void			*d_spare1[3];
:  
:  	/* These fields should not be messed with by drivers */
: -	LIST_ENTRY(cdevsw)	d_list;
:  	LIST_HEAD(, cdev)	d_devs;
: -	int			d_spare3;
: +	int			d_spare2;
:  	union {
:  		struct cdevsw		*gianttrick;
:  		SLIST_ENTRY(cdevsw)	postfree_list;
: 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20091031.190353.-332187386.imp>