On 11/1/23 17:05, Chuck Lever III wrote: > > >> On Nov 1, 2023, at 3:18 AM, Ilya Maximets <[email protected]> wrote: >> >> On 10/31/23 22:00, Chuck Lever III via discuss wrote: >>> Hi- >>> >>> I recently made some changes to tmpfs/shmemfs and someone has reported >>> an occasional memory leak, possibly when using ovs_vswitch.service. Can >>> I get someone to have a look at this report and perhaps make some >>> suggestions (or shoot down my working theory) ? >> >> Hi, Chuck. >> >> I looked at the bug, but the ovs-ctl script doesn't really do anything >> exceptional with tmpfs. It does use it though in a following way: >> >> 1. Create a couple of files with mktemp: >> >> https://github.com/openvswitch/ovs/blob/fdbf0bb2aed53e70b455eb1adcfda8d8278ea690/utilities/ovs-lib.in#L629 >> >> 2. Write some commands into these files, e.g.: >> >> https://github.com/openvswitch/ovs/blob/fdbf0bb2aed53e70b455eb1adcfda8d8278ea690/utilities/ovs-lib.in#L588 >> >> 3. Make them executable: >> >> https://github.com/openvswitch/ovs/blob/fdbf0bb2aed53e70b455eb1adcfda8d8278ea690/utilities/ovs-lib.in#L663 >> >> 4. Execute: >> >> https://github.com/openvswitch/ovs/blob/fdbf0bb2aed53e70b455eb1adcfda8d8278ea690/utilities/ovs-lib.in#L617 >> >> 5. Files are removed with 'rm -f' by an exit trap: >> >> https://github.com/openvswitch/ovs/blob/fdbf0bb2aed53e70b455eb1adcfda8d8278ea690/utilities/ovs-lib.in#L632 >> >> Also, ovsdb-server process is using the most basic version of a >> temporary file in its runtime, created by tmpfile(3). But this >> process is not even restarted by the ovs-vswitchd restart. > > Ilya, your response is very much appreciated. Would you mind > if I copied it to 218039 ?
Sure. No problem. The list is actually public: https://mail.openvswitch.org/pipermail/ovs-discuss/2023-November/052783.html (bugs@ is an alias for ovs-discuss@) Best regards, Ilya Maximets. _______________________________________________ discuss mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
