<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 21, 2020 at 8:16 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Li, Cyril,<br>
<br>
> > +   if (FILE_LINES_SCANF("/proc/meminfo",<br>
> > +                           "MemAvailable: %ld", &mem_avail)) {<br>
> > +           /*<br>
> > +            * Using "MemFree:" on kernel that doesn't have<br>
> > +            * "MemAvailable:" in Meminfo<br>
> > +            */<br>
> > +           tst_res(TINFO, "MemAvailable: not found in /proc/meminfo");<br>
> > +<br>
> > +           mem_avail = SAFE_READ_MEMINFO("MemFree:");<br>
> > +   }<br>
<br>
> I guess that we don't have to bother with MemAvailable here, since we do<br>
> drop the system caches the MemFree should be equal to MemAvailable. So I<br>
> would be for reading just MemFree unconditionally here.<br>
<br>
> If you agree with this change I can change it before applying.<br>
<br>
Merged both patches with this change and my docs change.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Thanks very much Petr. Btw, I added a commit to remove the surplus variable 'mem_avail'.(It's a tiny issue so I pushed directly:)</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>