On Aug 21 16:53, Livio Bertacco wrote: > Thanks Corinna, I tried the developer snapshot and the VmSize line from > "status" file looks good now. > > What do you think about adding the VmPeak line too? (I can create the patch > myself). > > > On second thought there's more wrong than just that. > > Just dividing by page_size or allocation_granularity results in too small > values. > > Yes, now that you mention it, this rounding up is actually still missing > for the vmrss and vmmaxrss values in the "stat" file (which just divides by > page_size): > 1150 > <https://cygwin.com/git/gitweb.cgi?p=newlib-cygwin.git;a=blob;f=winsup/cygwin/fhandler_process.cc#l1150> > vmrss = vmc.WorkingSetSize / page_size; > 1151 > <https://cygwin.com/git/gitweb.cgi?p=newlib-cygwin.git;a=blob;f=winsup/cygwin/fhandler_process.cc#l1151> > vmmaxrss = ql.MaximumWorkingSetSize / page_size; > and, by the way, is it correct that these two are reported in 4k pages > instead of 64k pages? > "statm" reports e.g. VmRss (that's the second value in the statm line) in > 64k pages so that value doesn't match the same Rss amount (24th value) from > "stat".
I'd actually be happy to get patches here. Just have a look at https://cygwin.com/contrib.html for the 2-clause BSD statement. Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat
signature.asc
Description: PGP signature
