Re: [PATCH] [13/21] Use blk_q_mask/get_pages_mask in sg driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Yeah, I know. It's not nice (as I wrote in the first mail), it doesn't
> work well, etc. But it's not the point (as I wrote in the previous
> mail). I'm against the patch just because I don't think the rules
> about the userspace API permit removing the exiting API like this.

Ok perhaps removing the define was a bit much, but at least stubbing
the ioctl should be transparent to the application.

Ok except that I return EINVAL in the set variant now. I could lie and return 0
instead, although I suspect it wouldn't make much difference either 
way.
 
> 
> If the rules permit, I'm happy to try to remove more old features of
> sg.

As long as old binaries don't break I don't see a problem with
any such changes (afaik that's true for my change)

If you want to break something you have to just go through
feature deprecation and printk, but normally it's only a good
idea to do that if it saves a lot of code or ugliness.

-Andi

-- 
ak@xxxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux