On Tue, Sep 17, 2024 at 7:07 PM Benjamin Marzinski <bmarzins@xxxxxxxxxx> wrote: > > The only input fc_rport_set_marginal_state() currently accepts is > "Marginal" when port_state is "Online", and "Online" when the port_state > is "Marginal". It should also allow setting port_state to its current > state, either "Marginal or "Online". > > Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> > --- Both scsi_host_set_state() and scsi_device_set_state() work this way, so this seems quite reasonable. Reviewed-by: Ewan D. Milne <emilne@xxxxxxxxxx>