Linux Test Project
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH v3 00/15] Convert LTP mem/testcase to new API
Date: Tue, 1 Aug 2017 17:04:32 +0200	[thread overview]
Message-ID: <20170801150432.GA3925@rei> (raw)
In-Reply-To: <20170718082312.3946-1-liwang@redhat.com>

Hi!
I've pushed this patches, after careful testing and with many changes,
thanks.

Most of the changes were more or less cosmetic, such as removing -1
handling after SAFE_FORK(), make use of more SAFE_MACROS() etc.

But there were some problems as well. It introduced a compilation
failure on 32bit for min_free_kbytes.c since it removed the total_mem
variable that is used only on 32bit.

The ksm06.c cleanup had to be changed, since in a case that
merge_across_nodes is missing it attempted to run cleanup (previously
there was NULL cleanup pointer for TCONF!) and failed the test. There is
probably more unvanted cleanups possibly run for the ksm test, we should
have a look at these later.

I've also reverted the change to the thp01 in the runtest file since I
belive that the value was reduced only for testing and slipped to the
patchset by accident.

I've changed the patch that adds possibility to disable timeouts, mainly
so that the API is consistent, which means that the tst_set_timeout()
takes -1 as parameter as well + wrote documentation for the change.

Also unsolved is a problem with TCONF tests. At the moment newlib test
fails if we pass any parameters to it, since the option parsing runs
with the tst_test structure without the option stucture. This fails, for
instance, ksm test if they were compiled out since we pass parameters to
them in the runtest file. I will have to think about this a bit more and
figure out a good solution.

-- 
Cyril Hrubis
chrubis@suse.cz

  parent reply	other threads:[~2017-08-01 15:04 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-18  8:22 [LTP] [PATCH v3 00/15] Convert LTP mem/testcase to new API Li Wang
2017-07-18  8:22 ` [LTP] [PATCH v3 01/15] mem/lib: convert " Li Wang
2017-07-18  8:22 ` [LTP] [PATCH v3 02/15] mem/oom: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 03/15] mem/oom: disable timeouts for testcase in demand Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 04/15] mem/ksm: convert to new API Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 05/15] mm/ksm: move the common code to ksm_common.h Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 06/15] mem/thp: convert to new API Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 07/15] mem/hugetlb: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 08/15] mem/hugemmap: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 09/15] mem/hugeshmat: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 10/15] mem/hugeshmctl: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 11/15] mm/hugeshmdt: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 12/15] mem/hugeshmget: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 13/15] mem/cpuset: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 14/15] mem/swapping: " Li Wang
2017-07-18  8:23 ` [LTP] [PATCH v3 15/15] mem/tunable: " Li Wang
2017-08-01 15:04 ` Cyril Hrubis [this message]
2017-08-02  4:00   ` [LTP] [PATCH v3 00/15] Convert LTP mem/testcase " Li Wang

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=20170801150432.GA3925@rei \
    --to=chrubis@suse.cz \
    --cc=ltp@lists.linux.it \
    /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