On Wed, 15 Sep 2010, Matt Mackall wrote: > On Wed, 2010-09-15 at 16:14 +0200, Richard Guenther wrote: > > On Wed, 15 Sep 2010, Balbir Singh wrote: > > > > > * Nikanth Karthikesan <knikanth@xxxxxxx> [2010-09-15 12:01:11]: > > > > > > > How? Current smaps information without this patch provides incorrect > > > > information. Just because a private dirty page became part of swap cache, it > > > > shown as clean and backed by a file. If it is shown as clean and backed by > > > > swap then it is fine. > > > > > > > > > > How is GDB using this information? > > > > GDB counts the number of dirty and swapped pages in a private mapping and > > based on that decides whether it needs to dump it to a core file or not. > > If there are no dirty or swapped pages gdb assumes it can reconstruct > > the mapping from the original backing file. This way for example > > shared libraries do not end up in the core file. > > This whole discussion is a little disturbing. > > The page is being reported clean as per the kernel's definition of > clean, full stop. > > So either there's a latent bug/inconsistency in the kernel VM or > external tools are misinterpreting this data. But smaps is just > reporting what's there, the fault doesn't lie in smaps. So fixing smaps > just hides the problem, wherever it is. > > Richard's report that the page is still clean after swapoff suggests the > inconsistency lies in the VM. Well - the discussion is about the /proc/smaps interface and inconsistencies in what it reports. In particular the interface does not have the capability of reporting all details the kernel has, so it might make sense to not "report a page clean as per the kernel's definition of clean", but only in a /proc/smaps context definition of clean that makes sense. So, for 7ffff81ff000-7ffff8201000 r--p 000a8000 08:01 16376 /bin/bash Size: 8 kB Rss: 8 kB Pss: 8 kB Shared_Clean: 0 kB Shared_Dirty: 0 kB Private_Clean: 8 kB Private_Dirty: 0 kB Referenced: 4 kB Swap: 0 kB I expect both pages of that mapping to be file-backed by /bin/bash. But surprisingly one page is actually backed by anonymous memory (it was changed, then mapped readonly, swapped out and swapped in again). Thus, the bug is the above inconsistency in /proc/smaps. Whether there are internal kernel inconsistencies as well doesn't really matter to this problem (as there is no way to distinguish pages that are now backed by anonymous memory in that interface). Richard. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>