
On 06/07/2016 11:59 AM, Jovanka Gulicoska wrote:
--- src/node_device/node_device_hal.c | 26 +++++++++++++------- src/node_device/node_device_udev.c | 49 +++++++++++++++++++++++--------------- 2 files changed, 47 insertions(+), 28 deletions(-)
I pushed patches 1-3, thanks! This patch needs to be rebased on master, since node_device_udev had a bunch of cleanups recently. There's also one issue I noticed:
diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c index 7d111c4..3315153 100644 --- a/src/node_device/node_device_udev.c +++ b/src/node_device/node_device_udev.c
@@ -1691,9 +1701,8 @@ static int udevPCITranslateInit(bool privileged ATTRIBUTE_UNUSED) * situation, but a non-privileged user won't benefit much * from udev in the first place. */ if (errno != ENOENT && (privileged || errno != EACCES)) { - char ebuf[256]; - VIR_ERROR(_("Failed to initialize libpciaccess: %s"), - virStrerror(pciret, ebuf, sizeof(ebuf))); + virReportError(VIR_ERR_INTERNAL_ERROR, "%s", + _("Failed to initialize libpciaccess: %s")); return -1; } }
I think you forgot to preserve the virStrerror call here, otherwise the string format is incorrect, with the trailing ": %s" Thanks, Cole