Date: Sat, 17 May 2003 15:21:29 +0200
From: Ingo Oeser <ingo.oeser@informatik.tu-chemnitz.de>
To: Greg KH <greg@kroah.com>
Cc: Manuel Estrada Sainz <ranty@debian.org>,
       LKML <linux-kernel@vger.kernel.org>,
       Simon Kelley <simon@thekelleys.org.uk>,
       Alan Cox <alan@lxorguk.ukuu.org.uk>,
       "Downing, Thomas" <Thomas.Downing@ipc.com>, jt@hpl.hp.com,
       Pavel Roskin <proski@gnu.org>
Subject: Re: request_firmware() hotplug interface, third round.
Message-ID: <20030517152129.F626@nightmaster.csn.tu-chemnitz.de>
References: <20030515200324.GB12949@ranty.ddts.net> <20030516223624.GA16759@kroah.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2i
In-Reply-To: <20030516223624.GA16759@kroah.com>; from greg@kroah.com on Fri, May 16, 2003 at 03:36:24PM -0700
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 834
Lines: 20

On Fri, May 16, 2003 at 03:36:24PM -0700, Greg KH wrote:
> Other than those very minor tweaks, I like this interface, it's looking
> very good.  I wouldn't worry about any "checksum" calcuation crud, it's
> up to the userspace tool dumping the firmware to the kernel to make sure

Ok, if that is true, then we could also have this tool enforce a
size. Otherwise we are reading and reading without ever ending
and allocating a lot of kernel resources while we are at it.

If we know the size, then we also now start and end. So the
"loading" attribute can certainly go.

Regards

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