On Sun, 2018-07-15 at 18:16 -0500, Mike Christie wrote: > +static void target_fabric_session_release(struct config_item *item) > +{ > + struct se_session *se_sess = container_of(to_config_group(item), > + struct se_session, group); > + target_release_session(se_sess); > +}
Please consider changing the target_fabric_ prefix into target_ for all new functions. Otherwise this patch looks fine to me. Bart.