On Sat, Sep 18, 2021 at 03:57:29PM -0600, Anton Eidelman wrote: > +void nvme_mpath_update(struct nvme_ctrl *ctrl) > +{ > + nvme_read_ana_log(ctrl); > +} What is the point of this wrapper vs just calling nvme_read_ana_log directly? Also shouldn;t we propagate the error here and through nvme_start_ctrl?