On Thu, Feb 16, 2017 at 10:18:59AM -0700, Jon Derrick wrote: > It looks good to me at first glance but I can't apply it. What tree are > you on? > > On 02/16/2017 12:58 AM, Christoph Hellwig wrote: > > I'd rather prefer to make the structure separately allocated as > > discussed before. Scott, can you test the patch below? I'm not near > > my devices I could test on. > > He doesn't have the most recent changes with the uapi IOW fixes. I got it applied and tested and it works. Going to review it more aggressively now. We had something very similar in one of the previous patches before we switched off of using a sed_context struct. If you do want to apply and test I did the following: 1) Pull Jens' for-next 2) reset fb2a77e4a25ef63ff5b51b3bd53027077b402b0d --hard 3) apply his patch 4) git pull https://kernel.googlesource.com/pub/scm/linux/kernel/git/axboe/linux-block.git for-next 5) resolve the 2 small conflicts in core.c and sed-opal.h in sed-opal.h take his init_opal_dev change and the newer sed_ioctl in core.c take his change but change arg to be (void __user *) arg);
