On Thu, Mar 16, 2017 at 11:18:33AM +0100, Stephan Müller wrote: > Am Donnerstag, 16. März 2017, 10:52:48 CET schrieb Herbert Xu: > > Hi Herbert, > > > First of all you're only limiting the amount of memory occupied > > by the SG list which is not the same thing as the memory pinned > > down by the actual recvmsg. > > I am fully aware of that. As this was present in the code, I thought I could > reuse that approach. > > Are you saying that you want to stop this approach?
No you're confusing things. Previously there was an explicit limit on the number of pages that can be pinned. Now you're only indirectly limiting it by limiting the size of the metadata through sock_kmalloc. The end result is that you're now allowing a huge amount of user memory to be pinned down by the system call. This is *unacceptable*. Cheers, -- Email: Herbert Xu <herb...@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt