[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH]Fix vPort Manage of FC vHBA creation



On 06/28/2013 03:22 AM, Dennis Chen wrote:
> When create a virtual FC HBA with virsh/libvirt API, an error message will be
> returned:"error: Node device not found",
> also the 'nodedev-dumpxml' shows wrong information of wwpn & wwnn for the new
> created device.
> 
> Signed-off-by:xschen tnsoft com cn
> ---
> src/util/virutil.c |    4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/util/virutil.c b/src/util/virutil.c
> index 6fa0212e..569d035 100644
> --- a/src/util/virutil.c
> +++ b/src/util/virutil.c
> @@ -1792,8 +1792,8 @@ virManageVport(const int parent_host,
>  
>      if (virAsprintf(&vport_name,
>                      "%s:%s",
> -                    wwnn,
> -                    wwpn) < 0) {
> +                    wwpn,
> +                    wwnn) < 0) {
>          virReportOOMError();
>          goto cleanup;
>      }
> 

Hmm, this is what we've had before commit f90af69 [1]
but according to scsi_fc_transport.txt [2] in kernel docs, it should be
<WWPN>:<WWNN>. I wonder what that commit was trying to fix.

Jan

[1] http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=f90af69
[2] https://www.kernel.org/doc/Documentation/scsi/scsi_fc_transport.txt


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]