On Wed, 27 Jul 2016 19:29:58 +0900 Takashi YAMAMOTO <[email protected]> wrote:
> On Wed, Jul 27, 2016 at 11:35 AM, Shinpei Muraoka <[email protected] >> wrote: > >> This patch will revert the API of NXAction for the following report. >> https://bugs.launchpad.net/dragonflow/+bug/1599367 >> revert commit id: 3c3896fd77f274ce953dcb78795aa5cc4fde6c65 >> >> This patch reverts recently introduced start and end arguments. >> Add the utility method for ofs_nbits. >> NXActionRegLoad now takes ofs_nbits argument instead of ofs >> and nbits even with this revert. >> > > is this necessary? > > FYI, there's unmerged dragonflow patch which uses NXActionRegLoad. > https://review.openstack.org/#/c/309746/9/dragonflow/controller/metadata_service_app.py@255 Thanks for the pointer. IIRC the discussion, there are some NXAction takes ofs_nbits instead of ofs and nbits. So taking ofs_nbits is more consistent with the rests? We could add a workaround for this. ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports.http://sdm.link/zohodev2dev _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
