On Tue, Apr 13, 2021 at 03:39:36PM -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). > > Changes in v2: > - rebase to lastest git master > > 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 (8): > 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 > tests: nodedev: remove unnecessary cleanup label The changes look good to me and I'm tempted to put my R-b even on the rest of the patches, but since I have a fair share on the patches that I didn't comment on, it suddenly feels like circumventing the rules (I know I suggested you go ahead and submit the patches). Let's give it a few days and hopefully someone else gives you a second opinion especially on the patches that bear my signoff. Regards, Erik
