Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964851AbVIKJq7 (ORCPT ); Sun, 11 Sep 2005 05:46:59 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S964850AbVIKJq7 (ORCPT ); Sun, 11 Sep 2005 05:46:59 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:53393 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S964841AbVIKJq6 (ORCPT ); Sun, 11 Sep 2005 05:46:58 -0400 Date: Sun, 11 Sep 2005 10:46:56 +0100 From: Christoph Hellwig To: Luben Tuikov Cc: James Bottomley , Luben Tuikov , Linux Kernel Mailing List , SCSI Mailing List Subject: Re: [PATCH 2.6.13 5/14] sas-class: sas_discover.c Discover process (end devices) Message-ID: <20050911094656.GC5429@infradead.org> Mail-Followup-To: Christoph Hellwig , Luben Tuikov , James Bottomley , Luben Tuikov , Linux Kernel Mailing List , SCSI Mailing List References: <1126308304.4799.45.camel@mulgrave> <20050910024454.20602.qmail@web51613.mail.yahoo.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050910024454.20602.qmail@web51613.mail.yahoo.com> User-Agent: Mutt/1.4.2.1i X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1661 Lines: 38 On Fri, Sep 09, 2005 at 07:44:54PM -0700, Luben Tuikov wrote: > > this one completely duplicates the > > mid-layer infrastructure for handling devices with Logical Units. > > No, it does *not*. James, you have _stop_ spreading FUD, relying > that other people have not read the SCSI Core code. > > See here: > SCSI Core has *no representation* of a SCSI Device with a > SCSI Target Port. struct scsi_target > I've _clearly_ outlined that in the comments of the code, > which you _conveniently_ did _not_ cut and paste here. * Discover logical units present in the SCSI device. I'd like this * to be moved to SCSI Core, but SCSI Core has no concept of a "SCSI * device with a SCSI Target port". A SCSI device with a SCSI Target * port is a device which the _transport_ found, but other than that, * the transport has little or _no_ knowledge about the device. * Ideally, a LLDD would register a "SCSI device with a SCSI Target * port" with SCSI Core and then SCSI Core would do LU discovery of * that device. So what does this mean except "Luben tries to impress everyone with standards gibberish, at the same time ignoring we soluitions that work despite maybe not 100% elegant". Sure, we'd like to move away from needing the ->id target id specifier. But right now we need it, even you're code sets it in over-complicated ways. But if you send a nice patch to get rid everyone will be happy. - 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/