On Sun, Apr 23, 2017 at 06:57:07PM +0400, Roman Bogorodskiy wrote:
Currently, virNetDevSetCoalesce() stub is always returning error. As it's used by virNetDevTapCreateInBridgePort(), it essentially breaks bridged networking if coalesce is not supported. To make it work, relax the stub to trigger error only when its coalesce argument is not NULL, otherwise report success. --- src/util/virnetdev.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
That should be fixed in the other stub as well. ACK with that added.
diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
index 131a56a8d..131c4d983 100644
--- a/src/util/virnetdev.c
+++ b/src/util/virnetdev.c
@@ -3155,8 +3155,11 @@ int virNetDevSetCoalesce(const char *ifname,
}
# else
int virNetDevSetCoalesce(const char *ifname,
- virNetDevCoalescePtr coalesce ATTRIBUTE_UNUSED)
+ virNetDevCoalescePtr coalesce)
{
+ if (!coalesce)
+ return 0;
+
virReportSystemError(ENOSYS,
_("Cannot set coalesce info on interface '%s'"),
ifname);
--
2.11.0
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
