On Tue, 2015-02-03 at 11:37 +0200, Michael S. Tsirkin wrote: > On Tue, Feb 03, 2015 at 06:30:01AM +0000, Nicholas A. Bellinger wrote: > > From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx> > > > > With the new ANY_LAYOUT logic in place for vhost_scsi_handle_vqal(), > > there is no longer a reason to keep around the legacy code with > > !ANY_LAYOUT assumptions. > > > > Go ahead and drop the pre virtio 1.0 logic in vhost_scsi_handle_vq() > > and associated helpers. > > Will probably be easier to review if you smash this with patch 5, > this way we see both old and new code side by side. > So was thinking it might be useful to leave this as a separate patch, just in case !ANY_LAYOUT needs to be resurrected for some reason, so both modes can function. But at this point I really don't see a case why that would be necessary.. > Also, let's rename _vqal to _vq in this patch? Merging this patch into 7, changing name to _vq. --nab -- 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