{"push_id":2877555683,"size":41,"distinct_size":0,"ref":"refs/heads/dpdk-hwol","head":"949758946767ff79b4c3eb5eca755c6cf21643e3","before":"28533f58e829eae213e76cca2798967805069709","commits":[{"sha":"3988f56ac4f2fac718d320ba2eb5eb8955cca82d","author":{"name":"Ilya Maximets","email":"e6f59449982e74b9a8013e88011af79c397f66d7@samsung.com"},"message":"vswitch.xml: Fix key type and description style of tc-policy.\n\nThe set of supported values specified.\nStyle fixed to look good in man page. Fixed indents.\n\nCC: Paul Blakey <paulb@mellanox.com>\nFixes: 691d20cbdcf3 (\"other-config: Add tc-policy switch to\n                      control tc flower flag\")\nSigned-off-by: Ilya Maximets <i.maximets@samsung.com>\nSigned-off-by: Simon Horman <simon.horman@netronome.com>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/3988f56ac4f2fac718d320ba2eb5eb8955cca82d"},{"sha":"fc253483d6c874295c1a9eb7a86f44d2fe4f57ee","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"nx-match: Avoid double-free on some error paths.\n\nReported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=9966\nReported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=9968\nFixes: f1eb32b9641c (\"ofp-group: Fix memory leak in error cases parsing group requests.\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/fc253483d6c874295c1a9eb7a86f44d2fe4f57ee"},{"sha":"3ee9b38d5cd6c7b5dc29a5854432ffb77aa4f7db","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ofp-actions: Re-fix error path for parsing OpenFlow actions.\n\nA previous commit attempted to fix the error path when the actions nested\nwithin clone provoked an error.  However, this commit just introduced a new\nproblem in another case, since it made ofpacts_pull_openflow_actions__()\nrestore a previously valid pointer to data that might have been\nreallocated.\n\nThis commit takes another approach.  Instead of trying to restore anything\nat all, it just defines ofpacts_pull_openflow_actions__() to clear the\noutput buffer when there's an error.  It seems that this is less error\nprone.  Most of the callers don't care; this commit fixes up the ones that\ndo.\n\nReported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=9975\nFixes: 20cdd1dbd546 (\"ofp-actions: Avoid assertion failure for clone(ct(...bad actions...)).\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nTested-by: Yifeng Sun <pkusunyifeng@gmail.com>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/3ee9b38d5cd6c7b5dc29a5854432ffb77aa4f7db"},{"sha":"db1dcb235feef8a5e13b26dc25705585fc5c325d","author":{"name":"Darrell Ball","email":"5cb751bd4da145a3789c9f4291568adbd95b709c@gmail.com"},"message":"datapath: Fix builds on older kernels.\n\nOn older kernels, for example 3.19, the function rt6_get_cookie() is\nnot available and used with ipv6 config enabled;  it was introduced in\n4.2.  Put back the replacement function if it does not exist.\nAdd a 3.19 version to travis.\n\nCC: Yifeng Sun <pkusunyifeng@gmail.com>\nFixes: bf61b8b1c1db (\"datapath: Add support for kernel 4.16.x & 4.17.x.\")\nSigned-off-by: Darrell Ball <dlu998@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nTested-by: Yifeng Sun <pkusunyifeng@gmail.com>\nReviewed-by: Yifeng Sun <pkusunyifeng@gmail.com>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/db1dcb235feef8a5e13b26dc25705585fc5c325d"},{"sha":"2235714c653d19e33aee653e2fe7500693fe5935","author":{"name":"Markos Chandras","email":"33d8b43ece1558bd62b361ce5e954c5d9a97a7a2@suse.de"},"message":"utilities: Drop shebang from bash completion script\n\nThis fixes the following warning when building Open vSwitch on the\nopenSUSE Build Service:\n\n  W: non-executable-script /usr/share/bash-completion/completions/ovs-appctl-bashcomp.bash\n  This text file contains a shebang or is located in a path dedicated\n  for executables, but lacks the executable bits and cannot thus be\n  executed. If the file is meant to be an executable script, add the\n  executable bits, otherwise remove the shebang or move the file\n  elsewhere.\n\nThe file is meant to be sourced instead of executed, so we can simply\ndrop the shebang.\n\nSigned-off-by: Markos Chandras <mchandras@suse.de>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/2235714c653d19e33aee653e2fe7500693fe5935"},{"sha":"e9b33ad780f3bc712a5de6be9e1e0803fadcd249","author":{"name":"Greg Rose","email":"85cf315691926e39e448eb0e637daca40a5fb312@gmail.com"},"message":"compat: Remove unused function\n\nThe compat function rpl_nf_conntrack_in() does not appear to be used\nanywhere and emits warnings as such during builds < 4.10.\n\nThe patch passes Travis:\n\nhttps://travis-ci.org/gvrose8192/ovs-experimental/builds/423097292\n\nRemove it.\n\nSigned-off-by: Greg Rose <gvrose8192@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/e9b33ad780f3bc712a5de6be9e1e0803fadcd249"},{"sha":"d0db81eac89e0b0304c6e468c9c4d590414181d8","author":{"name":"Justin Pettit","email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@ovn.org"},"message":"dpif-netdev: Don't check if xcalloc() failed when creating meter.\n\nxcalloc() can't return null.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nAcked-by: Flavio Leitner <fbl@sysclose.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/d0db81eac89e0b0304c6e468c9c4d590414181d8"},{"sha":"866bc7567ac5f6cf6ee52f6e3aa1253766898cf6","author":{"name":"Justin Pettit","email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@ovn.org"},"message":"dpif-netdev: Prevent unsafe access when retrieving meter stats.\n\ndpif_netdev_meter_get() retrieved a pointer to a meter entry without\nholding a lock.  It's possible that another thread could have deleted\nthat entry between retrieving the pointer and dereferencing the pointer.\nThis makes the function hold the lock the entire time the meter entry is\nneeded.\n\nFound by inspection.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nAcked-by: Flavio Leitner <fbl@sysclose.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/866bc7567ac5f6cf6ee52f6e3aa1253766898cf6"},{"sha":"71972fcaa2aa25d546f47078d9257797eb79eb86","author":{"name":"Bhargava Shastry","email":"9e48dd0716fc6d23ad906c8cecce1aca0d531a41@sect.tu-berlin.de"},"message":"ossfuzz: Improve coverage of flow_extract_target\n\nThis commit improves test coverage of the ossfuzz flow extract test harness\nby extending the harness with additional API calls from lib/flow.c\n\nAn additional minor change is adding a config option to\nflow_extract_target.options file in `tests/ossfuzz/config` to suppress\ndebug output while fuzzing.\n\nA cursory evaluation shows that the patch covers 8 additional files and\nimproves line coverage of lib/flow.c from 23% to 37%.\n\nSigned-off-by: Bhargava Shastry <bshastry at sect.tu-berlin.de>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/71972fcaa2aa25d546f47078d9257797eb79eb86"},{"sha":"9e8c33133c2b0ef6c49a07da4395483f23aff090","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: openvswitch-kmod-fedora.spec.in file bug fix\n\nPatch 22c33c303932 used /usr/src/linux/<kernel version> as path of the\nlinux headers, which does not work for SLES. Use /lib/modules/<kernel\nversion>/build instead.\n\nFixes 22c33c303932 (rhel: support kmod build against mulitple kernel versions,\nfedora)\n\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nCC: Markos Chandras <mchandras@suse.de>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Flavio Leitner <fbl@sysclose.org>\nReviewed-by: Markos Chandras <mchandras@suse.de>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/9e8c33133c2b0ef6c49a07da4395483f23aff090"},{"sha":"56bfa517d9c8face982efc852644c72e1b5b1564","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: use full path for /sbin/depmod\n\nFixes: 22c33c303932 (rhel: support kmod build against mulitple kernel versions, fedora)\nFixes: c8cd1307b248 (rhel: support kmod build against multiple 7.2 kernels, rhel6)\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Flavio Leitner <fbl@sysclose.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/56bfa517d9c8face982efc852644c72e1b5b1564"},{"sha":"eac9f26661aeb3bf565a4730e04f1f05f44bf444","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: allow passing more flags to configure, fedora\n\nDefine a variable _ovs_config_extra_flags to allow passing more flags to\nconfigure when building OVS kmod RPM. For example, to build with a\nnon-standard openssl and add an RPATH, use the following command\n\nmake rpm-fedora-kmod RPMBUILD_OPT='-D \"_ovs_config_extra_flags\n--with-openssl=<path to your openssl header> LDFLAGS=\\\"\\${LDFLAGS} -Xlinker\n-rpath=<path to your openssl lib>\\\"\"'\n\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nCC: Flavio Leitner <fbl@sysclose.org>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Flavio Leitner <fbl@sysclose.org>\nReviewed-by: Markos Chandras <mchandras@suse.de>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/eac9f26661aeb3bf565a4730e04f1f05f44bf444"},{"sha":"697f5993c0cf437faa7ec6753e8cf47850a44da3","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovs-macros: Make tests log how long they waited when they succeed.\n\nMany OVS tests wait up for 10 seconds for a condition to become true.\nUsually these conditions are ones that should take only a second or so to\nactually become true in practice, but on a busy and slow machine it's\npossible that some tests might fail or come close to failing because 10\nseconds is simply not enough there.\n\nThis commit adds logging for the case where a condition actually succeeds\nto indicate the amount of time that was waited.  This should make it easier\nto identify whether we need to increase the maximum wait time from 10\nseconds to something longer, by allowing us to see whether some of the\nsuccessful waits came close to timing out.\n\nReported-by: Thomas Goirand <zigo@debian.org>\nReported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-September/047340.html\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nTested-by: Thomas Goirand <zigo@debian.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/697f5993c0cf437faa7ec6753e8cf47850a44da3"},{"sha":"c3570519ecaf653d2b13ec8ed572d75396f7cc98","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: add 4.4 kernel in kmod build with mulitple versions, fedora\n\nExtends 22c33c303932 (rhel: support kmod build against mulitple kernel\nversions, fedora) to kernel version 4.4.x, x>=73 for SLES 12 SP3\n\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nCC: Markos Chandras <mchandras@suse.de>\nCC: Ben Pfaff <blp@ovn.org>\nReviewed-by: Greg Rose <gvrose8192@gmail.com>\nTested-by: Greg Rose <gvrose8192@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/c3570519ecaf653d2b13ec8ed572d75396f7cc98"},{"sha":"ed1e8ded84dcc490a8b4b815ef0fe9f5e25ab051","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"daemon-unix: Use same name for original or restarted children.\n\nLinux has an idea of process name that is visible in /proc/$pid/comm.  This\nis \"ovs-vswitchd\" for a freshly started ovs-vswitchd process.  When the\nmonitor code restarted a crash child, it changed it to the empty string.\nThis confused the daemon_is_running check in ovs-lib.in, which checks\ncomm.  This commit fixes the problem by setting the program name as comm\nin newly restarted children.\n\nVMware-BZ: #2191724\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Gurucharan Shetty <guru@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/ed1e8ded84dcc490a8b4b815ef0fe9f5e25ab051"},{"sha":"2600ef022e151976a6385abef838cbd87ff6005d","author":{"name":"Justin Pettit","email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@ovn.org"},"message":"ovn.at: Skip ACL rate-limiting test on slow/overloaded systems.\n\nIn ACL rate-limiting test, we send three sets of 100 packets.  One of\nthe sets drops packets at a rate of 10 per second, one at a rate of 5\nper second, and one not at all.  On my setup, it takes roughly 0.67\nseconds to send those 300 packets, but we have reports of it taking over\n15 seconds on others.  The test was intended to allow some flexibility\nin run-time, but it's very difficult to design a mechanism that can all\npossibilities.\n\nTo prevent false test failures, this patch changes the test to check\nthe duration count of the meter, and if it's greater than nine seconds,\njust skip the test.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nReported-by: Thomas Goirand <zigo@debian.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/2600ef022e151976a6385abef838cbd87ff6005d"},{"sha":"bc92e8a4788f63e4728d32f2d5baba080ede4004","author":{"name":"Xin Long","email":"ed9bd6f22d2a1e0b1507d3ddc300dc248265d19a@gmail.com"},"message":"erspan: set erspan_ver to 1 by default when adding an erspan dev\n\nUpstream commit:\n    commit 84581bdae9587023cea1d139523f0ef0f28bd88d\n    Author: Xin Long <lucien.xin@gmail.com>\n    Date:   Mon Aug 27 18:41:32 2018 +0800\n\n    erspan: set erspan_ver to 1 by default when adding an erspan dev\n\n    After erspan_ver is introudced, if erspan_ver is not set in iproute, its\n    value will be left 0 by default. Since Commit 02f99df1875c (\"erspan: fix\n    invalid erspan version.\"), it has broken the traffic due to the version\n    check in erspan_xmit if users are not aware of 'erspan_ver' param, like\n    using an old version of iproute.\n\n    To fix this compatibility problem, it sets erspan_ver to 1 by default\n    when adding an erspan dev in erspan_setup. Note that we can't do it in\n    ipgre_netlink_parms, as this function is also used by ipgre_changelink.\n\n    Fixes: 02f99df1875c (\"erspan: fix invalid erspan version.\")\n    Reported-by: Jianlin Shi <jishi@redhat.com>\n    Signed-off-by: Xin Long <lucien.xin@gmail.com>\n    Signed-off-by: David S. Miller <davem@davemloft.net>\n\nFixes: 5e720da59d (\"erspan: fix invalid erspan version.\")\nCc: Xin Long <lucien.xin@gmail.com>\nSigned-off-by: Greg Rose <gvrose8192@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/bc92e8a4788f63e4728d32f2d5baba080ede4004"},{"sha":"863fb61f84e764ea0322e2fa4162e280abf3cfa0","author":{"name":"Mark Michelson","email":"fd88180078559faac67d61aee549cbebded7c6e7@redhat.com"},"message":"ovn: Detect and prevent duplicate address assignments.\n\nThis patch alters the 'ovn-nbctl lsp-set-addresses' command to check if\nthe IP addresses being added are duplicates of already-set IP addresses.\nTest cases have been added for this detection.\n\nThis patch also adds a warning message to ovn-northd if duplicate IPv4\naddresses are detected on a switch.\n\nSigned-off-by: Mark Michelson <mmichels@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/863fb61f84e764ea0322e2fa4162e280abf3cfa0"},{"sha":"adf1b852301842a2d39ea363df816b6df183a76e","author":{"name":"Tiago Lam","email":"a5d814a3ba3c87e37a920688311be806714cc755@intel.com"},"message":"system-traffic: Fix conntrack per zone limit test.\n\nCommit 3f1087c added a per zone limit test which relied on the\nCHECK_CT_DPIF_FLUSH_BY_CT_TUPLE m4 macro to skip the test when executing\nin a userspace datapath (since the per zone limit feature is not yet\nimplemented in userspace). That macro, however, has been removed in\ncommit 271e48a (\"conntrack: Support conntrack flush by ct 5-tuple\")\nwhich was causing the test to fail when executing in userspace.\n\nInstead, a new m4 macro, CHECK_CT_DPIF_PER_ZONE_LIMIT, is introduced to\nmake the same differentiation, until userspace doesn't support the per\nzone limit.\n\nCC: Yi-Hung Wei <yihung.wei@gmail.com>\nFixes: 3f1087c (\"system-traffic: Add conntrack per zone limit test case\")\nSigned-off-by: Tiago Lam <tiago.lam@intel.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Yi-Hung Wei <yihung.wei@gmail.com>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/adf1b852301842a2d39ea363df816b6df183a76e"},{"sha":"70738f0b7815ca635b28ac554b2cb92c80df9d8d","author":{"name":"Darrell Ball","email":"5cb751bd4da145a3789c9f4291568adbd95b709c@gmail.com"},"message":"ovs-ctl: Allow add-remote without vswitchd started.\n\n'add_managers ()' is filtering add-remote if vswitchd is not started.\nHowever, if we actually filter here we end up with a bricked system,\nblackholing all traffic.  Allowing add_manager() to proceed may mean\nextra churn in controllers in some cases, but this is far better than\nthe alternative of a bricked system.\n\nSigned-off-by: Darrell Ball <dlu998@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/openvswitch/ovs/commits/70738f0b7815ca635b28ac554b2cb92c80df9d8d"}]}
{"actor":{"display_login":"istokes"}}