Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1675339

Re: [PATCH] firmware: remove request_firmware_into_buf()

Show all headers | View raw

On Mon, Jun 26, 2017 at 01:22:41PM -0700, Bjorn Andersson wrote:
> On Fri, Jun 23, 2017 at 9:03 AM, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> > As Luis pointed out, there are no in-kernel users of
> > request_firmware_into_buf(), so remove it, and the now unused internal
> > flag, which simplifies the logic around buffer handling a bit.
> >
> 
> This API was implemented to reduce the memory pressure during firmware
> load in the Qualcomm remoteprocs, but it wasn't available when I
> upstreamed that code and I apparently forgot to send out the patch
> moving us over to use this API...
> 
> Especially when loading the Qualcomm modem we have a couple of files
> that we request_firmware() that are 10-15MB in size, so this
> functionality is definitely wanted.
> 
> 
> As we are calling release_firmware() immediately following the
> request_firmware() I did attempt to just call
> kernel_read_file_from_path() directly, but as I don't have access to
> the fw_path[] this becomes inconsistent, so I would like to keep
> request_firmware_into_buf().

Why would we keep it if there is no in-tree user for it?  If you want it
sometime in the future, great, we can revert the deletion then, but
keeping it around for nothing isn't ok, you know that :)

thanks,

greg k-h

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] firmware: remove request_firmware_into_buf() Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-06-23 22:30 +0200
  Re: [PATCH] firmware: remove request_firmware_into_buf() Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-06-26 22:30 +0200
    Re: [PATCH] firmware: remove request_firmware_into_buf() Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-06-27 09:00 +0200
      Re: [PATCH] firmware: remove request_firmware_into_buf() Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-06-27 19:40 +0200
    Re: [PATCH] firmware: remove request_firmware_into_buf() Alan Cox <alan@linux.intel.com> - 2017-06-27 15:00 +0200

csiph-web