On Tue, Jan 12, 2021 at 10:31:26AM -0800, Siddharth Gupta wrote:If my understanding is correct these default (or generic if you prefer)
On 1/8/2021 6:44 AM, Greg KH wrote:Thanks for letting me know. I'll try to work on this on Monday and add
On Thu, Jan 07, 2021 at 02:03:47PM -0800, Siddharth Gupta wrote:Sorry for the delay, I tried out the patch, but I am still seeing the error.
On 1/6/2021 2:33 AM, Greg KH wrote:Can you try this tiny patch out below?
I tried to follow these fixes, but I am unfamiliar with fs code. I don't seeLook at the series of commits starting at fe33850ff798 ("proc: wire upI am not aware of this fix, could you provide me a link for reference? ISince the binary attributes don't support splice_{read,write} functions theHave you tried fixing this with a patch much like what we did for the
calls to splice_{read,write} used the default kernel_{read,write} functions.
With the above change this results in an -EINVAL return from
do_splice_from[4].
This essentially means that sendfile will not work for any binary attribute
in the sysfs.
proc files that needed this? If not, can you?
will try it out.
generic_file_splice_read for iter ops") for how this was fixed in procfs
as an example of what also needs to be done for binary sysfs files.
the generic_file_splice_write function anymore on newer kernels, also AFAICT
kernfs_ops does not define {read,write}_iter operations. If the solution is
simple and someone could provide the patches I would be happy to test them
out. If not, some more information about how to proceed would be nice.
Please take a look at these logs with
android running in the userspace[1]:
[ 62.295056][ T249] remoteproc remoteproc1: powering up
xxxxxxxx.remoteproc-cdsp
[ 62.304138][ T249] remoteproc remoteproc1: Direct firmware load for
cdsp.mdt failed with error -2
[ 62.312976][ T249] remoteproc remoteproc1: Falling back to sysfs
fallback for: cdsp.mdt
[ 62.469748][ T394] ueventd: firmware: loading 'cdsp.mdt' for '/devices/platform/soc/xxxxxxxx.remoteproc-cdsp/remoteproc/remoteproc1/cdsp.mdt'
[ 62.498700][ T394] ueventd: firmware: sendfile failed { '/sys/devices/platform/soc/xxxxxxxx.remoteproc-cdsp/remoteproc/remoteproc1/cdsp.mdt',
'cdsp.mdt' }: Invalid argument
Thanks,
Sid
[1]: https://android.googlesource.com/platform/system/core/+/refs/heads/master/init/firmware_handler.cpp#57
splice to the in-kernel firmware testing suite, as it would have caught
this...
greg k-h