{"push_id":2467156620,"size":274,"distinct_size":274,"ref":"refs/heads/conntrack","head":"e56cae20f2bbf5563775eb6d4e353317085c436c","before":"974d9cd9a6465d7bcd72a38ab3dc660940396745","commits":[{"sha":"186667a83c2b09ed9ae08b35c596987cf7d33cfb","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"classifier: Fix typo in comment.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/186667a83c2b09ed9ae08b35c596987cf7d33cfb"},{"sha":"10087cba9deec95aaea080c49f2cbe648ebe92c8","author":{"name":"Ciara Loftus","email":"b3d6c0da41dcf0b8333ab29bfd5c6eca84b2ba45@intel.com"},"message":"netdev-dpdk: Add support for vHost dequeue zero copy (experimental)\n\nZero copy is disabled by default. To enable it, set the 'dq-zero-copy'\noption to 'true' when configuring the Interface:\n\novs-vsctl set Interface dpdkvhostuserclient0\noptions:vhost-server-path=/tmp/dpdkvhostuserclient0\noptions:dq-zero-copy=true\n\nWhen packets from a vHost device with zero copy enabled are destined for\na single 'dpdk' port, the number of tx descriptors on that 'dpdk' port\nmust be set to a smaller value. 128 is recommended. This can be achieved\nlike so:\n\novs-vsctl set Interface dpdkport options:n_txq_desc=128\n\nNote: The sum of the tx descriptors of all 'dpdk' ports the VM will send\nto should not exceed 128. Due to this requirement, the feature is\nconsidered 'experimental'.\n\nTesting of the patch showed a ~8% improvement when switching 512B\npackets between vHost devices on different VMs on the same host when\nzero copy was enabled on the transmitting device.\n\nSigned-off-by: Ciara Loftus <ciara.loftus@intel.com>\nAcked-by: Ilya Maximets <i.maximets@samsung.com>\nSigned-off-by: Ian Stokes <ian.stokes@intel.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/10087cba9deec95aaea080c49f2cbe648ebe92c8"},{"sha":"46ab60bfe562f3ee6cde94888930e9ef9130831f","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"classifier: Refactor interface for classifier_remove().\n\nUntil now, classifier_remove() returned either null or the classifier rule\npassed to it, which is an unusual interface.  This commit changes it to\nreturn true if it succeeds or false on failure.\n\nIn addition, most of classifier_remove()'s callers know ahead of time that\nit must succeed, even though most of them didn't bother with an assertion,\nso this commit adds a classifier_remove_assert() function as a helper.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nTested-by: Yifeng Sun <pkusunyifeng@gmail.com>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/46ab60bfe562f3ee6cde94888930e9ef9130831f"},{"sha":"efefbcae0119b1a9d224e80ca39040c6aed58246","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ofp-actions: Make formatting and parsing functions take a struct argument.\n\nAn upcoming commit will add another parameter for parsing and formatting\nactions.  It is much easier to add these parameters if they are\nencapsulated in a struct, so this commit first makes that change.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/efefbcae0119b1a9d224e80ca39040c6aed58246"},{"sha":"a203f653b1a6a2544603be00966b8d4731d91b69","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ofp-util: New data structure for mapping between table names and numbers.\n\nThis shares the infrastructure for mapping port names and numbers.  It will\nbe used in an upcoming commit.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/a203f653b1a6a2544603be00966b8d4731d91b69"},{"sha":"4bc938ccb3124282c873084a7d11cd8bc37e27a7","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"Support accepting and displaying table names in OVS tools.\n\nOpenFlow has little-known support for naming tables.  Open vSwitch has\nsupported table names for ages, but it has never used or displayed them\noutside of commands dedicated to table manipulation.  This commit adds\nsupport for table names in ovs-ofctl.  When a table has a name, it displays\nthat name in flows and actions, so that, for example, the following:\n    table=1, arp, actions=resubmit(,2)\nmight become:\n    table=ingress_acl, arp, actions=resubmit(,mac_learning)\ngiven appropriately named tables.\n\nFor backward compatibility, only interactive ovs-ofctl commands by default\ndisplay table names; to display them in scripts, use the new --names\noption.\n\nThis feature was inspired by a talk that Kei Nohguchi presented at Open\nvSwitch 2017 Fall Conference.\n\nCC: Kei Nohguchi <kei@nohguchi.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/4bc938ccb3124282c873084a7d11cd8bc37e27a7"},{"sha":"500db308e2704fa9fc5ff69cdcd9cb3b22116e99","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"util: Document and rely on ovs_assert() always evaluating its argument.\n\nThe ovs_assert() macro always evaluates its argument, even when NDEBUG is\ndefined so that failure is ignored.  This behavior wasn't documented, and\nthus a lot of code didn't rely on it.  This commit documents the behavior\nand simplifies bits of code that heretofore didn't rely on it.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/500db308e2704fa9fc5ff69cdcd9cb3b22116e99"},{"sha":"9a3cf0ac3397555b38acd6ed7fed6e7eea5c0335","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovs-vswitchd: Fire RCU callbacks before exit to reduce memory leak warnings.\n\novs-vswitchd makes extensive use of RCU to defer freeing memory past the\nlatest time that it could be in use by a thread.  Until now, ovs-vswitchd\nhas not waited for RCU callbacks to fire before exiting.  This meant that\nin many cases, when ovs-vswitchd exits, many blocks of memory are stuck in\nRCU callback queues, which valgrind often reports as \"possible\" memory\nleaks.\n\nThis commit adds a new function ovsrcu_exit() that waits and fires as many\nRCU callbacks as it reasonably can.  It can only do so for the thread that\ncalls it and the thread that calls the callbacks, but generally speaking\novs-vswitchd shuts down other threads before it exits anyway, so this is\npretty good.\n\nIn my testing this eliminates most valgrind warnings for tests that run\novs-vswitchd.  This ought to make it easier to distinguish new leaks that\nare real from existing non-leaks.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: William Tu <u9012063@gmai.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/9a3cf0ac3397555b38acd6ed7fed6e7eea5c0335"},{"sha":"93f558421aadcd4fc0ff758bd61612c23747c72a","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovs-vswitchd: Avoid or suppress memory leak warning for glibc aio.\n\nThe asynchronous IO library in glibc starts threads that show up as memory\nleaks in valgrind.  This commit attempts to avoid the warnings by flushing\nall the asynchronous I/O to the log file before exiting.  This only does\npart of the job for glibc since it keeps the threads around for some\nundefined idle time before killing them, so in addition this commit adds a\nvalgrind suppression to stop displaying these warnings in any case.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: William Tu <u9012063@gmai.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/93f558421aadcd4fc0ff758bd61612c23747c72a"},{"sha":"0cca6870a9d5d0a8197d52c19cd24bd3cf3852d9","author":{"name":"Han Zhou","email":"7464aa2e5354010f6d29fa7343786b966ce7e966@gmail.com"},"message":"ovn-nbctl: update manpage for lsp-set-type.\n\nSigned-off-by: Han Zhou <zhouhan@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/0cca6870a9d5d0a8197d52c19cd24bd3cf3852d9"},{"sha":"18b5bd4b31215f73c71ac30bae2274e3c9865bed","author":{"name":"Han Zhou","email":"7464aa2e5354010f6d29fa7343786b966ce7e966@gmail.com"},"message":"ofctrl: Remove unused declaration.\n\nSigned-off-by: Han Zhou <zhouhan@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/18b5bd4b31215f73c71ac30bae2274e3c9865bed"},{"sha":"0506f184c53e90ef8abf7da5ea0d82ca92bbea22","author":{"name":"Huanle Han","email":"f7de45528064ea524b2667aa2c122e689a4d7ee8@gmail.com"},"message":"xlate: fix packets loopback caused by duplicate read of xcfgp.\n\nSome functions, such as xlate_normal_mcast_send_mrouters, test xbundle\npointers equality to avoid sending packet back to in bundle. However,\nxbundle pointers port from different xcfgp for same port are inequal.\nThis may lead to the packet loopback.\n\nThis commit stores xcfgp on ctx at first and always uses the same xcfgp\nduring one packet process period.\n\nSigned-off-by: Huanle Han <hanxueluo@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/0506f184c53e90ef8abf7da5ea0d82ca92bbea22"},{"sha":"96f2c290c1cd4aa18191d6adb93cfce3e56b18a7","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"Merge branch 'dpdk_merge' of https://github.com/istokes/ovs into HEAD","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/96f2c290c1cd4aa18191d6adb93cfce3e56b18a7"},{"sha":"fdfd3f7b01ef23c8b527d9e0a63d962945e7c7fd","author":{"name":"William Tu","email":"6aed7d52531456413f7e5a799f9150cf02425e08@gmail.com"},"message":"learn: improve test case\n\nCurrent learn test cases use only ovs-ofctl add/del flows.\nThe patch add a new test case for learn with delete_learned and\nlimit option enabled.\n\nSigned-off-by: William Tu <u9012063@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/fdfd3f7b01ef23c8b527d9e0a63d962945e7c7fd"},{"sha":"da1af6e279a438a0b5bf854731b5b3546d3a2eaf","author":{"name":"Venkata Anil","email":"a9fbb981dc3c21313fc8579c83c81e225ec0177b@redhat.com"},"message":"Add unixctl option for ovn-northd\n\nSigned-off-by: Venkata Anil <vkommadi@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/da1af6e279a438a0b5bf854731b5b3546d3a2eaf"},{"sha":"c9ccdec7516560cc859e90ab87557600fc9bdf58","author":{"name":"Ashish Varma","email":"60d1860bd0e0b8f5ce20904bbcf556ed4f1c463d@gmail.com"},"message":"doc: Added OVS Conntrack tutorial\n\nOVS supports connection tracker related match fields and actions.\nAdded a tutorial to demonstrate the basic use cases for some of these\nmatch fields and actions.\n\nSigned-off-by: Ashish Varma <ashishvarma.ovs@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/c9ccdec7516560cc859e90ab87557600fc9bdf58"},{"sha":"53d416af570934a92d8c2f6471229599e53cbe6f","author":{"name":"William Tu","email":"6aed7d52531456413f7e5a799f9150cf02425e08@gmail.com"},"message":"ovs-router: fix router entry cast\n\nThe offsetof(struct ovs_router_entry, cr) should always be 0,\nthus the else statement should never be reached.\n\nSigned-off-by: William Tu <u9012063@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/53d416af570934a92d8c2f6471229599e53cbe6f"},{"sha":"d1ba7d54c2b917694bbe255ad890b0140ec9c9b9","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"tests: Make OVS_WAIT_UNTIL and OVS_WAIT_WHILE failures easier to debug.\n\nUntil now, when OVS_WAIT_UNTIL or OVS_WAIT_WHILE ran, little information\nwas available: usually nothing at all in the log, unless the wait failed,\nin which case there was a line number.  This commit adds a note saying\nwhat is being waited for in any case, and a message saying that the wait\nfailed if it does.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nTested-by: Yifeng Sun <pkusunyifeng@gmail.com>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/d1ba7d54c2b917694bbe255ad890b0140ec9c9b9"},{"sha":"5ee33cbdceb042d18416900061329f25e04f98a4","author":{"name":"Guoshuai Li","email":"e8e45cafcfe2b289e0683ee2e9c0bdb11516d5c7@dtdream.com"},"message":"ovn-nbctl: Add QoS commands.\n\nThis patch provides the command line to add/delete/list QoS rule on the\nlogical switch.\n\nSigned-off-by: Guoshuai Li <ligs@dtdream.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/5ee33cbdceb042d18416900061329f25e04f98a4"},{"sha":"98f2d36d2394bfa6f9717fc5c91db2921092464f","author":{"name":"Frode Nordahl","email":"daf0624e8606551c41ff9f027049d1a6c1c00f23@gmail.com"},"message":"debian: Do not modify pre-existing defaults file\n\nCurrently, on installation or upgrade the openvswitch-switch deb package\nwill in some circumstances modify a pre-existing\n/etc/default/openvswitch-switch configuration file.\n\nThis does not play well with modeling and configuration management tools\nand may lead to unnecessary restarts of the openvswitch-switch service\nafter the initial restart done as part of the package upgrade. As\nrestarting the openvswitch-switch affects the datapath this is\nsomething we should try to avoid.\n\nI also believe the current behaviour to be in conflict with best practices\nset out in the config files section of the\n[Debian Policy](https://www.debian.org/doc/debian-policy/#s-config-files).\n\nThis commit addresses this by removing the part of the postinst script\nthat attempts to append missing documentation parts of the template\nand leaves the installed defaults file alone when it exists.\n\nFixes: 0aaa379d99f4 (\"Debian packaging: Add several new settings to /etc/default/openflow-switch.\")\nSigned-off-by: Frode Nordahl <frode.nordahl@gmail.com>\nReported-at: https://github.com/openvswitch/ovs-issues/issues/137\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/darball/ovs_master/commits/98f2d36d2394bfa6f9717fc5c91db2921092464f"}]}
{"actor":{"display_login":"darball"}}