linux-kernel - Re: [PATCH 5/6] bfa: Brocade BFA FC SCSI Driver submission

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] [day] [month] [year] [list]

Date:	Wed, 24 Sep 2008 22:18:16 -0700
From:	James Bottomley <James.Bottomley@...senPartnership.com>
To:	Jing Huang <huangj@...cade.COM>
Cc:	linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
	Ramkumar Vadivelu <rvadivel@...cade.COM>,
	Vinodh Ravindran <vravindr@...cade.COM>,
	"Srikanth Rayas (CW)" <srayas@...cade.COM>
Subject: Re: [PATCH 5/6] bfa: Brocade BFA FC SCSI Driver submission

On Wed, 2008-09-24 at 21:09 -0700, Jing Huang wrote:
> From: Jing Huang <huangj@...cade.com>
> 
> This patch contains document of brocade specific sysfs interface. It is
> created using 2.6.27-rc7 kernel.
> 
> Signed-off-by: Jing Huang <huangj@...cade.com>
> ---
>  Documentation/ABI/testing/sysfs-devices-bfa |  600
> ++++++++++++++++++++++++++++
>  1 files changed, 600 insertions(+)
> 
> diff -urpN orig/Documentation/ABI/testing/sysfs-devices-bfa
> patch/Documentation/ABI/testing/sysfs-devices-bfa
> --- orig/Documentation/ABI/testing/sysfs-devices-bfa	1969-12-31
> 16:00:00.000000000 -0800
> +++ patch/Documentation/ABI/testing/sysfs-devices-bfa	2008-09-24
> 12:08:25.000000000 -0700
> @@ -0,0 +1,600 @@
> +What:		/sys/devices/<Brocade HBA PCI path>/bfa
> +Date:		July 2008
> +Contact:	Jing Huang <huangj@...cade.com>
> +Description:
> +		The /sys/devices/<Brocade HBA device path>/bfa will
> contain
> +		files and subdirectories to display internal states and
> +		statistics of Brocade HBA. It also provides a
> configuration and
> +		management interface for each HBA instance.

This is completely the wrong thing to do.  The driver needs to bind to
the Fibre Channel transport class which provides all of these features
in a large measure through infrastructure shareable with the other FC
drivers.  Any other pieces that are brocade specific rather than FC
general can go in host attributes in the same way as the rest of the FC
drivers do it.

James


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/