On Thu, Feb 16, 2017 at 10:37:55AM -0700, Scott Bauer wrote: > 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); 6) pull the newest commit from https://github.com/ScottyBauer/sed-opal-temp to get the uapi header change