On 5/3/21 8:03 AM, Hannes Reinecke wrote: > - tag = sc->request->tag; > - if (unlikely(tag < 0)) { > - /* > - * Really should fix the midlayer to pass in a proper > - * request for ioctls... > - */ > - tag = fnic_scsi_host_start_tag(fnic, sc); > - if (unlikely(tag == SCSI_NO_TAG)) > - goto fnic_device_reset_end; > - tag_gen_flag = 1; > - new_sc = 1; > - } Since this patch removes the only callers of fnic_scsi_host_start_tag() and fnic_scsi_host_end_tag(), please modify this patch such that it also removes these functions. Thanks, Bart.