From: Amos Kong <akong@redhat.com>
To: pradeep <psuriset@linux.vnet.ibm.com>
Cc: Lucas Meneghel Rodrigues <lmr@redhat.com>,
autotest@test.kernel.org, kvm@vger.kernel.org
Subject: Re: [AUTOTEST] [PATCH 1/2] KVM : ping6 test
Date: Thu, 14 Oct 2010 18:05:04 +0800 [thread overview]
Message-ID: <20101014100504.GA8065@z> (raw)
In-Reply-To: <20101014145659.41e45b35@skywalker>
On Thu, Oct 14, 2010 at 02:56:59PM +0530, pradeep wrote:
> This patch is for Ping6 testing
>
> * ping6 with various message sizes guest to/from local/remote host
> using link-local addresses
> By default IPv6 seems to be disabled on virbr0. Enable it by
> doing echo 0 > /proc/sys/net/ipv6/conf/virbr0/disable_ipv6
>
> Please find the below attached patch
We also need update related code in kvm_test_utils.py, and consider the difference of
'ping' and 'ping6'.
> Signed-off-by: Pradeep K Surisetty <psuriset@linux.vnet.ibm.com>
> ---
> --- autotest/client/tests/kvm/tests/ping.py 2010-10-14 14:20:52.523791118 +0530
> +++ autotest_new/client/tests/kvm/tests/ping.py 2010-10-14 14:46:57.711797139 +0530
> @@ -1,5 +1,6 @@
> -import logging
> +import logging, time
> from autotest_lib.client.common_lib import error
> +from autotest_lib.client.bin import utils
> import kvm_test_utils
>
>
> @@ -27,10 +28,18 @@ def run_ping(test, params, env):
> nics = params.get("nics").split()
> strict_check = params.get("strict_check", "no") == "yes"
>
> + address_type = params.get("address_type")
> + #By default IPv6 seems to be disabled on virbr0.
> + ipv6_cmd = "echo %s > /proc/sys/net/ipv6/conf/virbr0/disable_ipv6"
We may use other bridge, so 'virbr0', need replace this hardcode name.
We can reference to 'autotest-upstream/client/tests/kvm/scripts/qemu-ifup'
switch=$(/usr/sbin/brctl show | awk 'NR==2 { print $1 }')
> +
> packet_size = [0, 1, 4, 48, 512, 1440, 1500, 1505, 4054, 4055, 4096, 4192,
> 8878, 9000, 32767, 65507]
>
> try:
> + if address_type == "ipv6":
> + utils.run(ipv6_cmd % "0" )
> + time.sleep(5)
> +
> for i, nic in enumerate(nics):
> ip = vm.get_address(i)
> if not ip:
> @@ -68,5 +77,9 @@ def run_ping(test, params, env):
> if status != 0:
> raise error.TestFail("Ping returns non-zero value %s" %
> output)
> + if address_type == "ipv6":
> + utils.run(ipv6_cmd % "1" )
> + time.sleep(5)
> +
> finally:
> session.close()
> ---
next prev parent reply other threads:[~2010-10-14 10:05 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-14 4:24 [PATCH 1/2] KVM test: Make cache=none the default on KVM autotest runs Lucas Meneghel Rodrigues
2010-10-14 4:24 ` [PATCH 2/2] KVM test: Remove image_boot=yes from virtio_blk variant Lucas Meneghel Rodrigues
2010-10-14 6:37 ` pradeep
2010-10-14 9:26 ` [AUTOTEST] [PATCH 1/2] KVM : ping6 test pradeep
2010-10-14 10:05 ` Amos Kong [this message]
2010-10-14 10:45 ` [Autotest] " pradeep
2010-10-14 9:29 ` pradeep
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=20101014100504.GA8065@z \
--to=akong@redhat.com \
--cc=autotest@test.kernel.org \
--cc=kvm@vger.kernel.org \
--cc=lmr@redhat.com \
--cc=psuriset@linux.vnet.ibm.com \
/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