[PATCH] [NetRASD]: Fill up NetworkName attribute for non-bridged network

# HG changeset patch # User Eduardo Lima (Etrunko) <eblima@br.ibm.com> # Date 1308246930 10800 # Node ID 6c29c5b284443e226e7d97e7e5f11848b0c4e761 # Parent a37f80f80d4b2dab73391989168592866c23ba9b [NetRASD]: Fill up NetworkName attribute for non-bridged network This patch fixes the behavior for the read operation. Signed-off-by: Eduardo Lima (Etrunko) <eblima@br.ibm.com> diff --git a/src/Virt_RASD.c b/src/Virt_RASD.c --- a/src/Virt_RASD.c +++ b/src/Virt_RASD.c @@ -370,8 +370,7 @@ (CMPIValue *)dev->dev.net.mac, CMPI_chars); - if ((dev->dev.net.source != NULL) && - (STREQ(dev->dev.net.type, "bridge"))) + if (dev->dev.net.source != NULL) CMSetProperty(inst, "NetworkName", (CMPIValue *)dev->dev.net.source,

+1. Looking at device_parsing.c, 'net.source' is only supported with bridge and network types, so this patch really adds support for network types, not all non-bridged types as the title suggests. Pushed. On 06/16/2011 01:56 PM, Eduardo Lima (Etrunko) wrote:
# HG changeset patch # User Eduardo Lima (Etrunko)<eblima@br.ibm.com> # Date 1308246930 10800 # Node ID 6c29c5b284443e226e7d97e7e5f11848b0c4e761 # Parent a37f80f80d4b2dab73391989168592866c23ba9b [NetRASD]: Fill up NetworkName attribute for non-bridged network
This patch fixes the behavior for the read operation.
Signed-off-by: Eduardo Lima (Etrunko)<eblima@br.ibm.com>
diff --git a/src/Virt_RASD.c b/src/Virt_RASD.c --- a/src/Virt_RASD.c +++ b/src/Virt_RASD.c @@ -370,8 +370,7 @@ (CMPIValue *)dev->dev.net.mac, CMPI_chars);
- if ((dev->dev.net.source != NULL)&& - (STREQ(dev->dev.net.type, "bridge"))) + if (dev->dev.net.source != NULL) CMSetProperty(inst, "NetworkName", (CMPIValue *)dev->dev.net.source,
_______________________________________________ Libvirt-cim mailing list Libvirt-cim@redhat.com https://www.redhat.com/mailman/listinfo/libvirt-cim
-- Chip Vincent Open Virtualization IBM Linux Technology Center cvincent@linux.vnet.ibm.com
participants (2)
-
Chip Vincent
-
Eduardo Lima (Etrunko)