Re: [PATCH] Remove pty_ops_bsd and pty_bsd_ioctl() as they're not used

From: David Howells
Date: Wed Sep 30 2009 - 06:21:17 EST


Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:

> So I do wonder whether the right thing to do would not be to simple remove
> the whole pty_ops_bsd code entirely. Or maybe mode the trivial TIOCSPTLCK
> into the generic tty ioctl handling, and just make it test for
> "driver->subtype == PTY_TYPE_MASTER" - and at least get rid of this subtle
> thing that was broken for over a year without anybody noticing..

Also, does it matter if someone issues TIOCGPTN on a legacy pty master? There
is a number in tty->index that can be returned. If it can, then
pty_bsd_ioctl() can be merged with pty_unix98_ioctl(), and possibly both
ioctls can be given to the generic tty ioctl handling with the condition that
driver->subtype == PTY_TYPE_MASTER.

David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/