Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262749AbVDHIAa (ORCPT ); Fri, 8 Apr 2005 04:00:30 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262756AbVDHH7x (ORCPT ); Fri, 8 Apr 2005 03:59:53 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:56245 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S262748AbVDHH4u (ORCPT ); Fri, 8 Apr 2005 03:56:50 -0400 Date: Fri, 8 Apr 2005 08:56:43 +0100 From: Christoph Hellwig To: Jeremy Higdon Cc: Christoph Hellwig , Hannes Reinecke , Linux Kernel , SCSI Mailing List Subject: Re: [PATCH] Use proper seq_file api for /proc/scsi/scsi Message-ID: <20050408075643.GA5514@infradead.org> Mail-Followup-To: Christoph Hellwig , Jeremy Higdon , Hannes Reinecke , Linux Kernel , SCSI Mailing List References: <42550173.1040503@suse.de> <20050407103123.GB9586@infradead.org> <425517B3.2010702@suse.de> <20050407112412.GA12072@infradead.org> <20050408072345.GA1018765@sgi.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050408072345.GA1018765@sgi.com> User-Agent: Mutt/1.4.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: 1043 Lines: 23 On Fri, Apr 08, 2005 at 12:23:46AM -0700, Jeremy Higdon wrote: > > It works for those setups that already worked with 2.4.x, aka only a few > > luns. > > 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 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. - 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/