[libvirt] [PATCH 3/5] nodedev: Fix sysfs paths for vport operations

Eric Blake eblake at redhat.com
Tue Aug 17 21:27:47 UTC 2010


On 08/17/2010 02:51 PM, Jiri Denemark wrote:
> +    if (stat(operation_path, &st) != 0) {
> +        VIR_FREE(operation_path);
> +        if (virAsprintf(&operation_path,
> +                        "%shost%d%s",
> +                        LINUX_SYSFS_SCSI_HOST_PREFIX,
> +                        parent_host,
> +                        operation_file) < 0) {

It's slightly more efficient to write:

virAsprintf(&operation_path, LINUX_SYSFS_SCSI_HOST_PREFIX "host%d%s",
            parent_host, operation_file)

but it doesn't bother me if you think it is easier to leave it as-written.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100817/9393c321/attachment-0001.sig>


More information about the libvir-list mailing list