Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262757AbVDHITN (ORCPT ); Fri, 8 Apr 2005 04:19:13 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262767AbVDHIPu (ORCPT ); Fri, 8 Apr 2005 04:15:50 -0400 Received: from omx2-ext.sgi.com ([192.48.171.19]:26787 "EHLO omx2.sgi.com") by vger.kernel.org with ESMTP id S262770AbVDHILr (ORCPT ); Fri, 8 Apr 2005 04:11:47 -0400 Date: Fri, 8 Apr 2005 01:10:11 -0700 From: Jeremy Higdon To: Christoph Hellwig , Hannes Reinecke , Linux Kernel , SCSI Mailing List Subject: Re: [PATCH] Use proper seq_file api for /proc/scsi/scsi Message-ID: <20050408081011.GB1018765@sgi.com> References: <42550173.1040503@suse.de> <20050407103123.GB9586@infradead.org> <425517B3.2010702@suse.de> <20050407112412.GA12072@infradead.org> <20050408072345.GA1018765@sgi.com> <20050408075643.GA5514@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050408075643.GA5514@infradead.org> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1252 Lines: 29 On Fri, Apr 08, 2005 at 08:56:43AM +0100, Christoph Hellwig wrote: > On Fri, Apr 08, 2005 at 12:23:46AM -0700, Jeremy Higdon wrote: > > Even if it's deprecated, wouldn't it be good to fix it as long as > > it's there, unless it hurts something else? Or at least fix the > > out of memory error, even if it doesn't display all the luns? > > What other error would you return? I don't particularly care what exact Hmm. Well maybe you're right -- an error is probably better than no error if not printing all devices. > error code to return, but putting in Hannes patch would be a bad idea because > it > > a) poke deep into driver model internals, and we absolutely want to avoid > that > b) sets a bad precedence that we'll continue adding features to deprecated > interface and thus encurage people to contiue using it. Note that > /proc/scsi/* has been deprecated since mid-2.5.x. Just as a sanity check, you meant "lsscsi" and not "lssci" in your original reply, right? jeremy - 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/