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



Reply via email to