Message ID | 1678229374-18788-1-git-send-email-bmarzins@redhat.com (mailing list archive) |
---|---|
Headers | show |
Series | handle transitioning devices in TUR checker | expand |
Brian, could you give this patch set a test in your environment? On Tue, 2023-03-07 at 16:49 -0600, Benjamin Marzinski wrote: > This patchset is based on Brian Bunker's "libmultipath: return > 'ghost' state when port is in transition" patch: > > https://listman.redhat.com/archives/dm-devel/2023-February/053344.html > https://github.com/opensvc/multipath-tools/pull/60 > > Instead of setting the state to PATH_GHOST, it uses PATH_PENDING. The > other two patches are small cleanups to the TUR checker that I > noticed > while writing the first patch. > > Benjamin Marzinski (3): > libmultipath: return 'pending' state when port is in transition > libmultipath: set init failure message when init fails > libmultipath: reset nr_timeouts if we freed the context > > libmultipath/checkers/tur.c | 24 +++++++++++++++++++----- > 1 file changed, 19 insertions(+), 5 deletions(-) > I'd like to wait for Brian's results. From code inspection, and with Ben's explanation for the 3/3 logic, For the set: Reviewed-by: Martin Wilck <mwilck@suse.com> -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel
> On Mar 9, 2023, at 12:38 AM, mwilck@googlemail.com wrote: > > Brian, > > could you give this patch set a test in your environment? > > On Tue, 2023-03-07 at 16:49 -0600, Benjamin Marzinski wrote: >> This patchset is based on Brian Bunker's "libmultipath: return >> 'ghost' state when port is in transition" patch: >> >> https://listman.redhat.com/archives/dm-devel/2023-February/053344.html >> https://github.com/opensvc/multipath-tools/pull/60 >> >> Instead of setting the state to PATH_GHOST, it uses PATH_PENDING. The >> other two patches are small cleanups to the TUR checker that I >> noticed >> while writing the first patch. >> >> Benjamin Marzinski (3): >> libmultipath: return 'pending' state when port is in transition >> libmultipath: set init failure message when init fails >> libmultipath: reset nr_timeouts if we freed the context >> >> libmultipath/checkers/tur.c | 24 +++++++++++++++++++----- >> 1 file changed, 19 insertions(+), 5 deletions(-) >> > > I'd like to wait for Brian's results. From code inspection, and with > Ben's explanation for the 3/3 logic, > > For the set: > Reviewed-by: Martin Wilck <mwilck@suse.com> Ben and Martin, This works well for me against my array. A couple of things in this part Can you make the above match your formatting since it is better: if( asc == 0x04 && ascq == 0x0b){ if (asc == 0x04 && ascq == 0x0a) { And I don’t think you need else if since the line above it returns in its case, so else is not hit if the first if is true: return PATH_GHOST; } else if (asc == 0x04 && ascq == 0x0a) { Thanks, Brian -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel
On Fri, Mar 10, 2023 at 11:55:19AM -0800, Brian Bunker wrote: > > On Mar 9, 2023, at 12:38 AM, mwilck@googlemail.com wrote: > > > > Brian, > > > > could you give this patch set a test in your environment? > > > > On Tue, 2023-03-07 at 16:49 -0600, Benjamin Marzinski wrote: > >> This patchset is based on Brian Bunker's "libmultipath: return > >> 'ghost' state when port is in transition" patch: > >> > >> https://listman.redhat.com/archives/dm-devel/2023-February/053344.html > >> https://github.com/opensvc/multipath-tools/pull/60 > >> > >> Instead of setting the state to PATH_GHOST, it uses PATH_PENDING. The > >> other two patches are small cleanups to the TUR checker that I > >> noticed > >> while writing the first patch. > >> > >> Benjamin Marzinski (3): > >> libmultipath: return 'pending' state when port is in transition > >> libmultipath: set init failure message when init fails > >> libmultipath: reset nr_timeouts if we freed the context > >> > >> libmultipath/checkers/tur.c | 24 +++++++++++++++++++----- > >> 1 file changed, 19 insertions(+), 5 deletions(-) > >> > > > > I'd like to wait for Brian's results. From code inspection, and with > > Ben's explanation for the 3/3 logic, > > > > For the set: > > Reviewed-by: Martin Wilck <mwilck@suse.com> > Ben and Martin, > > This works well for me against my array. A couple of things in this part > > Can you make the above match your formatting since it is better: > if( asc == 0x04 && ascq == 0x0b){ > > if (asc == 0x04 && ascq == 0x0a) { > > And I don’t think you need else if since the line above it returns in its > case, so else is not hit if the first if is true: > > return PATH_GHOST; > } else if (asc == 0x04 && ascq == 0x0a) { > Sure. -Ben > Thanks, > Brian > > -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel