linux-kernel - Re: [PATCH 2/14] bfa: Brocade BFA FC SCSI driver (bfa part1)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <200908211935.46316.eike-kernel@sf-tec.de>
Date:	Fri, 21 Aug 2009 19:35:44 +0200
From:	Rolf Eike Beer <eike-kernel@...tec.de>
To:	Jing Huang <huangj@...cade.com>
Cc:	James.Bottomley@...senpartnership.com,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	rvadivel@...cade.com, vravindr@...cade.com
Subject: Re: [PATCH 2/14] bfa: Brocade BFA FC SCSI driver (bfa part1)

Jing Huang wrote:
> From: Jing Huang <huangj@...cade.com>
>
> This patch contains code to access Brocade Fibre channel HBA
> firmware/hardware, part-1
>
> +/*
> + * Copyright (c)  2005-2008 Brocade Communications Systems, Inc.
> + * All rights reserved
> + * www.brocade.com

Shouldn't that be 2009?

> +char *
> +wwn2str(char *buf, int bufsize, u64 wwn)
> +{
> +	char *ptr;
> +	union {
> +	wwn_t           wwn;
> +	u8         byte[sizeof(wwn_t)];
> +	} w;
> +	int i = sizeof(wwn_t);
> +
> +	ptr = buf + bufsize;
> +	*--ptr = '\0';
> +	bufsize--;
> +
> +	w.wwn = wwn;
> +	while (i > 0) {
> +		i--;
> +		*--ptr = n2b(w.byte[i] & 0x0f);
> +		*--ptr = n2b((w.byte[i] >> 4) & 0x0f);
> +		if (i > 0)
> +			*--ptr = ':';
> +	}
> +
> +	return ptr;
> +}

I searched for 20 minutes and found nothing. Is there really no standard 
function to printf a WWN? I mean, every FC driver needs this, no?

fnic:
-"%llx", wwn

ipr:
-"%08X%08X", be32_to_cpu(wwid[0]), be32_to_cpu(wwid[1]) 

lpfc:
-"%02x%02x%02x%02x%02x%02x%02x%02x", wwn[0], wwn[1], ...
-"%02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x", wwn[0], wwn[1], ...
-"0x%llx", cfg_soft_wwnn

qla2xxx:
-like lpfc, version 1. Harder to find as there it is called port_name and 
node_name instead of wwpn or wwnn or wwn

Is there really nothing like this? Should be an easy job to write one and 
convert all existing drivers over to use this, no? Would also add the benefit 
as that all drivers would print that in the same way. 

Ei-printk("%WWN", wwn)-ke

Download attachment "signature.asc " of type "application/pgp-signature" (199 bytes)