Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754689AbZD2KbD (ORCPT ); Wed, 29 Apr 2009 06:31:03 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751748AbZD2Kaw (ORCPT ); Wed, 29 Apr 2009 06:30:52 -0400 Received: from mail-ew0-f176.google.com ([209.85.219.176]:53055 "EHLO mail-ew0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751009AbZD2Kav (ORCPT ); Wed, 29 Apr 2009 06:30:51 -0400 MIME-Version: 1.0 In-Reply-To: <437908170904282019h6a10ef9dg594002e22842b366@mail.gmail.com> References: <20090427132041.GA18391@sc1430.michaels-house.net> <20090427172710.5a0c7f48@hyperion.delvare> <437908170904271023k3eefd4c1ma0cacdb78ad7acd7@mail.gmail.com> <20090428101020.0e4c2b0c@hyperion.delvare> <20090428173929.GA5006@sysman-doug.us.dell.com> <20090428215844.GA1396@sysman-doug.us.dell.com> <437908170904282019h6a10ef9dg594002e22842b366@mail.gmail.com> From: Kay Sievers Date: Wed, 29 Apr 2009 12:30:30 +0200 Message-ID: Subject: Re: Class device namespaces To: Michael Brown Cc: Doug Warzecha , Jean Delvare , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Mauro Carvalho Chehab , Matt Domsch Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3267 Lines: 72 On Wed, Apr 29, 2009 at 05:19, Michael Brown wrote: > On Tue, Apr 28, 2009 at 8:42 PM, Kay Sievers wrote: >> On Tue, Apr 28, 2009 at 23:58, Doug Warzecha wrote: >>> On Tue, Apr 28, 2009 at 07:46:57PM +0200, Kay Sievers wrote: >>>> On Tue, Apr 28, 2009 at 19:39, Doug Warzecha wrote: >>>> >>>> > /sys/class/firmware/dell_rbu/loading >>>> > /sys/class/firmware/dell_rbu/data >>>> >>>> These pathes only ever exist for a fraction of a second, during a >>>> kernel initiated firmware request. This request is usually only >>>> handled by the udev firmware loader script, and not interesting for >>>> anything else. Are you replacing the udev firmware script here? Are >>>> you sure you do something else here? >>> >>> A Dell library writes data to those paths when a Dell BIOS Update Package is run and requests the BIOS image to be loaded into memory. >> >> How do you do this? These paths exist for a short time only. How do >> you get udev out of your way handling the firmware request for you? >> How do you plug into the event processing with a custom tool? > > Overall, the interface is pretty pathetic because it doesnt fit our > use case *at all*, but way back in the day that is what we had to do > to get it reviewed and approved for merge upstream. At least it is > better than the earlier versions where we had to do a separate > request_firmware() for every 4k packet (and spin waiting for files to > disappear/reappear) It handles huge files just fine, since the beginning. The space is realloced in the kernel. I have a modem which loads 200kb files since years. > All this code is in libsmbios(*) if you want to see the ugly details, > but the short version is this: > > 1) we poke the image_type file and packet_size files with appropriate > data, which causes the driver to do a request firmware > 2) spin until request_firmware creates the files we need This is totally insane! > 2) loop over our file packet_size bytes at a time writing data into > the data file > 3) set loading to 0 when done > > *) src/python/libsmbios_c/rbu_update.py > > The code basically "packetizes" the bios update into page-sized chunks > with a header so BIOS can find them and reassemble them for the final > update. > > For older systems (monolithic mode), the kernel code has to allocate > physically contiguous region to hold the entire firmware image. Yeah, maybe you can not use the in-kernel firmware-request filled memory. But you can copy it just fine, once it is in the kernel. There is no reason to fiddle around with chunks here in userspace. How do you make sure udev does not handle the request at the same time? This interface is not public at all, on a general system. As long as udev is active you can not use it at all. Udev will run at the same time and write to the same files, and even cancel the request if the requested file is not found. How does your software cope with that? Thanks, Kay -- 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/