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
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
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
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
--
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
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 |
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
-
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
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
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
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
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:
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
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
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 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
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
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
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
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.
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
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
}
> > 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,
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
> &
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
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
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
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
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
31 matches
Mail list logo