[LTP] [PATCH v2 1/2] syscalls/fcntl30: clean up && add more range test

Cyril Hrubis chrubis@suse.cz
Fri Feb 21 15:16:56 CET 2020


Hi!
> > Do we really need this? Looking at the kernel code the fcntl() just
> > reallocates the array that is holding the slots, so we only allocate new
> > array of struct pipe_buffer which contains a pointer for the actuall
> > page that is allocated when we _WRITE_ to the pipe.
> > 
> Yes, you are right. But this case indeed fail when on low memory machine 
> (4kb page size, 256/512M memory).

That's strange, I had a look at the code today again and as far as I can
tell we only check for user ulimit there.

What was the errno when the ioctl() has failed? Was it EPERM or ENOMEM?
The ENOMEM may have happened if the system overcommit was disabled and
the system was out of memory.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list