irst asap else if it's fine we can send "Vacancy
Events" Feature patches too.
We are looking forward to hear from you soon.
Rishi
Thank You
Regards
Rishi Bamba
- Original Message -
> From: "Rishi Bamba"
> To: "Ben Pfaff"
> Cc: dev@openvs
provide further inputs on the same(if any) or if any action
is required from our end.
Rishi
Thank You
Regards
Rishi Bamba
=-=-=
Notice: The information contained in this e-mail
message and/or attachments to it may contain
confidential or privileged information. If you are
n
evised patch for "ovs-ofctl: Implementation of eviction on the
basis of Importance" today i.e 11.02.2015.
Thank you for the review.
Rishi
Thank You
Regards
Rishi Bamba
=-=-=
Notice: The information contained in this e-mail
message and/or attachments to it may contai
From: Saloni Jain
This commit enables the eviction mechanism on the basis of importance as
per the openflow specification 1.4.
ovs-ofctl -O OpenFlow14 mod-table evict
-Enable eviction on of . Eviction adds a mechanism
enabling the switch to automatically eliminate entries of lower
i
From: Saloni Jain
This commit enables the eviction mechanism on the basis of importance as
per the openflow specification 1.4.
ovs-ofctl -O OpenFlow14 mod-table evict
-Enable eviction on of . Eviction adds a mechanism
enabling the switch to automatically eliminate entries of lower
i
pm.table_vacancy);
> lib/ofp-print.c: td->table_vacancy);
>
>
Hopefully these will not be there now. Request to kindly check with the revised
patch.
Rishi
Thank You
Regards
Rishi Bamba
=-=-=
Notice: The information contain
From: Saloni Jain
This commit enables the eviction mechanism on the basis of importance as
per the openflow specification 1.4.
ovs-ofctl -O OpenFlow14 mod-table evict
-Enable eviction on of . Eviction adds a mechanism
enabling the switch to automatically eliminate entries of lower
i
Eviction on the basis of "importance".This patch also enhances the
> > diff-flows & replace-flows CLI for addition of importance parameter
> > in
> > a rule.
> >
> > Also changes are made to DESIGN.md,NEWS,ovs-ofctl.8.in and added
> > test cases
> &
}
> > bash-3.2$
>
> On my bash, I get 1 3 5 7 for the latter.
>
> This is my fault because I did not read the tests in this patch
> carefully. If I had, I would have replaced both of these nonportable
> constructs (which I hadn't even heard of before). Anyway,
From: Rishi Bamba
This patch enables a user to set importance for a new rule via add-flow
OF1.4+ in the OVS and display the same via dump-flows command OF1.4+ .
The changes are made in accordance with OpenFlow 1.4 specs to implement
Eviction on the basis of "importance".This patch als
Eviction on the basis of "importance".This patch also enhances the
> > diff-flows & replace-flows CLI for addition of importance parameter
> > in
> > a rule.
> >
> > Also changes are made to DESIGN.md and added test cases for
> > add-flow &
> &g
o enhances the
diff-flows & replace-flows CLI for addition of importance parameter in
a rule.
Also changes are made to DESIGN.md and added test cases for add-flow &
replace-flows.
Signed-off-by: Rishi Bamba
---
DESIGN.md | 7 --
include/openflow/openflow-1.
specs to
> > implement
> > Eviction on the basis of "importance".
> >
> > Signed-off-by: Rishi Bamba
>
> Thanks for the patch! It applies properly to my tree. I am glad
> that
> you were able to work out how to send patches by email.
Yeah we managed
From: rishibamba
This patch ensures setting of importance parameter of a flow
everywhere it should be. This also correct formatting issues
as per the coding guidelines.
Signed-off-by: Rishi Bamba
---
include/openflow/openflow-1.1.h | 4 ++--
lib/learn.c | 1 +
lib/learning
From: rishibamba
This patch adds a test case for testing the replace-flows after addition
of importance parameter in a rule in accordance with OpenFlow Specs 1.4.
Signed-off-by: Rishi Bamba
---
tests/ovs-ofctl.at | 25 +
1 file changed, 25 insertions(+)
diff --git a
to DESIGN and added test cases
Signed-off-by: Rishi Bamba
---
DESIGN.md | 7 +--
tests/ovs-ofctl.at | 29 +
2 files changed, 34 insertions(+), 2 deletions(-)
diff --git a/DESIGN.md b/DESIGN.md
index 6f8d090..bd0ed27 100644
--- a/DESIGN.md
+++ b/DESIGN.md
From: rishibamba
This patch enhances the diff-flows & replace-flows CLI that takes into
account the addition of importance parameter in a rule as per the
OpenFlow Specs 1.4.
Signed-off-by: Rishi Bamba
---
utilities/ovs-ofctl.c | 8
1 file changed, 8 insertions(+)
diff --g
From: rishibamba
This patch enables a user to set importance for a new rule via add-flow
OF1.1+ in the OVS and display the same via dump-flows command OF1.1+ .
The changes are made in accordance with OpenFlow 1.4 specs to implement
Eviction on the basis of "importance".
Signed-off
k You
Regards
Rishi Bamba
- Original Message -
From: "Ben Pfaff"
To: "Rishi Bamba"
Cc: dev@openvswitch.org, "partha datta" , "deepankar
gupta"
Sent: Monday, October 27, 2014 9:14:10 PM
Subject: Re: [PATCH] ovs-ofctl : Setting of importance p
the same.Hope this
clarifies.
Rishi
Thank You
Regards
Rishi Bamba
- Original Message -
From: "Ben Pfaff"
To: "Rishi Bamba"
Cc: dev@openvswitch.org, "partha datta" , "deepankar
gupta"
Sent: Saturday, October 11, 2014 2:04:18 AM
Subject: Re:
provided,the flow will be considered permanent
else if set than evictable which defaults to the Zero value only.
Also attached is the file which contains all the 5 patches for your reference.
Thank You
Regards
Rishi Bamba
- Original Message -
From: "Rishi Bamba"
To: &q
This patch ensures setting of importance parameter of a flow
everywhere it should be. This also correct formatting issues
as per the coding guidelines.
Signed-off-by: Rishi Bamba
---
include/openflow/openflow-1.1.h | 4 ++--
lib/learn.c | 1 +
lib/learning-switch.c
replace-flows after addition
of importance parameter in a rule in accordance with OpenFlow Specs 1.4.
Signed-off-by: Rishi Bamba
---
tests/ovs-ofctl.at | 25 +
1 file changed, 25 insertions(+)
diff --git a/tests/ovs-ofctl.at b/tests/ovs-ofctl.at
index c6990f8..564c480
Hi Ben,
---
This patch enhances the diff-flows & replace-flows CLI that takes into
account the addition of importance parameter in a rule as per the
OpenFlow Specs 1.4.
Signed-off-by: Rishi Bamba
---
utilities/ovs-ofctl.c | 8
1 file changed, 8 insertions(+)
diff --git a/utili
importance for a new rule via add-flow
OF1.1+ in the OVS and display the same via dump-flows command OF1.1+ .
The changes are made in accordance with OpenFlow 1.4 specs to implement
Eviction on the basis of "importance".
#Changes made to DESIGN and added test cases
Signed-off-by: Rishi Bamba
-
for a new rule via add-flow
OF1.1+ in the OVS and display the same via dump-flows command OF1.1+ .
The changes are made in accordance with OpenFlow 1.4 specs to implement
Eviction on the basis of "importance".
Signed-off-by: Rishi Bamba
---
include/openflow/openflow-1.1.h |
Hi All,
I just need to mention and get a confirmation on the process we are following
for contribution to the OpenvSwitch i.e in line with contributing document of
the Dev forum.This is mainly how we are using GIT for the same.
1.
Fork the repository from the main OVS official
re
This patch enables a user to set importance for a new rule via add-flow
OF1.1+ in the OVS and display the same via dump-flows command OF1.1+ .
The changes are made in accordance with OpenFlow 1.4 specs to implement
Eviction on the basis of "importance".
Signed-off-by: Rishi Bamba
--
ne.
Note: On finalizing the the above mentioned approach we will be submitting the
patch for the same at the earliest. Need your confirmation on the above
working.
Thank You
Regards
Rishi Bamba
0-9899863386
=-=-=
Notice: The information contained in this e-mail
Hi All,
We are working on the implementation of eviction on the basis of importance in
accordance to the OpenFlow Specs 1.4.Also we are using Open vSwitch 2.3.0
stable version downloaded from
"http://openvswitch.org/releases/openvswitch-2.3.0.tar.gz"; for development.
The following changes h
Hi All,
I am facing a compilation issue on making a change in openflow-1.0.h struct
"ofp10_flow_mod"(adding a new parameter) , though the ASSERT value has been
correctly modified for the struct and build has passed the assert code block.
The following error is faced currently :
"""
PYTHONP
31 matches
Mail list logo
