From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753858AbdCWAgS (ORCPT ); Wed, 22 Mar 2017 20:36:18 -0400 Received: from mail-qt0-f195.google.com ([209.85.216.195]:33989 "EHLO mail-qt0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752004AbdCWAgJ (ORCPT ); Wed, 22 Mar 2017 20:36:09 -0400 MIME-Version: 1.0 In-Reply-To: <130097e7-fe9b-dce5-8bda-f22f352f7a44@linux.intel.com> References: <1489105090-4996-1-git-send-email-yi1.li@linux.intel.com> <1489105090-4996-2-git-send-email-yi1.li@linux.intel.com> <130097e7-fe9b-dce5-8bda-f22f352f7a44@linux.intel.com> From: Alan Tull Date: Wed, 22 Mar 2017 19:34:41 -0500 Message-ID: Subject: Re: [RFC 1/2] firmware class: Add stream_firmware API. To: "Li, Yi" Cc: Greg Kroah-Hartman , ming.lei@canonical.com, mcgrof@kernel.org, atull , Moritz Fischer , linux-kernel , linux-fpga@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 22, 2017 at 5:05 PM, Li, Yi wrote: > Alan > > > On 3/20/2017 1:34 PM, Alan Tull wrote: >> >> On Mon, Mar 20, 2017 at 1:00 PM, Alan Tull >> wrote: >> >>>> +int >>>> +stream_firmware(const struct firmware **firmware_p, const char *name, >>>> + struct device *device, size_t offset, size_t length) >>>> +{ >>>> + size_t ret; >>>> + >>>> + /* Need to pin this module until return */ >>>> + __module_get(THIS_MODULE); >>>> + ret = _stream_firmware(firmware_p, name, device, NULL, 0, >>>> + FW_OPT_UEVENT | FW_OPT_NO_WARN, offset, >>>> length); >> >> IIUC, here you are setting size == 0 and buf == NULL to prevent >> _request_firmware_prepare from attempting to load from built in >> firmware. >> >> So three of the parameters buf, size, and opt_flags are fixed and >> don't need to be passed to _stream_firmware(). > > > Sure. > >> Alternatively, I wonder how hard it would be to code this so that the >> streaming interface will fall back and successfully get the built in >> or cached firmware if it exists and stream it out in PAGE_SIZE chunks. > > > That's an interesting idea, I will try it out and submit patch for review > later. On another hand, if the kernel already cache the whole firmware > image, why should we use streaming instead of regular request_firmware? The caller doesn't know whether the firmware is cached or not. The caller just wants the firmware if it exists, wherever it is. If that can be made automatic then the caller doesn't have to first attempt stream_firmware and then if that fails fall back to calling request_firmware. > >> >> Alan Tull >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-fpga" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > >