Re: [ovs-dev] 答复: Re: 答复: Re: 答复: Re: [PATCH v2] ovn: Support for taas(tap-as-a-service) function

2017-09-08 Thread Takashi YAMAMOTO
On Wed, Sep 6, 2017 at 3:57 AM, Russell Bryant wrote: > What if a mirror port *only* receives mirrored packets? If the only > packets it ever receives are mirrored packets, a new flag would not be > necessary. > > Do you intend for the port to operate as both a regular port

Re: [ovs-dev] [PATCH] acinclude.m4: Avoid error from printf.

2017-07-18 Thread Takashi YAMAMOTO
oops, thank you. On Tue, Jul 18, 2017 at 2:12 AM, Ben Pfaff wrote: > GNU (at least) printf interprets -I as an option, but we want to print it > literally, so use %s. > > CC: YAMAMOTO Takashi > Fixes: 27d41afaa446 ("acinclude.m4: Avoid echo -n") > Signed-off-by:

Re: [ovs-dev] [PATCH] acinclude.m4: Avoid echo -n

2017-07-15 Thread Takashi YAMAMOTO
On Sun, Jul 16, 2017 at 7:45 AM, Ben Pfaff wrote: > On Sun, Jul 16, 2017 at 12:17:28AM +0900, YAMAMOTO Takashi wrote: > > -n option for echo is not portable. Use printf instead. > > This fixes OSX build on travis-ci. > > > > Signed-off-by: YAMAMOTO Takashi > >

Re: [ovs-dev] [PATCH 5/6] ovn.at: Use a correct operator in a test expression

2017-07-15 Thread Takashi YAMAMOTO
On Sat, Jul 15, 2017 at 6:56 AM, Ben Pfaff wrote: > On Sat, Jul 15, 2017 at 02:39:45AM +0900, YAMAMOTO Takashi wrote: > > == is a GNU extension. > > > > Signed-off-by: YAMAMOTO Takashi > > I wish this didn't exist in the GNU version. It's a useless extension. >

Re: [ovs-dev] [PATCH 6/6] packet-type-aware.at: Avoid GNU extension

2017-07-15 Thread Takashi YAMAMOTO
On Sat, Jul 15, 2017 at 6:56 AM, Ben Pfaff wrote: > On Sat, Jul 15, 2017 at 02:39:46AM +0900, YAMAMOTO Takashi wrote: > > \t is GNU sed extension. > > > > Signed-off-by: YAMAMOTO Takashi > > Acked-by: Ben Pfaff > applied to master. thank you.

Re: [ovs-dev] [PATCH 3/6] pmd.at: Avoid using GNU sed extension

2017-07-15 Thread Takashi YAMAMOTO
On Sat, Jul 15, 2017 at 6:55 AM, Ben Pfaff wrote: > On Sat, Jul 15, 2017 at 02:39:43AM +0900, YAMAMOTO Takashi wrote: > > BRE alternative (\|) is an GNU sed extension. [1] > > It isn't available in NetBSD sed. > > > > [1]

Re: [ovs-dev] [PATCH 2/6] NetBSD: Stop recommending pkg_alternatives

2017-07-15 Thread Takashi YAMAMOTO
On Sat, Jul 15, 2017 at 6:55 AM, Ben Pfaff wrote: > On Sat, Jul 15, 2017 at 02:39:42AM +0900, YAMAMOTO Takashi wrote: > > Because: > > - It's no longer necessary > > - It can cause problems for some utilities (e.g. ovs-pcap) > > Cf.

Re: [ovs-dev] [PATCH 1/6] interface-reconfigure.at: Use $PYTHON explicitly

2017-07-15 Thread Takashi YAMAMOTO
On Sat, Jul 15, 2017 at 6:55 AM, Ben Pfaff wrote: > On Sat, Jul 15, 2017 at 02:39:41AM +0900, YAMAMOTO Takashi wrote: > > Workaround pkg_alternative issue for NetBSD: > > http://gnats.netbsd.org/cgi-bin/query-pr-single.pl?number=51152 > > > > An alternative would be generating

Re: [ovs-dev] [PATCH 4/6] testsuite: Drain the list of jobs in the shell

2017-07-14 Thread Takashi YAMAMOTO
Hi, 2017/07/15 6:55 "Ben Pfaff" : On Sat, Jul 15, 2017 at 02:39:44AM +0900, YAMAMOTO Takashi wrote: > SUS says: > When jobs reports the termination status of a job, > the shell removes its process ID from the list of those > "known in the current shell execution

Re: [ovs-dev] [PATCH] socket-util: Support sendmmsg() on Linux and non-Linux platforms.

2017-07-12 Thread Takashi YAMAMOTO
On Thu, Jul 13, 2017 at 1:23 AM, Ben Pfaff wrote: > This will have its first user in an upcoming commit. > > Signed-off-by: Ben Pfaff > --- > configure.ac| 3 ++- > include/sparse/sys/socket.h | 5 + > lib/socket-util.c | 38

Re: [ovs-dev] [PATCH] ovn.at: Fix "ovn -- 1 LR with distributed router gateway port" test

2017-05-01 Thread Takashi YAMAMOTO
On Tue, May 2, 2017 at 5:21 AM, Ben Pfaff wrote: > On Fri, Apr 21, 2017 at 10:32:57AM +0900, YAMAMOTO Takashi wrote: > > NetBSD implementation of wc command outputs extra whitespaces > > like the following. Tweak the test to success on such environments. > > > > % echo hoge|wc

Re: [ovs-dev] [PATCH] ovn.at: Fix "ovn -- 1 LR with distributed router gateway port" test

2017-04-24 Thread Takashi YAMAMOTO
On Mon, Apr 24, 2017 at 9:48 AM, Mickey Spiegel wrote: > > On Thu, Apr 20, 2017 at 6:32 PM, YAMAMOTO Takashi > wrote: > >> NetBSD implementation of wc command outputs extra whitespaces >> like the following. Tweak the test to success on such

Re: [ovs-dev] [PATCH] Update netbsd install doc

2016-12-21 Thread Takashi YAMAMOTO
On Wed, Dec 21, 2016 at 3:25 AM, Hui Kang wrote: > - test ovs on netbsd 7.0.2 > - use gmake to compile and install > > Signed-off-by: Hui Kang > --- > Documentation/intro/install/netbsd.rst | 5 +++-- > 1 file changed, 3 insertions(+), 2

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-12-13 Thread Takashi YAMAMOTO
On Wed, Dec 14, 2016 at 9:02 AM, Jarno Rajahalme <ja...@ovn.org> wrote: > > On Dec 13, 2016, at 3:32 PM, Takashi YAMAMOTO <yamam...@ovn.org> wrote: > > > > On Tue, Dec 13, 2016 at 6:49 PM, Thomas Morin <thomas.mo...@orange.com> > wrote: > >

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-12-13 Thread Takashi YAMAMOTO
On Tue, Dec 13, 2016 at 6:49 PM, Thomas Morin wrote: > Hi Jarno, > > 2016-12-10, Jarno Rajahalme: > > On Dec 9, 2016, at 7:04 AM, Thomas Morin wrote: > > > 2016-12-09, Thomas Morin: > > In the same setup as the one on which the bug was observed,

Re: [ovs-dev] [PATCH] mpls: Fix MPLS restoration after patch port and group bucket.

2016-12-03 Thread Takashi YAMAMOTO
and 'base flow' > and issuing MPLS actions if they can transform the packet from an MPLS > packet to a non-MPLS packet. For an IP packet the change in ttl can > then be correctly committed before the last two output actions. > > Two test cases are added to prevent future regressions. &

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-12-02 Thread Takashi YAMAMOTO
hi, On Fri, Dec 2, 2016 at 10:57 AM, Jarno Rajahalme wrote: > > > On Nov 30, 2016, at 8:50 PM, Ben Pfaff wrote: > > > > On Wed, Nov 30, 2016 at 06:58:57PM -0800, Jarno Rajahalme wrote: > >> > >>> On Nov 30, 2016, at 8:41 AM, Ben Pfaff wrote: > >>> >