Hello All, Thanks for taking the time to look into this. I will submit a patch later today, it's literally a one line change. I was looking into the OF1.3 spec and if I understand this correctly, I can't mix and match oxm and nxm fields because of the experimenter flag at the beginning of the OF header. If I need to do a IP header match on nw_src, nw_dst and nw_ttl, I would also need entries in oxm_fields to handle the NXM_OF_IP_SRC and NXM_OF_IP_DST code points.
I could add those and test them before I submit the patch if anyone else needs it. Thank you, -(James) Shridatt Sugrim -----Original Message----- From: Iwase Yusuke [mailto:[email protected]] Sent: Wednesday, September 28, 2016 4:10 AM To: [email protected] Cc: [email protected]; Sugrim, Shridatt <[email protected]> Subject: Re: [Ryu-devel] Match on ttl Hi Fujita-San, Either is fine. Because now we are testing the patch for the other missing OXMs including NXM_NX_IP_TTL. We will post it in a few days. Thanks, Iwase On 2016年09月28日 16:24, FUJITA Tomonori wrote: > On Tue, 27 Sep 2016 09:12:27 +0900 > Iwase Yusuke <[email protected]> wrote: > >> Hi, >> >> Thank you for submitting your patch. We very welcome your contributing! >> You can add your code to Ryu by sending your patch to this mailing list. >> >> https://github.com/osrg/ryu/blob/master/CONTRIBUTING.rst#submitting-a-change >> >> For the NXM match fields, nicra_ext.py is the appropriate place to add. >> Also, for the doc for NXM, please refer to the following. >> >> http://ryu.readthedocs.io/en/latest/nicira_ext_ref.html#module-ryu.ofproto.nicira_ext >> >> FYI, I made a patch for NXM_NX_IP_TTL before. >> (Sorry, I have not post this as a formatted patch...) >> https://www.mail-archive.com/[email protected]/msg12147.html > > Better to merge the above change into the mainline? >
smime.p7s
Description: S/MIME cryptographic signature
------------------------------------------------------------------------------
_______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
