Hi Thomas, > > + } > > else if (!strcmp(lgopts[option_index].name, > OPT_CREATE_UIO_DEV)) > > another code style issue reported by checkpatch.pl ;) > > But it should be fixed by removing this code as Stephen suggests.
I'm not sure this could should be removed. Igb_uio allows to pick interrupt mode, so why not VFIO? I've modified my code to try all interrupt modes if nothing was explicitly specified, but why should that preclude the user from selecting a specific interrupt type if he so desires? As for the style error - the whole chunk of code uses the same style there, so either we fix all of that (in a separate patch?), or leave it as it is. Best regards, Anatoly Burakov DPDK SW Engineer