Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754604Ab1ENItx (ORCPT ); Sat, 14 May 2011 04:49:53 -0400 Received: from verein.lst.de ([213.95.11.211]:43675 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754043Ab1ENItu (ORCPT ); Sat, 14 May 2011 04:49:50 -0400 Date: Sat, 14 May 2011 10:49:49 +0200 From: Christoph Hellwig To: Dan Williams Cc: James.Bottomley@hansenpartnership.com, Christoph Hellwig , linux-kernel , linux-scsi , Linus Torvalds , Dave Jiang , David Milburn , Ed Ciechanowski , Ed Nadolski , Jacek Danecki , Jeff Skirvin , Jeff Garzik Subject: Re: [GIT PULL] isci merge candidate Message-ID: <20110514084949.GA23984@lst.de> References: <1305317680.21099.83.camel@dwillia2-linux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1305317680.21099.83.camel@dwillia2-linux> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1657 Lines: 31 On Fri, May 13, 2011 at 01:14:40PM -0700, Dan Williams wrote: > The isci driver team has now completed the major rework items addressed > in the review on linux-scsi (including removal of state handlers, > merging lldd and 'core', cleaning up the source code layout). I've looked over the driver a bit and I'm quite impressed with what you're archived in the short time since taking over the driver from whoever came up with the mess that it was initially. I don't think you're quite done yet with the todo list that was given to you yet. One thing that springs to mind is wrappers in timers.c, which are not just ugly, but in case of isci_task_execute_tmf is plain wrong as the implementation assumes all timers have the same lifetime rules as the isci_host. You'll need to at least replace that last usage with a direct wait_for_completion_timeout, and even better get rid of it entirely. Also not quite done yet, although I'm happy with postponing that for now is the unification of the various data structures from the different layers of the original driver, e.g. isci_phy vs scic_sds_phy, isci_port vs scic_sds_port, isci_remote_device vs scic_sds_remote_device and isci_request vs scic_sds_request. And of course there's a lot of room for additional further cleanups that should be able to shave off another couple thousands of lines, but these never were on the plate for the initial merge anyway. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/