On Fri, Jul 20, 2012 at 11:39:21PM +1200, Joe Stringer wrote:
> On 20 July 2012 15:53, Ben Pfaff wrote:
> ...
> > For the string representations, as I told Joe Stringer, I'm leaning
> > toward requiring them to be in the correct order for instructions listed
> > above. Otherwise it seems risky th
Your message dated Sat, 21 Jul 2012 06:02:39 +
with message-id
and subject line Bug#682187: fixed in openvswitch 1.4.2+git20120612-5
has caused the Debian Bug report #682187,
regarding openvswitch-controller: unowned files after purge (policy 6.8, 10.8)
to be marked as done.
This means that y
Accepted:
openvswitch-brcompat_1.4.2+git20120612-5_i386.deb
to main/o/openvswitch/openvswitch-brcompat_1.4.2+git20120612-5_i386.deb
openvswitch-common_1.4.2+git20120612-5_i386.deb
to main/o/openvswitch/openvswitch-common_1.4.2+git20120612-5_i386.deb
openvswitch-controller_1.4.2+git20120612-5
openvswitch_1.4.2+git20120612-5_i386.changes uploaded successfully to localhost
along with the files:
openvswitch_1.4.2+git20120612-5.dsc
openvswitch_1.4.2+git20120612-5.debian.tar.gz
openvswitch-common_1.4.2+git20120612-5_i386.deb
openvswitch-switch_1.4.2+git20120612-5_i386.deb
openvswit
On Fri, Jul 20, 2012 at 01:45:22PM -0700, Ansis Atteka wrote:
> On Fri, Jul 20, 2012 at 1:16 PM, Ben Pfaff wrote:
>
> > From: Ben Pfaff
> > Date: Fri, 20 Jul 2012 13:15:36 -0700
> > Subject: [PATCH] debian: Remove controller keys on openvswitch-controller
> > package purge.
> >
> > A Debian pack
I'll review this series.
Ethan
On Jul 20, 2012, at 16:24, Ben Pfaff wrote:
> This patch series reduces the memory usage of classifier rules while
> speeding up classification. It reduces the cost of adding more fields
> to the classifier.
>
> The first two commits are bug fixes for existing
On Thu, Jul 12, 2012 at 03:23:16PM -0700, Ben Pfaff wrote:
> Signed-off-by: Ben Pfaff
I re-posted this as the first patch in a larger series:
http://openvswitch.org/pipermail/dev/2012-July/019275.html
Please review it there.
___
dev mailing list
A cls_rule is 324 bytes on i386 now. The cost of a flow table lookup is
currently proportional to this size, which is going to continue to grow.
However, the required cost of a flow table lookup, with the classifier that
we currently use, is only proportional to the number of bits that a rule
actu
When cls_cursor_init() is given a NULL target, it can skip an expensive
step comparing the rule against the target for every table and every rule
in the classifier. collect_rule_loose() and other callers could take
advantage of this optimization, except that they actually pass in a rule
that match
Until now, "struct cls_rule" didn't own any data outside its own memory
block. An upcoming commit will make "struct cls_rule" sometimes own blocks
of memory, so it needs "destroy" and to a lesser extent "clone" functions.
This commit adds these in advance, even though they are mostly no-ops, to
ma
Signed-off-by: Ben Pfaff
---
lib/classifier.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/classifier.c b/lib/classifier.c
index 88c8e1a..fdd242a 100644
--- a/lib/classifier.c
+++ b/lib/classifier.c
@@ -817,7 +817,7 @@ classifier_is_empty(const struct classifier *
This is the fastest portable implementation among the ones below, as
measured with GCC 4.4 on a Xeon X3430. The measeured times were, in
seconds:
popcount125.6
popcount2 6.9 (but is not portable)
popcount331.4
popcount425.6
popcount561.6 (and is buggy)
popcount664.6
popcou
It's probably easier to understand
x = zero_rightmost_1bit(x);
than
x &= x - 1;
Signed-off-by: Ben Pfaff
---
lib/util.h |8
ofproto/ofproto-dpif.c |4 ++--
tests/test-classifier.c |2 +-
3 files changed, 11 insertions(+), 3 deletions(-)
diff --
Now that "struct flow" and "struct flow_wildcards" have the same simple
and uniform structure, it's easy to handle common operations by just
iterating over the bits inside them.
Signed-off-by: Ben Pfaff
---
lib/classifier.c | 71 ---
lib/flow.c | 262 +---
It's only used in a not-very-useful assertion in some test code. In
general, exact-match flows make very little sense anymore, and they're
basically on their way out.
Signed-off-by: Ben Pfaff
---
lib/flow.c | 42 --
lib/flow.h
Since we know these bytes are always 0 in both structures, we can use
faster functions that only work with full words.
Signed-off-by: Ben Pfaff
---
lib/flow.c |6 +-
lib/flow.h | 20 +++-
2 files changed, 8 insertions(+), 18 deletions(-)
diff --git a/lib/flow.c b/lib/f
Signed-off-by: Ben Pfaff
---
lib/flow.h |2 +-
lib/ofp-util.c |1 +
tests/test-bundle.c |1 +
tests/test-classifier.c |1 +
tests/test-multipath.c |1 +
5 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/lib/flow.h b/lib/flow.h
index 7a40
Signed-off-by: Ben Pfaff
---
lib/classifier.c|8 ++---
lib/flow.c | 27 ++
lib/flow.h | 27 ++
lib/meta-flow.c | 90 ++-
lib/meta-flow.h |1 -
lib/nx-match.c
Signed-off-by: Ben Pfaff
---
lib/classifier.c| 12 ++--
lib/flow.c | 29 +
lib/flow.h | 14 +++---
lib/meta-flow.c | 12 +++-
lib/nx-match.c | 15 +++
lib/ofp-util.c
Signed-off-by: Ben Pfaff
---
lib/classifier.c| 14 +++---
lib/flow.c | 29 +
lib/flow.h | 14 +++---
lib/meta-flow.c | 22 ++
lib/nx-match.c |7 +++
lib/ofp-util.c
Signed-off-by: Ben Pfaff
---
lib/classifier.c | 10 +-
lib/flow.c | 41 -
lib/flow.h | 13 +++--
lib/meta-flow.c | 11 +++
lib/nx-match.c |4 ++--
lib/ofp-util.c | 12 +---
6 files changed, 50 i
Signed-off-by: Ben Pfaff
---
lib/classifier.c| 11 +--
lib/flow.c | 24
lib/flow.h | 10 --
lib/meta-flow.c | 24
lib/nx-match.c |4 ++--
lib/ofp-util.c | 16
It might have been a useful optimization at one point to have FWW_*
correspond in OFPFW10_* where possible, but it doesn't seem worthwhile for
only 3 corresponding values. It also makes the code somewhat more
confusing.
Signed-off-by: Ben Pfaff
---
lib/flow.h | 12 +---
lib/ofp-ut
Signed-off-by: Ben Pfaff
---
ofproto/ofproto-dpif.c | 58 ++-
ofproto/ofproto-provider.h | 14 ---
ofproto/ofproto.c | 16 +---
3 files changed, 36 insertions(+), 52 deletions(-)
diff --git a/ofproto/ofproto-dpif.c b/ofpr
This allows port numbers in actions and elsewhere in OpenFlow messages to
be checked at a higher level.
Signed-off-by: Ben Pfaff
---
ofproto/ofproto-dpif.c | 16 +++-
ofproto/ofproto-provider.h | 30 +++---
ofproto/ofproto.c | 26 +++
Signed-off-by: Ben Pfaff
---
lib/ofp-util.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 1c9763d..4d5dfbb 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -443,7 +443,6 @@ ofputil_cls_rule_to_ofp11_match(const struct cls_rule *r
This comment is talking about the obsolete "tun_id_from_cookie" protocol
extension that we removed from OVS long ago.
Signed-off-by: Ben Pfaff
---
lib/ofp-util.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index faa0687..1c9763d 10064
Casts are sometimes necessary. One common reason that they are necessary
is for discarding a "const" qualifier. However, this can impede
maintenance: if the type of the expression being cast changes, then the
presence of the cast can hide a necessary change in the code that does the
cast. Using
Signed-off-by: Ben Pfaff
---
tests/test-sha1.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/test-sha1.c b/tests/test-sha1.c
index 8eb5c5e..2be7942 100644
--- a/tests/test-sha1.c
+++ b/tests/test-sha1.c
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2009, 2011 Nicira,
'packet' is already a struct ofpbuf *.
---
ofproto/ofproto-dpif.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index dc15c15..6bc2cf4 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -4725,7 +4725,7 @@
Signed-off-by: Ben Pfaff
---
lib/flow.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/flow.c b/lib/flow.c
index eef84df..222983a 100644
--- a/lib/flow.c
+++ b/lib/flow.c
@@ -179,7 +179,7 @@ parse_ipv6(struct ofpbuf *packet, struct flow *flow)
|
This probably means that some classifier functions based on the fragment
type of packets have never worked properly.
Signed-off-by: Ben Pfaff
---
lib/flow.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/lib/flow.c b/lib/flow.c
index 5ba3e10..eef84df 100644
--- a/li
Signed-off-by: Ben Pfaff
---
tests/ofproto.at | 27 ---
1 files changed, 24 insertions(+), 3 deletions(-)
diff --git a/tests/ofproto.at b/tests/ofproto.at
index 2d28131..e525c67 100644
--- a/tests/ofproto.at
+++ b/tests/ofproto.at
@@ -934,17 +934,38 @@ echo adds=$adds d
This patch series reduces the memory usage of classifier rules while
speeding up classification. It reduces the cost of adding more fields
to the classifier.
The first two commits are bug fixes for existing code:
ofproto: Avoid sensitivity to hash order in flow monitor pause/resume
test.
From: Jesse Gross
Date: Fri, 20 Jul 2012 15:26:43 -0700
> A few bug fixes and small enhancements for net-next/3.6.
>
> The following changes since commit bf32fecdc1851ad9ca960f56771b798d17c26cf1:
>
> openvswitch: Add length check when retrieving TCP flags. (2012-04-02
> 14:28:57 -0700)
>
>
From: Leo Alterman
Signed-off-by: Leo Alterman
Signed-off-by: Jesse Gross
---
Documentation/networking/openvswitch.txt |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/networking/openvswitch.txt
b/Documentation/networking/openvswitch.txt
index b8a048b..8fa2
From: Ben Pfaff
At the point where it was used, skb_shinfo(skb)->gso_type referred to a
post-GSO sk_buff. Thus, it would always be 0. We want to know the pre-GSO
gso_type, so we need to obtain it before segmenting.
Before this change, the kernel would pass inconsistent data to userspace:
packe
From: Pravin B Shelar
Fix return check typo.
Signed-off-by: Pravin B Shelar
Signed-off-by: Jesse Gross
---
net/openvswitch/datapath.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c
index b512cb8..670e630 10064
It's possible that packets that are sent on internal devices (from
the OVS perspective) have already traversed the local IP stack.
After they go through the internal device, they will again travel
through the IP stack which may get confused by the presence of
existing information in the skb. The pr
From: Raju Subramanian
Replaced all instances of Nicira Networks(, Inc) to Nicira, Inc.
Signed-off-by: Raju Subramanian
Signed-off-by: Ben Pfaff
Signed-off-by: Jesse Gross
---
net/openvswitch/actions.c|2 +-
net/openvswitch/datapath.c |2 +-
net/openvswitch/data
From: Ansis Atteka
There is no need to send a notification if ovs_vport_set_options() failed
and ovs_vport_cmd_set() did not change anything.
Signed-off-by: Ansis Atteka
Signed-off-by: Jesse Gross
---
net/openvswitch/datapath.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff
We currently check that a packet is IPv4 and TCP before fetching the
TCP flags. This enables fetching from IPv6 packets as well.
Reported-by: Michael Mao
Signed-off-by: Jesse Gross
---
net/openvswitch/flow.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/openvswit
A few bug fixes and small enhancements for net-next/3.6.
The following changes since commit bf32fecdc1851ad9ca960f56771b798d17c26cf1:
openvswitch: Add length check when retrieving TCP flags. (2012-04-02 14:28:57
-0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/li
On Fri, Jul 20, 2012 at 1:16 PM, Ben Pfaff wrote:
> On Fri, Jul 20, 2012 at 12:16:30PM -0700, Ansis Atteka wrote:
> > On Fri, Jul 20, 2012 at 9:24 AM, Ben Pfaff wrote:
> >
> > > A Debian package is expected to remove all its configuration files
> (which
> > > includes all files in /etc) when it
On Fri, Jul 20, 2012 at 12:16:30PM -0700, Ansis Atteka wrote:
> On Fri, Jul 20, 2012 at 9:24 AM, Ben Pfaff wrote:
>
> > A Debian package is expected to remove all its configuration files (which
> > includes all files in /etc) when it is purged, but the
> > openvswitch-controller package wasn't do
Thanks, pushed to master.
On Fri, Jul 20, 2012 at 12:33:24PM -0700, Mehak Mahajan wrote:
> Hey Ben,
>
> Looks good to me.
>
> thanx!
> mehak
>
> On Fri, Jul 20, 2012 at 10:51 AM, Ben Pfaff wrote:
>
> > At one point I got the "lcov" utilities to work well with OVS. Then I
> > didn't try to us
You're welcome.
On Fri, Jul 20, 2012 at 12:46:24PM -0700, Ethan Jackson wrote:
> This has always bothered me, thanks for removing them.
>
> Ethan
>
> On Fri, Jul 20, 2012 at 10:39 AM, Ben Pfaff wrote:
> > The daemon code works fine, these tests didn't. No one has debugged them
> > over a perio
Thanks, I applied this to master.
On Fri, Jul 20, 2012 at 12:33:02PM -0700, Mehak Mahajan wrote:
> Hey Ben,
>
> Looks good to me.
>
> thanx!
> mehak
>
> On Fri, Jul 20, 2012 at 10:39 AM, Ben Pfaff wrote:
>
> > The daemon code works fine, these tests didn't. No one has debugged them
> > over
On Fri, Jul 20, 2012 at 07:53:02PM +, Kyle Mestery (kmestery) wrote:
> On Jul 20, 2012, at 1:52 PM, Ben Pfaff wrote:
> > On Wed, May 23, 2012 at 09:34:07AM -0700, Ben Pfaff wrote:
> >> The function ofputil_decode_msg_type_partial() can figure out the type of
> >> a truncated OpenFlow message, i
> There will still be a problem if initialization of the two endpoints
> isn't well-synchronized, i.e. if one end of the tunnel gets set up 5
> seconds after the other one. I don't know whether there's anything we
> can do about that.
Yes, the case I'm particularly worried about is when a control
On Wed, Jul 18, 2012 at 11:54:55PM -0700, Ethan Jackson wrote:
> This patch makes a two improvements to CFM logging which should
> make debugging connectivity problems a bit more intuitive. First,
> when a remote_mp disappears, the length of time since its last CCM
> reception is logged. Second,
[your clock is off again]
On Wed, Jul 18, 2012 at 11:45:05PM -0700, Ethan Jackson wrote:
> When CFM is first configured, it detects no remote endpoints, and
> thus sets RDI on its CCMs. This can cause the receiver of these
> CCMs to think there is a problem when really things are simply
> initial
This patch makes a two improvements to CFM logging which should
make debugging connectivity problems a bit more intuitive. First,
when a remote_mp disappears, the length of time since its last CCM
reception is logged. Second, the "CFM fault status changed"
message is reformatted in a more intuiti
On Jul 20, 2012, at 1:52 PM, Ben Pfaff wrote:
> On Wed, May 23, 2012 at 09:34:07AM -0700, Ben Pfaff wrote:
>> The function ofputil_decode_msg_type_partial() can figure out the type of
>> a truncated OpenFlow message, if the message is long enough that its type
>> can be determined, so we should pri
This has always bothered me, thanks for removing them.
Ethan
On Fri, Jul 20, 2012 at 10:39 AM, Ben Pfaff wrote:
> The daemon code works fine, these tests didn't. No one has debugged them
> over a period of years so we might as well remove them.
>
> Signed-off-by: Ben Pfaff
> ---
> tests/daemo
When CFM is first configured, it detects no remote endpoints, and
thus sets RDI on its CCMs. This can cause the receiver of these
CCMs to think there is a problem when really things are simply
initializing. This patch fixes the issue by not setting the RDI
bit in CCMs until at least one fault int
Hey Ben,
Looks good to me.
thanx!
mehak
On Fri, Jul 20, 2012 at 10:51 AM, Ben Pfaff wrote:
> At one point I got the "lcov" utilities to work well with OVS. Then I
> didn't try to use them again for a year or so, and when I did I found that
> it didn't work at all. I wasn't able to fix the pr
Hey Ben,
Looks good to me.
thanx!
mehak
On Fri, Jul 20, 2012 at 10:39 AM, Ben Pfaff wrote:
> The daemon code works fine, these tests didn't. No one has debugged them
> over a period of years so we might as well remove them.
>
> Signed-off-by: Ben Pfaff
> ---
> tests/daemon.at | 46 ---
On Fri, Jul 20, 2012 at 9:24 AM, Ben Pfaff wrote:
> A Debian package is expected to remove all its configuration files (which
> includes all files in /etc) when it is purged, but the
> openvswitch-controller package wasn't doing that. This fixes the problem.
>
> Debian bug #682187.
> CC: 682...@
On Wed, May 23, 2012 at 09:34:07AM -0700, Ben Pfaff wrote:
> The function ofputil_decode_msg_type_partial() can figure out the type of
> a truncated OpenFlow message, if the message is long enough that its type
> can be determined, so we should print this information. This is
> especially valuabl
> I do think that's an improvement. One more tweak: how about "CFM
> faults" instead of "CFM fault status". To me the former more clearly
> implies that everything in each list is a fault, whereas the latter
> talks more vaguely about a "status".
Sounds good, I'll tweak the patch and send out an
I do think that's an improvement. One more tweak: how about "CFM
faults" instead of "CFM fault status". To me the former more clearly
implies that everything in each list is a fault, whereas the latter
talks more vaguely about a "status".
Thanks,
Ben.
On Fri, Jul 20, 2012 at 11:14:32AM -0700,
I'm not sure what a better wording would be. Perhaps something like
the following?
CFM fault status changed from [] to [rdi interval]. The brakets make
it clear that it's a list.
Ethan
On Thu, Jul 19, 2012 at 8:12 PM, Ben Pfaff wrote:
> On Wed, Jul 18, 2012 at 09:10:59PM -0700, Ethan Jackson
Thanks, applied to master.
On Fri, Jul 20, 2012 at 11:09:00AM -0700, Pravin Shelar wrote:
> Looks good.
>
> Thanks.
>
> On Fri, Jul 20, 2012 at 10:25 AM, Ben Pfaff wrote:
> > Otherwise the kernel will reject it later and the result is no tunnel,
> > whereas a tunnel with an unexpected TOS seems
Looks good.
Thanks.
On Fri, Jul 20, 2012 at 10:25 AM, Ben Pfaff wrote:
> Otherwise the kernel will reject it later and the result is no tunnel,
> whereas a tunnel with an unexpected TOS seems like a better result.
>
> Bug #12566.
> Reported-by: Luca Giraudo
> Signed-off-by: Ben Pfaff
> ---
>
At one point I got the "lcov" utilities to work well with OVS. Then I
didn't try to use them again for a year or so, and when I did I found that
it didn't work at all. I wasn't able to fix the problem easily, so it
seems better to remove the feature than to leave around broken code.
Signed-off-b
The daemon code works fine, these tests didn't. No one has debugged them
over a period of years so we might as well remove them.
Signed-off-by: Ben Pfaff
---
tests/daemon.at | 46 --
1 files changed, 0 insertions(+), 46 deletions(-)
diff --git a/te
On Fri, Jul 20, 2012 at 10:29:25AM +0900, Simon Horman wrote:
> Add OFPUTIL_P_OF12 and NXFF_OPENFLOW12 for Open Flow 1.2
>
> OFPUTIL_P_OF12_TID and in turn OFPUTIL_P_OF12_ANY is not provided as
> OFPUTIL_P_OF12 supports the use of table ids in modify flow messages.
>
> Signed-off-by: Simon Horman
Otherwise the kernel will reject it later and the result is no tunnel,
whereas a tunnel with an unexpected TOS seems like a better result.
Bug #12566.
Reported-by: Luca Giraudo
Signed-off-by: Ben Pfaff
---
lib/netdev-vport.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --
On Fri, Jul 20, 2012 at 08:51:09PM +0900, Simon Horman wrote:
> Signed-off-by: Simon Horman
>
> ---
>
> Tests pass with the "[PATCH 0/6 v3] OXM VLAN VID and VLAN PCP" series
> applied on top of the current master branch (df77824).
I got a patch reject on patch 6/6 in that series and then on thi
A Debian package is expected to remove all its configuration files (which
includes all files in /etc) when it is purged, but the
openvswitch-controller package wasn't doing that. This fixes the problem.
Debian bug #682187.
CC: 682...@bugs.debian.org
Reported-by: Andreas Beckmann
Signed-off-by: B
Your message dated Fri, 20 Jul 2012 07:42:18 -0700
with message-id <87mx2ucx4l@blp.benpfaff.org>
and subject line Re: Bug#682182: ERROR: could not insert 'brcompat_mod': Exec
format error when inserting brcompat_mod.ko
has caused the Debian Bug report #682182,
regarding ERROR: could not insert
Dear Sir or Madam,
I am writing to let you know that the charter rates for our 26m Motor yacht
Lady Tatiana of London that is based in Mallorca (and that charters in
Mallorca, Ibiza, Menorca, Barcelona and the neighbouring waters) have been
dramatically reduced Please note that the new rates
On Fri, Jul 20, 2012 at 03:27:25PM +0800, Aron Xu wrote:
> I was trying to enable Linux bridge compatibility with openvswitch in
> Wheezy, and when doing modprobe something appears to be wrong:
> # modprobe brcompat_mod
> ERROR: could not insert 'brcompat_mod': Exec format error
What does "file" o
On Fri, Jul 20, 2012 at 10:13 PM, Ben Pfaff wrote:
> On Fri, Jul 20, 2012 at 03:27:25PM +0800, Aron Xu wrote:
>> I was trying to enable Linux bridge compatibility with openvswitch in
>> Wheezy, and when doing modprobe something appears to be wrong:
>> # modprobe brcompat_mod
>> ERROR: could not in
On Thu, Jul 19, 2012 at 09:51:29PM -0700, Ben Pfaff wrote:
> On Fri, Jul 20, 2012 at 01:47:09PM +0900, Simon Horman wrote:
> > Thanks, that helps a lot.
> >
> > I will see about implementing the OF1.2 version of these tests and
> > fix my implementation as needed.
>
> Awesome. If, after consider
Signed-off-by: Simon Horman
---
Tests pass with the "[PATCH 0/6 v3] OXM VLAN VID and VLAN PCP" series
applied on top of the current master branch (df77824).
---
tests/ovs-ofctl.at| 12 ++--
utilities/ovs-ofctl.c | 14 +++---
2 files changed, 21 insertions(+), 5 deletions(-)
On 20 July 2012 15:53, Ben Pfaff wrote:
...
> For the string representations, as I told Joe Stringer, I'm leaning
> toward requiring them to be in the correct order for instructions listed
> above. Otherwise it seems risky that one could confuse the user.
> Similarly for the OF1.0 extension repre
Package: openvswitch-controller
Version: 1.4.2+git20120612-4
Severity: important
User: debian...@lists.debian.org
Usertags: piuparts
Hi,
during a test with piuparts I noticed your package left unowned files on
the system after purge, which is a violation of policy 6.8 (or 10.8):
http://www.debia
Package: openvswitch-datapath-dkms
Severity: serious
Version: 1.4.2+git20120612-2
Hi,
I was trying to enable Linux bridge compatibility with openvswitch in
Wheezy, and when doing modprobe something appears to be wrong:
# modprobe brcompat_mod
ERROR: could not insert 'brcompat_mod': Exec format er
80 matches
Mail list logo
