On 01/26/2017 12:36 AM, Bart Van Assche wrote: > Remove the code that clears .se_lun from > transport_cmd_check_stop_to_fabric() such that the > transport_lun_remove_cmd() call can be moved into > target_release_cmd_kref(). Because this guarantees that > transport_lun_remove_cmd() will be called exactly once, it is > safe to change the cmpxchg() call into a test of > se_cmd.lun_ref_active. Inline transport_lun_remove_cmd() because > it is not worth to keep it as a separate function. > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxx> > Cc: Andy Grover <agrover@xxxxxxxxxx> > Cc: David Disseldorp <ddiss@xxxxxxx> > --- > drivers/target/target_core_transport.c | 30 +++--------------------------- > 1 file changed, 3 insertions(+), 27 deletions(-) > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- 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