<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 Thu, Sep 12, 2019 at 5:41 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@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!<br>
> Thanks for the quick work on this.<br>
> <br>
> There are two problems that you probably ignored.<br>
> <br>
> 1. On ppc64 system, the page size is 64KB, and 970 * 64KB is almost 60MB<br>
> which we do allocate. So we'd better raise<br>
> the limit_in_bytes/memsw.limit_in_bytes a little in cgroup correspondingly.<br>
> <br>
> 2. As Cascardo points out, madvise09 gets retries in an infinite loop when<br>
> hitting OOM. That's because, in the run() function, it does not limit the<br>
> retry times. We need to take care of this as well.<br>
<br>
I guess that the debug printing would go out of hand as well, 970<br>
characters printed into the console is no fun either. The test needs a<br>
bit more than just bumping the number of pages. However as we are<br>
getting really close to release I guess that the best course of action<br>
would be workarounding the issue for now by bumping the number of pages<br>
to something as 128 pages. Then we can rework the test once LTP has been<br>
released.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">OK for me.</div></div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>