Hi Padma,
I have described my patch as part of the mail, let me quote it. Please find
the reference to the OF specs as part of the quote.
###### Quote
This is with regards to the patch that I sent with subject "[PATCH 1/2]
simple_switch_13.py: Added ability to use buffer_id in FlowMod" (the second
one is just fixing a pep8 warning).
In the attached packet capture you can see at #52-55 that there is only the
FlowMod sent to the switch and no longer a second "PacketOut" packet, which
should be more efficient.
This is in accordance with the OF v1.3.1 specs A.3.4.1 page 65.
## Quote ##
If no buffered packet is associated with the flow mod, it must be set to
OFP_NO_BUFFER. A flow mod that includes a valid buffer_id is effectively
equivalent to sending a two-message sequence of a flow mod and a packet-out
to OFPP_TABLE, with the requirement that the switch must fully process the
flow mod before the packet out.
## / Quote ##
I hope this patch is up to your standards, I really had some painful
happenings with git and all the toolchain until it finally allowed me to
send mails in the format you wish to see.
###### / Quote
Benny
On Tue, Jan 20, 2015 at 6:28 PM, Padma Jayasankar <[email protected]>
wrote:
> Hi,
> One basic doubt. Most of the simple_switch apps have this part of code
> # install a flow to avoid packet_in next time
> if out_port != ofproto.OFPP_FLOOD:
> match = parser.OFPMatch(in_port=in_port, eth_dst=dst)
> # verify if we have a valid buffer_id, if yes avoid to send
> both
> # flow_mod & packet_out
> if msg.buffer_id != ofproto.OFP_NO_BUFFER:
> self.add_flow(datapath, 1, match, actions, msg.buffer_id)
> return
> else:
> self.add_flow(datapath, 1, match, actions)
> data = None
> if msg.buffer_id == ofproto.OFP_NO_BUFFER:
> data = msg.data
>
> In this they check *msg.bufferid*. When will the value of this be
> ofproto.OFP_NO_BUFFER:. What is its significance?
> If the message has valid bufferid the packetout message is not sent? why
> is it so? Please clarify
>
> Thanks and Regards,
> Padma V
>
>
> ------------------------------------------------------------------------------
> New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
> GigeNET is offering a free month of service with a new server in Ashburn.
> Choose from 2 high performing configs, both with 100TB of bandwidth.
> Higher redundancy.Lower latency.Increased capacity.Completely compliant.
> http://p.sf.net/sfu/gigenet
> _______________________________________________
> Ryu-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ryu-devel
>
>
------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel
