Re: [PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-11 Thread Anand Jain
On 12/08/2017 09:09 PM, Austin S. Hemmelgarn wrote: On 2017-12-07 21:17, Duncan wrote: Anand Jain posted on Fri, 08 Dec 2017 08:51:43 +0800 as excerpted: On 12/07/2017 10:52 PM, Austin S. Hemmelgarn wrote: On 2017-12-07 09:36, Anand Jain wrote: Add ability to deregister a or all devices.

Re: [PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-08 Thread Austin S. Hemmelgarn
On 2017-12-07 21:17, Duncan wrote: Anand Jain posted on Fri, 08 Dec 2017 08:51:43 +0800 as excerpted: On 12/07/2017 10:52 PM, Austin S. Hemmelgarn wrote: On 2017-12-07 09:36, Anand Jain wrote: Add ability to deregister a or all devices. I have named this sub cmd as deregister, but I am open

Re: [PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-07 Thread Duncan
Anand Jain posted on Fri, 08 Dec 2017 08:51:43 +0800 as excerpted: > On 12/07/2017 10:52 PM, Austin S. Hemmelgarn wrote: >> On 2017-12-07 09:36, Anand Jain wrote: >>> Add ability to deregister a or all devices. I have named this sub cmd >>> as deregister, but I am open to your suggestions. >>

Re: [PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-07 Thread Anand Jain
On 12/07/2017 10:52 PM, Austin S. Hemmelgarn wrote: On 2017-12-07 09:36, Anand Jain wrote: Add ability to deregister a or all devices. I have named this sub cmd as deregister, but I am open to your suggestions. Being a bit picky here, but from the perspective of a native speaker of American

Re: [PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-07 Thread Austin S. Hemmelgarn
On 2017-12-07 09:36, Anand Jain wrote: Add ability to deregister a or all devices. I have named this sub cmd as deregister, but I am open to your suggestions. Being a bit picky here, but from the perspective of a native speaker of American English, I would say that 'deregister' sounds rather

[PATCH v5 0/3] Add cli and ioctl to deregister devices

2017-12-07 Thread Anand Jain
Add ability to deregister a or all devices. I have named this sub cmd as deregister, but I am open to your suggestions. Further I am using /dev/btrfs-control and struct btrfs_ioctl_vol_args for now, just similar to other ioctls in super.c including btrfs dev scan, if rather it makes sense to use