Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-28 Thread Greg KH
On Wed, Oct 28, 2020 at 05:00:11PM +0100, Greg KH wrote: > On Tue, Oct 27, 2020 at 04:32:13PM +, Christoph Hellwig wrote: > > On Tue, Oct 27, 2020 at 10:17:25AM +0100, Greg KH wrote: > > > Ok, I couldn't get a clean merge of that old branch on top of your > > > 5.10-rc1 tree, so I can't give it

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-28 Thread Christoph Hellwig
On Wed, Oct 28, 2020 at 07:33:59PM +0100, Greg KH wrote: > That worked too, I'll send a patch for this for the top of your branch > to resolve this issue as a response to this email. In the old branch I had a scripted conversion of all proc_read instances that point to seq_read to use seq_read_ite

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-28 Thread Greg KH
On Tue, Oct 27, 2020 at 04:32:13PM +, Christoph Hellwig wrote: > On Tue, Oct 27, 2020 at 10:17:25AM +0100, Greg KH wrote: > > Ok, I couldn't get a clean merge of that old branch on top of your > > 5.10-rc1 tree, so I can't give it a run-through. If you have an updated > > series you want me to

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Greg KH
On Tue, Oct 27, 2020 at 04:32:13PM +, Christoph Hellwig wrote: > On Tue, Oct 27, 2020 at 10:17:25AM +0100, Greg KH wrote: > > Ok, I couldn't get a clean merge of that old branch on top of your > > 5.10-rc1 tree, so I can't give it a run-through. If you have an updated > > series you want me to

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Christoph Hellwig
On Tue, Oct 27, 2020 at 10:17:25AM +0100, Greg KH wrote: > Ok, I couldn't get a clean merge of that old branch on top of your > 5.10-rc1 tree, so I can't give it a run-through. If you have an updated > series you want me to test, I'll be glad to do so. Can you give this branch a spin? http://git

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Greg KH
On Tue, Oct 27, 2020 at 09:14:20AM +0100, Greg KH wrote: > On Tue, Oct 27, 2020 at 08:07:45AM +, Christoph Hellwig wrote: > > On Tue, Oct 27, 2020 at 08:55:41AM +0100, Greg KH wrote: > > > This is just a test, part of the bionic test suite to verify that bionic > > > is working properly, and is

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Christoph Hellwig
On Tue, Oct 27, 2020 at 09:14:20AM +0100, Greg KH wrote: > Luckily /proc/cpuinfo seems to use the seq_file interface, so this > series would work for that. > > What's the odds of this series getting into 5.10-final? I'll go run it > through the Android build system right now to see if it fixes th

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Greg KH
On Tue, Oct 27, 2020 at 08:07:45AM +, Christoph Hellwig wrote: > On Tue, Oct 27, 2020 at 08:55:41AM +0100, Greg KH wrote: > > This is just a test, part of the bionic test suite to verify that bionic > > is working properly, and is run on new kernels as a verification that > > nothing functional

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Christoph Hellwig
On Tue, Oct 27, 2020 at 08:55:41AM +0100, Greg KH wrote: > This is just a test, part of the bionic test suite to verify that bionic > is working properly, and is run on new kernels as a verification that > nothing functional broke in the kernel update. > > I don't know about "real applications" ye

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Greg KH
On Tue, Oct 27, 2020 at 07:49:11AM +, Christoph Hellwig wrote: > On Tue, Oct 27, 2020 at 07:48:32AM +0100, Greg KH wrote: > > On Sun, Oct 25, 2020 at 03:40:27PM -0700, Linus Torvalds wrote: > > > The most interesting - to me - change here is Christoph's setf_fs() > > > removal (it got merged th

Re: problems with splice from /proc (was Linux 5.10-rc1)

2020-10-27 Thread Christoph Hellwig
On Tue, Oct 27, 2020 at 07:48:32AM +0100, Greg KH wrote: > On Sun, Oct 25, 2020 at 03:40:27PM -0700, Linus Torvalds wrote: > > The most interesting - to me - change here is Christoph's setf_fs() > > removal (it got merged through Al Viro, as you can see in my mergelog > > below). It's not a _huge_