public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Xiao Yang <yangx.jy@cn.fujitsu.com>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] netns/netns_helper.sh: Fix failure when testing ipv6 in netns
Date: Wed, 20 Dec 2017 17:39:55 +0800	[thread overview]
Message-ID: <5A3A2FEB.7090809@cn.fujitsu.com> (raw)
In-Reply-To: <9318bde0-cca5-7428-b046-b1f506bfdbba@oracle.com>

On 2017/12/19 16:24, Alexey Kodanev wrote:
> On 12/18/2017 05:50 AM, Xiao Yang wrote:
>> On 2017/12/15 22:06, Alexey Kodanev wrote:
>>> On 12/15/2017 04:13 PM, Xiao Yang wrote:
>>>> When testing ipv6 in network namespace, we got the following
>>>> error on some distros(e.g. RHEL7.5Alpha):
>>>> ----------------------------------------------------------------------------------
>>>> connect: Cannot assign requested address
>>>> netns_comm_ns_exec_ipv6_netlink 1 TFAIL: configuration and communication over veth0
>>>> connect: Cannot assign requested address
>>>> netns_comm_ns_exec_ipv6_netlink 2 TFAIL: configuration and communication over veth1
>>>> -----------------------------------------------------------------------------------
>>>>
>>>> We could fix this issue by waiting enough time(5s) for address
>>>> to pass duplicate address detection.
>>> netns tests disable DAD in setup so this shouldn't happen. It's quite
>>> likely that the kernel doesn't have the following upstream patch-fix:
>>>
>>> 094009531612 ("ipv6: set all.accept_dad to 0 by default")
>> Hi Alexey,
>>
>> Thanks for your explanation.  The patch mentioned in your reply can fix
>> the error.
>>
>> Should we fix the error by setting all.accept_dad to 0 manually in netns
>> or leave the
>> error to expose this issue?
> Ideally, we could check if the kernel is 4.14 and all.accept_dad
> not equals 0, then apply workaround and print some warning message...
Hi Alexey,

Checking kernel version seems unreliable because commit 35e015e1f577 and 
a2d3f3e33853
may be ported into some distros(e.g. RHEL7.5Alpha with kernel v3.10.0).

Thanks,
Xiao Yang
> Anyway, I see that the patch in the queue for stable:
> http://patchwork.ozlabs.org/bundle/davem/stable/?state=*
>
> Thanks,
> Alexey
>
>
>




  reply	other threads:[~2017-12-20  9:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-15 13:13 [LTP] [PATCH] netns/netns_helper.sh: Fix failure when testing ipv6 in netns Xiao Yang
2017-12-15 14:06 ` Alexey Kodanev
2017-12-18  2:50   ` Xiao Yang
2017-12-19  8:24     ` Alexey Kodanev
2017-12-20  9:39       ` Xiao Yang [this message]
2017-12-21  3:22       ` [LTP] [PATCH v2] " xiao yang
2018-01-05  2:55         ` Xiao Yang

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=5A3A2FEB.7090809@cn.fujitsu.com \
    --to=yangx.jy@cn.fujitsu.com \
    --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