util-linux.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Adam Sampson <ats@offog.org>
To: Karel Zak <kzak@redhat.com>
Cc: util-linux@vger.kernel.org
Subject: Re: [ANNOUNCE] util-linux v2.25.1
Date: Wed, 3 Sep 2014 16:55:54 +0100	[thread overview]
Message-ID: <20140903155554.GS885@cartman.at.offog.org> (raw)
In-Reply-To: <20140903105427.GH21325@x2.net.home>

[-- Attachment #1: Type: text/plain, Size: 329 bytes --]

On Wed, Sep 03, 2014 at 12:54:27PM +0200, Karel Zak wrote:
>    - make ipcs limit calculations work when numbers are large

That change isn't right -- it reverses the sense of the test, making the
testcase fail on 3.16 again. Patch attached.

Thanks,

-- 
Adam Sampson <ats@offog.org>                         <http://offog.org/>

[-- Attachment #2: 0001-Fix-the-test-for-shmall-being-too-big-to-show.patch --]
[-- Type: text/plain, Size: 929 bytes --]

>From 1269b6bb74727043c16852bc34727a0c8d0696c7 Mon Sep 17 00:00:00 2001
From: Adam Sampson <ats@offog.org>
Date: Wed, 3 Sep 2014 16:49:40 +0100
Subject: [PATCH] Fix the test for shmall being too big to show.

It was correct when originally introduced, but the shell overflow fix in
3a9ec12d6664527fad9c56347c88f3447d6c0856 accidentally reversed the sense
of the test.

Signed-off-by: Adam Sampson <ats@offog.org>
---
 tests/ts/ipcs/limits2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/ts/ipcs/limits2 b/tests/ts/ipcs/limits2
index 1a49c46..19f49fb 100755
--- a/tests/ts/ipcs/limits2
+++ b/tests/ts/ipcs/limits2
@@ -29,7 +29,7 @@ ts_check_prog "bc"
 
 # TODO https://github.com/karelzak/util-linux/issues/51
 SHMALL=$(</proc/sys/kernel/shmall)
-if [ $(bc <<<"2^64 / $PAGE_SIZE < $SHMALL") -ne 1 ]; then
+if [ $(bc <<<"2^64 / $PAGE_SIZE < $SHMALL") -eq 1 ]; then
 	TS_KNOWN_FAIL="yes"
 fi
 
-- 
2.0.4


  reply	other threads:[~2014-09-03 15:55 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-03 10:54 [ANNOUNCE] util-linux v2.25.1 Karel Zak
2014-09-03 15:55 ` Adam Sampson [this message]
2014-09-03 22:19   ` Ruediger Meier
  -- strict thread matches above, loose matches on Subject: below --
2014-09-06  7:27 Sami Kerola
2014-09-06  8:51 ` Ruediger Meier
2014-10-01  8:14   ` Karel Zak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140903155554.GS885@cartman.at.offog.org \
    --to=ats@offog.org \
    --cc=kzak@redhat.com \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).