Hi Antti,
El 08/06/15 a les 07:53, Antti Kantee ha escrit:
>> +int rumpcomp_pci_init(int, int *);
>
> Now that rumpcomp_userfeatures_pci.h is available, that signature can be
simplified.
That'd be "int rumpcomp_pci_init(void);" ?
El 08/06/15 a les 23:20, Antti Kantee ha escrit:
I'm just worried that your print will look too much like an error when it is
not necessary so. Perhaps there's a way to make the print softer, something to
indicate that some features are missing most likely because the program is not
being run as root or with CAP_SYS_RAWIO. In other words, treat EPERM as the
common case error.
Also, the print and the call doesn't match: pci_init() vs. "iopl failed".
Finally, commenting on the hypercall side of the patch, you should not return
the return value of iopl(), but rather the translated errno.