[ovs-dev] Compilation issue on adding a new parameter

2014-09-10 Thread Rishi Bamba
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

[ovs-dev] Addition of importance parameter in flow_mod structure for implementation of eviction according to OF1.4

2014-09-12 Thread Rishi Bamba
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

Re: [ovs-dev] Addition of importance parameter in flow_mod structure for implementation of eviction according to OF1.4

2014-09-22 Thread 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

[ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-09-26 Thread Rishi Bamba
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 --

[ovs-dev] Understanding the contribution model of OVS with GIT

2014-09-30 Thread 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

Re: [ovs-dev] [PATCH 1/4] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-10-10 Thread 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 |

[ovs-dev] [PATCH 2/4] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-10-10 Thread Rishi Bamba
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 -

[ovs-dev] [PATCH 3/4] ovs-ofctl:Enhanced diff-flows & replace-flows(OF14)

2014-10-10 Thread 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

[ovs-dev] [PATCH 4/4] ovs-ofctl:Enhanced diff-flows & replace-flows(OF14)

2014-10-10 Thread Rishi Bamba
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

[ovs-dev] [PATCH] ovs-ofctl : Setting of importance parameter

2014-10-27 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH] ovs-ofctl : Setting of importance parameter

2014-10-27 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH 2/4] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-10-27 Thread Rishi Bamba
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:

Re: [ovs-dev] [PATCH] ovs-ofctl : Setting of importance parameter

2014-10-30 Thread Rishi Bamba
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

[ovs-dev] [PATCH 3/5] ovs-ofctl:Enhanced diff-flows & replace-flows(OF14)

2014-10-31 Thread Rishi Bamba
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

[ovs-dev] [PATCH 2/5] ovs-ofctl:Changes made to DESIGN and added test cases(OF14)

2014-10-31 Thread Rishi Bamba
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

[ovs-dev] [PATCH 1/5] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-10-31 Thread Rishi Bamba
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

[ovs-dev] [PATCH 4/5] ovs-ofctl:Test Case for replace-flows(OF14)

2014-10-31 Thread Rishi Bamba
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

[ovs-dev] [PATCH 5/5] ovs-ofctl : Setting of importance parameter

2014-10-31 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH 1/5] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-05 Thread Rishi Bamba
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

[ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-05 Thread Rishi Bamba
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.

Re: [ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-07 Thread Rishi Bamba
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

[ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-07 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-10 Thread Rishi Bamba
} > > 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,

Re: [ovs-dev] [PATCH] ovs-ofctl:To set importance of a rule for eviction(OF14)

2014-11-11 Thread Rishi Bamba
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 > &

[ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2014-11-21 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-01-16 Thread Rishi Bamba
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

[ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-01-16 Thread Rishi Bamba
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

[ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-02-11 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-02-11 Thread Rishi Bamba
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

Re: [ovs-dev] [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-02-22 Thread Rishi Bamba
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

[ovs-dev] REMINDER: [PATCH] ovs-ofctl: Implementation of eviction on the basis of Importance

2015-03-21 Thread Rishi Bamba
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