On Fri, Apr 09, 2021 at 04:32:55PM -0500, Jonathon Jongsma wrote: > Erik recommended several changes to the mdev series that was just merged and > suggested to address them in a follow-up patch series. They're mostly related > to simplifying the testing, and shouldn't actually change any behavior (aside > from switching to using the long commandline options when executing mdevctl). > > Erik Skultety (4): > nodedev: driver: Swap virMdevctlStart and virMdevctlCreate > nodedev: driver: Introduce internal mdevctl commands enum > nodedev: driver: Create a generic mdevctl command translator > tests: nodedev: Make the mdevctl test function and helper generic > > Jonathon Jongsma (7): > nodedev: don't log error in nodeDeviceFindAddressByName() > nodedev: avoid use of VIR_ERR_NO_* errors internally > tests: nodedev: switch all test macros to accept a filename > nodedev: Switch to using long options for mdevctl > nodedev: Remove GetMdevctl*Command() wrappers > tests: nodedev: simplify test macros > tests: nodedev: remove unused variable This fails to apply: Applying: tests: nodedev: Make the mdevctl test function and helper generic error: sha1 information is lacking or useless (tests/nodedevmdevctltest.c). error: could not build fake ancestor Patch failed at 0009 tests: nodedev: Make the mdevctl test function and helper generic There were a bunch of other merge conflicts even before this patch, so please respin a rebased version. Regards Erik
