[dm-devel] [PATCH 3/5] libmultipath: change loading and resetting in directio
Benjamin Marzinski
bmarzins at redhat.com
Wed Feb 19 15:29:33 UTC 2020
On Wed, Feb 19, 2020 at 10:42:50AM +0100, Martin Wilck wrote:
> On Wed, 2020-02-19 at 00:54 -0600, Benjamin Marzinski wrote:
> > The directio checker previously made sure to always keep an aio_group
> > around after loading or resetting the checker. There is no need to do
> > this, and removing this code simplifies the checker. With this
> > change,
> > there is no longer a need for a load or unload checker function, only
> > a
> > reset function which is run when the checker is reset or unloaded.
> > Changing this broke a number of tests, so the unit tests have been
> > updated as well.
> >
> > Signed-off-by: Benjamin Marzinski <bmarzins at redhat.com>
> > ---
> > libmultipath/checkers.c | 26 ++---
> > libmultipath/checkers.h | 2 +-
> > libmultipath/checkers/directio.c | 43 +-------
> > tests/directio.c | 177 +++++++++++++--------------
> > ----
> > 4 files changed, 81 insertions(+), 167 deletions(-)
>
> Many thanks for making this change. I feared that the directio tests
> were tied quite closely to implementation details...
I've always been looking at the unit tests as a way to make sure that we
exercise code paths that are hard to hit with functional tests in a more
real-world environment. As such, the idea is to write them specifically
to match the code.
>
> Reviewed-by: Martin Wilck <mwilck at suse.com>
>
More information about the dm-devel
mailing list