DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Phil Yang (Arm Technology China)" <Phil.Yang@arm.com>
To: "Joyce Kong (Arm Technology China)" <Joyce.Kong@arm.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Cc: nd <nd@arm.com>, "thomas@monjalon.net" <thomas@monjalon.net>,
	"david.marchand@redhat.com" <david.marchand@redhat.com>,
	"stephen@networkplumber.org" <stephen@networkplumber.org>,
	"jerin.jacob@caviumnetworks.com" <jerin.jacob@caviumnetworks.com>,
	"konstantin.ananyev@intel.com" <konstantin.ananyev@intel.com>,
	Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>,
	"Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>,
	nd <nd@arm.com>
Subject: Re: [dpdk-dev] [PATCH v1] test/ticketlock: implement ticketlock autotest
Date: Mon, 15 Apr 2019 15:45:03 +0000	[thread overview]
Message-ID: <DB7PR08MB338504E54B343E7C79AD3F8FE92B0@DB7PR08MB3385.eurprd08.prod.outlook.com> (raw)
Message-ID: <20190415154503.Q9xuP24717hjHs62BNn487RA6E8wfTaZ3WTGdISwo4o@z> (raw)
In-Reply-To: <1555319079-21542-1-git-send-email-joyce.kong@arm.com>

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Joyce Kong
> Sent: Monday, April 15, 2019 5:05 PM
> To: dev@dpdk.org
> Cc: nd <nd@arm.com>; thomas@monjalon.net;
> david.marchand@redhat.com; stephen@networkplumber.org;
> jerin.jacob@caviumnetworks.com; konstantin.ananyev@intel.com;
> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Gavin Hu (Arm
> Technology China) <Gavin.Hu@arm.com>
> Subject: [dpdk-dev] [PATCH v1] test/ticketlock: implement ticketlock
> autotest
> 
> Add ticketlock_autotest implementation in python.
> 
> Fixes: efbcdaa55b93 ("test/ticketlock: add test cases")
> 
> Signed-off-by: Joyce Kong <joyce.kong@arm.com>
> ---
>  app/test/autotest_data.py       |  2 +-
>  app/test/autotest_test_funcs.py | 34
> ++++++++++++++++++++++++++++++++++
>  2 files changed, 35 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test/autotest_data.py b/app/test/autotest_data.py index
> db25274..72c56e5 100644
> --- a/app/test/autotest_data.py
> +++ b/app/test/autotest_data.py
> @@ -175,7 +175,7 @@
>          "Command": "ticketlock_autotest",
>          "Func":    ticketlock_autotest,
>          "Report":  None,
> -    }
> +    },
>      {
>          "Name":    "Byte order autotest",
>          "Command": "byteorder_autotest", diff --git
> a/app/test/autotest_test_funcs.py b/app/test/autotest_test_funcs.py
> index 65fe335..31cc0f5 100644
> --- a/app/test/autotest_test_funcs.py
> +++ b/app/test/autotest_test_funcs.py
> @@ -131,6 +131,40 @@ def rwlock_autotest(child, test_name):
>      return 0, "Success"
> 
> 
> +def ticketlock_autotest(child, test_name):
> +    i = 0
> +    ir = 0
> +    child.sendline(test_name)
> +    while True:
> +        index = child.expect(["Test OK",
> +                              "Test Failed",
> +                              "Hello from core ([0-9]*) !",
> +                              "Hello from within recursive locks "
> +                              "from ([0-9]*) !",
> +                              pexpect.TIMEOUT], timeout=5)
> +        # ok
> +        if index == 0:
> +            break
> +
> +        # message, check ordering
> +        elif index == 2:
> +            if int(child.match.groups()[0]) < i:
> +                return -1, "Fail [Bad order]"
> +            i = int(child.match.groups()[0])
> +        elif index == 3:
> +            if int(child.match.groups()[0]) < ir:
> +                return -1, "Fail [Bad order]"
> +            ir = int(child.match.groups()[0])
> +
> +        # fail
> +        elif index == 4:
> +            return -1, "Fail [Timeout]"
> +        elif index == 1:
> +            return -1, "Fail"
> +
> +    return 0, "Success"
> +
> +
>  def logs_autotest(child, test_name):
>      child.sendline(test_name)
> 
> --
> 2.7.4

Hi,

Reviewed-by: Phil Yang <phil.yang@arm.com>
Tested-by: Phil Yang <phil.yang@arm.com>

# python ./app/test/autotest.py x86_64-native-linuxapp-gcc/app/test ./x86_64-native-linuxapp-gcc  Ticketlock autotest
x86_64-native-linuxapp-gcc/app/test -c f -n 4
Running tests with 4 workers

Test name                     Test result                     Test    Total
================================================================================
Parallel autotests:
Start test2:                         Success                      [00m 00s][00m 00s]
Start test1:                         Success                      [00m 00s][00m 00s]
Start test0:                         Success                      [00m 01s][00m 01s]
Start test3:                         Success                      [00m 01s][00m 01s]
Ticketlock autotest:          Success                      [00m 00s][00m 01s]
================================================================================
Total run time: 00m 01s

# python ./app/test/autotest.py arm64-armv8a-linuxapp-gcc/app/test ./arm64-armv8a-linuxapp-gcc  Ticketlock autotest
arm64-armv8a-linuxapp-gcc/app/test -c f -n 4
Running tests with 4 workers

Test name                     Test result                     Test    Total
================================================================================
Parallel autotests:
Start test0:                         Success                      [00m 00s][00m 00s]
Start test1:                         Success                      [00m 00s][00m 00s]
Start test3:                         Success                      [00m 00s][00m 00s]
Start test2:                         Success                      [00m 00s][00m 00s]
Ticketlock autotest:          Success                      [00m 00s][00m 00s]
================================================================================
Total run time: 00m 00s

Thanks,
Phil

  parent reply	other threads:[~2019-04-15 15:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-15  9:04 Joyce Kong
2019-04-15  9:04 ` Joyce Kong
2019-04-15 15:45 ` Phil Yang (Arm Technology China) [this message]
2019-04-15 15:45   ` Phil Yang (Arm Technology China)
2019-04-15 20:41   ` Thomas Monjalon
2019-04-15 20:41     ` Thomas Monjalon

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=DB7PR08MB338504E54B343E7C79AD3F8FE92B0@DB7PR08MB3385.eurprd08.prod.outlook.com \
    --to=phil.yang@arm.com \
    --cc=Gavin.Hu@arm.com \
    --cc=Honnappa.Nagarahalli@arm.com \
    --cc=Joyce.Kong@arm.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=jerin.jacob@caviumnetworks.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=nd@arm.com \
    --cc=stephen@networkplumber.org \
    --cc=thomas@monjalon.net \
    /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).