On Wed, 2014-02-19 at 10:59 +0100, Geert Uytterhoeven wrote: > On Wed, Feb 19, 2014 at 10:52 AM, Paul Bolle <pebolle@xxxxxxxxxx> wrote: > > - ret = core_alua_state_nonoptimized(cmd, cdb, > > - nonop_delay_msecs, &alua_ascq); > > + core_alua_state_nonoptimized(cmd, cdb, nonop_delay_msecs); > > I suggest making core_alua_state_nonoptimized() return void, too. > Currently it always returns zero. Good catch. Not sure how I missed that, since I deliberately ignore its return value here. I'll wait with sending a v2 until after Nicholas or the other target developers have given their feedback. Paul Bolle -- 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
