dev_t / udev_t confusion ?

Scott Long scottl at freebsd.org
Wed Jun 9 17:03:25 GMT 2004


Poul-Henning Kamp wrote:

> In message <53993.1086779790 at critter.freebsd.dk>, Poul-Henning Kamp writes:
> 
> 
>>The change proposed is more or less to do:
>>	s/dev_t/struct cdev */
>>	s/udev_t/dev_t/
>>over all the kernel sources (366 files or so).
> 
> 
> Looks like a "yea" so far, so I have a couple of follow-up questions:
> 
> 	struct cdev currently has members named si_* because it
> 	used to be called "specinfo", do we want to change that
> 	inconsistency at the same time ? (either by reverting to
> 	the specinfo name or by changing to a cd_ prefix ?

Sounds fine to me.  No prefix at all would work too.

> 
> 	cdevsw->ioctl() takes a caddr_t pointer argument which
> 	really should be a void *, do we want to change that
> 	as well (since it is all the same files we'll have to
> 	change).
> 

Is this going to have any consequences on COMPAT_LINUX code or anything
else that calls ioctl() through obscure means?

Scott


More information about the freebsd-arch mailing list