<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">On 01/22/2018 06:23 PM, Andrew Morton wrote: </div> <blockquote type="cite" cite="mid:20180122152315.749d88f3c91ffce4d70ac450@linux-foundation.org"> <pre wrap="">On Thu, 18 Jan 2018 11:47:48 -0500 Andrey Grodzovsky <a class="moz-txt-link-rfc2396E" href="mailto:andrey.grodzovsky@amd.com"><andrey.grodzovsky@amd.com></a> wrote: </pre> <blockquote type="cite"> <pre wrap="">Hi, this series is a revised version of an RFC sent by Christian König a few years ago. The original RFC can be found at <a class="moz-txt-link-freetext" href="https://lists.freedesktop.org/archives/dri-devel/2015-September/089778.html">https://lists.freedesktop.org/archives/dri-devel/2015-September/089778.html</a> This is the same idea and I've just adressed his concern from the original RFC and switched to a callback into file_ops instead of a new member in struct file. </pre> </blockquote> <pre wrap=""> Should be in address_space_operations, I suspect. If an application opens a file twice, we only want to count it once?</pre> </blockquote> Makes sense <blockquote type="cite" cite="mid:20180122152315.749d88f3c91ffce4d70ac450@linux-foundation.org"> <pre wrap=""> But we're putting the cart ahead of the horse here. Please provide us with a detailed description of the problem which you are addressing so that the MM developers can better consider how to address your requirements.</pre> </blockquote> I will just reiterate the problem statement from the original RFC, should have put it in the body of the RFC and not just a link, as already commented by Michal. Bellow is the quoted RFC. Thanks, Andrey P.S You can also check the follow up discussion after this first email. " <pre style="white-space: pre-wrap; color: rgb(0, 0, 0); font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial;">I'm currently working on the issue that when device drivers allocate memory on behalf of an application the OOM killer usually doesn't knew about that unless the application also get this memory mapped into their address space. This is especially annoying for graphics drivers where a lot of the VRAM usually isn't CPU accessible and so doesn't make sense to map into the address space of the process using it. The problem now is that when an application starts to use a lot of VRAM those buffers objects sooner or later get swapped out to system memory, but when we now run into an out of memory situation the OOM killer obviously doesn't knew anything about that memory and so usually kills the wrong process " </pre> <blockquote type="cite" cite="mid:20180122152315.749d88f3c91ffce4d70ac450@linux-foundation.org"> <pre wrap=""> </pre> </blockquote> </body> </html>