xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 5 years ago

Closed 4 years ago

Last modified 16 months ago

#1662 closed defect (worksforme)

server memory leak

Reported by: nathan-renniewaldock Owned by: nathan-renniewaldock
Priority: critical Milestone: 2.2
Component: server Version: trunk
Keywords: Cc:


It looks like there's a memory leak somewhere. After running firefox for 19 days, xpra was using over 8GB RAM.

$ ps -eo rss,etime,cmd | grep :64
8912696 19-06:49:25 /usr/bin/python /usr/bin/xpra --bind-tcp= --no-daemon --tcp-auth=file:filename=/home/nathan/.winswitch/server/sessions/64/session.pass --systemd-run=no start :64
38588 19-06:49:23 /usr/lib/xorg/Xorg-for-Xpra-:64 -noreset -novtswitch -nolisten tcp +extension GLX +extension RANDR +extension RENDER -auth /home/nathan/.Xauthority -logfile /run/user/1002/xpra/Xorg.:64.log -configdir /home/nathan/.xpra/xorg.conf.d -config /etc/xpra/xorg.conf -depth 24 :64

$ xpra info :64 | grep memory

Ubuntu 17.04 x64, xpra 2.1.2-r16903

Currently running glxgears with XPRA_DETECT_LEAKS=1. Anything else I can do to help track this down?

Attachments (2)

memleak-examples.patch (1.1 KB) - added by Antoine Martin 5 years ago.
example of patch to enable memleak debugging for the classes that seemed to cause problems
leak-show-lists.patch (2.2 KB) - added by Antoine Martin 5 years ago.
show the lists that leak and their backref (applies to r17356)

Download all attachments as: .zip

Change History (14)

comment:1 Changed 5 years ago by Antoine Martin

Owner: changed from Antoine Martin to nathan-renniewaldock

Can you try turning off as many features as you can (sound forwarding, etc) to see if that helps?
Do you need to have any screen activity to trigger it? Does the window have to be shown? Or does it leak no matter what?
Reproducing with glxgears would help.
It would also be useful to see if using mmap (local connection) still leaks.
Another interesting test would be to run the server with XPRA_SCROLL_ENCODING=0 xpra start ... and see if that helps.

Last edited 5 years ago by Antoine Martin (previous) (diff)

comment:2 Changed 5 years ago by Antoine Martin

Owner: changed from nathan-renniewaldock to Antoine Martin
Status: newassigned

I can reproduce it with gtkperf -a in a loop.

comment:3 Changed 5 years ago by Antoine Martin

Watching the server memory usage with xpra info | grep server.maxrss=, then running ./tests/xpra/test_apps/simulate_console_user.py in an xterm, the value goes up regularly by about ~0.2 to 2KB/s.
This also happens with mmap.
When re-connecting with a new client, the increase only occurs after the memory usage has reached the point where it left off when the previous client disconnected.

First had to fix memleak debugging (XPRA_DETECT_MEMLEAKS=1 xpra start ..) which broke with (newer versions?) numpy: r17300. (r17302 also helps debugging)

Then found a leak in the protocol layer, so "xpra info" would leak yet more memory when I was trying to find where the real leak was... fixed in r17299.
And then another leak in the window source class fixed in r17301.

Both of those should be backported.
I'll let it run for a few hours more to try to see if there are more leaks to be found..

Changed 5 years ago by Antoine Martin

Attachment: memleak-examples.patch added

example of patch to enable memleak debugging for the classes that seemed to cause problems

comment:4 Changed 5 years ago by Antoine Martin

Priority: majorcritical

There are still some small leaks, so:

  • r17307 improves leak debugging, can generate graphs using objgraph
  • r17306 + r17308: potential leaks of protocol instances - not sure if these should be backported (a little bit too intrusive)
  • r17309, r17310: code refactoring, r17311: minor bug fix

What makes this particularly difficult is that the leak debugging slows things down dramatically and blocking the main thread, so it can cause things to get backed up so much that they look like leaks when they're not.

Another problem is the "traceback reference cycle problem"

(Exception leaks in Python 2 and 3).

And more importantly, we're still leaking somewhere as this gets printed every time the leak detection code runs (always exactly the same leak count):

leaks: count : object
      15 :                             cell :   1469 matches
      14 :                            tuple :   4117 matches
      13 :                            frame :   1017 matches
       2 :                             list :   4145 matches

comment:5 Changed 5 years ago by Antoine Martin

By turning off the ping feature, the leaks are reduced. It also looks like generating network traffic (ie: moving the mouse around) also causes more leaking.

I suspect that this comes from the non-blocking socket timeouts, like this shown at debug level:

    <bound method SocketConnection.is_active of unix-domain socket:/run/user/1000/xpra/desktop-3>, \
    <bound method SocketConnection.can_retry of unix-domain socket:/run/user/1000/xpra/desktop-3>, \
    <built-in method recv of _socket.socket object at 0x7f6ab0585b90>, \
    (65536,), {}) timed out, retry=socket.timeout
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/net/bytestreams.py", line 101, in untilConcludes
    return f(*a, **kw)
timeout: timed out

comment:6 Changed 5 years ago by Antoine Martin

Lots of related changes:

The main leak is still there though...

Last edited 5 years ago by Antoine Martin (previous) (diff)

comment:7 Changed 5 years ago by Antoine Martin

r17328 (+r17330 fixup) fixes a leak caused by logging.
The alternative fix would be to add a kwargs option to not track the loggers when we know we're not going to be re-using them.

Dumping all the cell objects (matched by type string since there does not seem to be a python type exposed for it), the recurring entries seem to be:

2017-11-06 17:31:17,250 [355] '<cell at 0x7f248ea09fa0: list object at 0x7f2486445518>': '[{\'__setattr__\': <slot wrapper \'__setattr__\ ..  124: <type \'set\'>}, (VideoSubregion(None),)]'
2017-11-06 17:31:17,250 [356] '<cell at 0x7f248ea09ef8: type object at 0x7f24bb122c60>': "<type 'frame'>"
2017-11-06 17:31:17,250 [357] '<cell at 0x7f248ea09e18: tuple object at 0x55c31868b020>': '(<frame object at 0x7f24bb20c790>, <frame objec .. 5c319abbd50>, <frame object at 0x7f2470003610>)'
2017-11-06 17:31:17,250 [358] '<cell at 0x7f248ea09ec0: dict object at 0x7f248e514050>': "{1: <type 'list'>, 4: <type 'cell'>}"

Not sure where they're from yet... could even be the leak debugging code itself.

Last edited 5 years ago by Antoine Martin (previous) (diff)

comment:8 Changed 5 years ago by Antoine Martin

Left "xpra info" running in a loop for 4 hours and those leaks are definitely gone.
However, gtkperf -a still causes another leak - and a pretty big one. At least now we can measure things without causing further misleading leaks:

  • r17332: more leak avoidance in exception handling
  • r17333: more thorough and reliable cleanup of window-source objects
  • r17334: dumps all known frames on SIGUSR2

Changed 5 years ago by Antoine Martin

Attachment: leak-show-lists.patch added

show the lists that leak and their backref (applies to r17356)

comment:9 Changed 5 years ago by Antoine Martin

More improvements:

  • r17349 switch to using pympler for object tracking
  • r17356 do leak detection in a thread (so we don't lock up the server)
  • r17352 use structured data for "xpra info" response (flatten client side)
  • r17350: don't records fds if we don't need them
  • r17345 + r17347: keep track of timers, and cancel them when no longer needed (ie: during cleanup)
  • r17348 + r17351: avoid using closures
  • r17346: make object tracking more legible

This is hard...

comment:10 Changed 5 years ago by Antoine Martin

Owner: changed from Antoine Martin to nathan-renniewaldock
Status: assignednew

Related improvements: r17358 + r17360: avoid churn

I think the leaks are gone (at least the big ones), it just takes a very long time for the maxrss value to settle on its high water mark, probably because of memory fragmentation.

It would be worth playing with MALLOC_MMAP_THRESHOLD_ to validate this assumption, but I've already spent far too much time on this ticket.

@nathan-renniewaldock: can I close this?

comment:11 Changed 4 years ago by Antoine Martin

Resolution: worksforme
Status: newclosed

comment:12 Changed 16 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1662

Note: See TracTickets for help on using tickets.