On Mon, Jul 28, 2014 at 05:35:34AM +0200, J?rgen Gro? wrote: > On 07/26/2014 05:23 PM, Christoph Hellwig wrote: > >Just a quick glance: > > > > a) this should move to drivers/target with the other target code > > I don't mind. I just followed the example of drivers/vhost/vhost.c and > thought the similar xen module should be located under drivers/xen. Oh sorry, I overlooked that you moved it to drivers/xen from drivers/scsi in the initial submission. That's fine with me. > > b) you're still having your own CDB emulation in there, the target code > > should be taking care of all that. > > Okay, I'll remove the CDB emulation. Thanks. If there's anything missing that Xen except feel free to adopt the target core to it. -- 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