<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, Jul 2, 2020 at 7:31 AM Petr Vorel <<a href="mailto:petr.vorel@gmail.com">petr.vorel@gmail.com</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">when numastat -p did not give a value, the resulting bc calculation was<br>
empty string instead of 0, thus shell -eq comparison lacked the first<br>
operator:<br>
<br>
Mem_curr=$(echo "$(extract_numastat_p $pid $node) * $MB" |bc)<br>
if [ $(echo "$Mem_curr < $MB" |bc ) -eq 1 ]; then<br>
<br>
(standard_in) 1: syntax error<br>
(standard_in) 1: syntax error<br>
/root/ltp-install/testcases/bin/numa01.sh: line 93: [: -eq: unary operator expected<br>
<br>
Also fix style (use local and lowercase local function variables).<br>
<br>
Fixes: 702<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:petr.vorel@gmail.com" target="_blank">petr.vorel@gmail.com</a>></blockquote><div><span class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></span></div><div><span class="gmail_default" style="font-size:small"></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
Hi,<br>
<br>
Simple fix for <a href="https://github.com/linux-test-project/ltp/issues/702" rel="noreferrer" target="_blank">https://github.com/linux-test-project/ltp/issues/702</a>,<br>
I guess there should be more checks. Not tested.<br>
BTW I thought numa01.sh was intended to be replaced with C (@Cyril: am I<br>
correct?), thus I didn't bother to split syntax fix into 2 commits.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I think yes, I remember the syscalls/set_mempolicy* is going to replace numa.sh tests.</div></div><div class="gmail_default" style="font-size:small"><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>