linux-kernel - Re: [PATCH v2 0/2] scsi: Port buslogic driver to 64 bits

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]

Message-ID: <1371222427.8771.1.camel@concerto>
Date:	Fri, 14 Jun 2013 09:07:07 -0600
From:	Khalid Aziz <khalid.aziz@...cle.com>
To:	James Bottomley <James.Bottomley@...senPartnership.com>
Cc:	linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] scsi: Port buslogic driver to 64 bits

On Thu, 2013-05-16 at 19:44 -0600, Khalid Aziz wrote:
> This patchset ports buslogic driver to 64-bit.
> 
> Current buslogic driver is composed of two components - SCCB manager
> which communicates with adapter to execute SCSI commands (contained in
> FlashPoint.c), and Linux driver part that interfaces with rest of the
> kernel (contained in BusLogic.c). SCCB manager code came from the
> Mylex SDK. SCCB manager code is used by flashpoint adapters only.
> Multimaster adapters do not need SCCB manager since the SCSI state
> machine is implemented in firmware on these adapters. If the filenames
> alone did not give it away already, buslogic driver code is full of
> CamelCase, besides being full of very ling lines, and is just very
> difficult to read and understand. So the first step was to clean up
> the existing code. First patch in the set does just that. Second patch
> includes necessary code modifications to allow the driver to build and
> run on 64-bit kernel. Since SCCB manager code came from Mylex SDK, I
> have tried to touch it only when necessary which includes not fixing
> all CamelCase issues in FlashPoint.c. Many lines over 80 characters
> remain in BusLogic.c. These fall into two categories generally -
> (1) it prints a message and I didn't want to touch driver messages
> in case there are scripts out there that parse driver messages,
> (2) code is indented deeply and is hard to keep it under 80 characters.
> Such code could use refactoring at some point.
> 
> I have tested this patch with a flashpoint adapter on 64-bit and 
> 32-bit kernels with fio running random read/write test while verifying
> data. I also measured performance for current buslogic driver and 
> buslogic driver with these patches with 32-bit and 64-bit kernel and
> ensured there was no degradation in performance.
> 
> Changelog:
> v2:
> 	- Updated to apply on top of current Linus' tree as of May 16, 2013. 
> 	  No functional changes.
> 
> Khalid Aziz (2):
>   Fix CamelCase and extra long lines in the buslogic driver.
>   Port buslogic driver to 64-bit.
> 
>  drivers/scsi/BusLogic.c   | 4452 ++++++++++++++++++++++++---------------------
>  drivers/scsi/BusLogic.h   | 1487 ++++++++-------
>  drivers/scsi/FlashPoint.c |  626 +++----
>  drivers/scsi/Kconfig      |    2 +-
>  4 files changed, 3377 insertions(+), 3190 deletions(-)
> 

Hello James,

Any feedback on this set of patches?

Thanks,
Khalid


--
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/