Re: [PATCH] remove TCGETS

[Andreas Dilger <adilger@clusterfs.com>]

On Aug 12, 2003  22:48 +0100, Matthew Wilcox wrote:
> I see no reason to handle TCGETS in presto_ioctl, and even if you do get
> it somehow, the action is the same as the default.  This happens to break
> on PA-RISC and not on i386 due to i386 using an old-style definition of
> TCGETS and PA-RISC using one which invovles sizeof().
> 
> diff -urpNX dontdiff linus-2.6/fs/intermezzo/dir.c parisc-2.6/fs/intermezzo/dir.c
> --- linus-2.6/fs/intermezzo/dir.c	Tue Aug 12 13:11:17 2003
> +++ parisc-2.6/fs/intermezzo/dir.c	Tue Aug 12 13:29:43 2003
> @@ -1304,10 +1304,6 @@ int presto_ioctl(struct inode *inode, st
>                  return rc;
>          }
>  
> -        case TCGETS:
> -                EXIT;
> -                return -EINVAL;
> -
>          default:
>                  EXIT;
>                  return -EINVAL;

I can tell you why this was originally in there - because the "default" case
used to print out an error message for unhandled ioctls.  Perl used to call
TCGETS all the time on files in lustre, so we put in the no-op case to shut
up the error messages.  Now that the error messages are gone from the default
case there is no need to have this there anymore, so removing it is fine.

Cheers, Andreas
--
Andreas Dilger
http://sourceforge.net/projects/ext2resize/
http://www-mddsp.enel.ucalgary.ca/People/adilger/