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