On Fri, Jul 24, 2015 at 12:11:46PM -0700, Roland Dreier wrote: > From: Roland Dreier <roland@xxxxxxxxxxxxxxx> > > > Shouldn't the code actually report the virtual lun zero instead of just > > accounting for it? > > Yeah, I think there is an implicit memset to 0 somewhere that is > saving us, but the below is probably better. > > - R. > > -------- 8< -------- > > Subject: [PATCH] target: REPORT LUNS should return LUN 0 even for dynamic ACLs > From: Roland Dreier <roland@xxxxxxxxxxxxxxx> > > If an initiator doesn't have any real LUNs assigned, we should report > LUN 0 and a LUN list length of 1. Some versions of Solaris at least > go beserk if we report a LUN list length of 0. > > Signed-off-by: Roland Dreier <roland@xxxxxxxxxxxxxxx> Thanks, this looks better. Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
