Re: [lustre-devel] [PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-22 Thread NeilBrown
On Mon, May 21 2018, Patrick Farrell wrote: > This, and the rest of the series, look good. Feel free to add a Reviewed-by. Thanks a lot! NeilBrown signature.asc Description: PGP signature

Re: [lustre-devel] [PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-22 Thread NeilBrown
On Mon, May 21 2018, Patrick Farrell wrote: > This, and the rest of the series, look good. Feel free to add a Reviewed-by. Thanks a lot! NeilBrown signature.asc Description: PGP signature

Re: [lustre-devel] [PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-21 Thread Patrick Farrell
This, and the rest of the series, look good. Feel free to add a Reviewed-by. Thanks, Neil. On 5/20/18, 11:39 PM, "lustre-devel on behalf of NeilBrown" wrote: libcfs allows other modules to register handlers for ioctls.

Re: [lustre-devel] [PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-21 Thread Patrick Farrell
This, and the rest of the series, look good. Feel free to add a Reviewed-by. Thanks, Neil. On 5/20/18, 11:39 PM, "lustre-devel on behalf of NeilBrown" wrote: libcfs allows other modules to register handlers for ioctls. The implementation it uses for this is nearly identical to a

[PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-20 Thread NeilBrown
libcfs allows other modules to register handlers for ioctls. The implementation it uses for this is nearly identical to a blocking notifier chain, so change to use that. The biggest difference is that the return value from notifier has a defined format, where libcfs_register_ioctl uses -EINVAL to

[PATCH 13/30] staging: lustre: replace libcfs_register_ioctl with a blocking notifier_chain

2018-05-20 Thread NeilBrown
libcfs allows other modules to register handlers for ioctls. The implementation it uses for this is nearly identical to a blocking notifier chain, so change to use that. The biggest difference is that the return value from notifier has a defined format, where libcfs_register_ioctl uses -EINVAL to