Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 9 Apr 2008 11:59:30 -0600
From:      John E Hein <jhein@timing.com>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        arch@FreeBSD.org, phk@phk.freebsd.dk
Subject:   Re: tt_ioctl 
Message-ID:  <18429.1026.716346.691351@gromit.timing.com>
In-Reply-To: <20080409.101816.1824031653.imp@bsdimp.com>
References:  <40914.1207681578@critter.freebsd.dk> <20080409.044228.-201314267.imp@bsdimp.com> <18428.59782.318085.53492@gromit.timing.com> <20080409.101816.1824031653.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
M. Warner Losh wrote at 10:18 -0600 on Apr  9, 2008:
 > Yes.  I almost just quietly committed it to FreeBSD at the time, but I
 > got busy on another project and never got back to it.  Maybe I should
 > have just done it and saw if phk noticed :-)

Never try to accomplish through planning what can be done more simply
through subterfuge.


 > : Well, I could go either way on this issue - 'specious' might be a bit
 > : too strong.  I could see issues with pass-through device-specific
 > : ioctls on tty devs - especially due to the fact that it's a tty device
 > : is somewhat obscured in the case of ucom children.
 > 
 > I'm not sure I follow what you are saying here...

I'm not sure which part your asking about, so I'll just try to be a
bit more verbose.

 - re: issues with pass-thru ioctls... I'm just commenting that I
   understand phk's concerns about potentially ill-advised ioctls on
   the tty dev.

 - re: ucom-based driver ioctls... those drivers just set the
   ucomioctl method and so don't know (without digging) that it hooks
   up to the tty ioctl method... hence the 'obscured' comment and
   since the relationship to the tty device is somewhat obscured
   through an indirect relationship, the ucom-based driver writer
   may not be aware of the potential problems with exposing ioctls
   on the tty dev.




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