On Tue, Feb 11, 2014 at 03:34:53PM +0100, Hannes Reinecke wrote: > Instead of modifying attributes after the device has been created > we should be using the 'is_visible' callback to avoid races. Can't say I like the fact that the is_visible function needs is per-group and not per attribute and thus needs to explicitly check which attribute we call it for, but that seems to be a design issue in the driver core and not the patch. Otherwise looks good with the minor nitpick that it would have looked cleaner in the patch if you had left the queue_type store function in the location it previously was. Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- 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
