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.

HTH.

Best regards, Ilya Maximets.
_______________________________________________
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss

Reply via email to