touch-packages team mailing list archive

[Alexandros Frantzis <alexandros.frantzis@xxxxxxxxxxxxx>]

** Attachment added: "valgrind/massif run of Mir client with the Mesa Mir EGL fix applied"
   https://bugs.launchpad.net/mir/+bug/1418486/+attachment/4313100/+files/massif.out.fixed

** Also affects: mesa (Ubuntu)
   Importance: Undecided
       Status: New

** Changed in: mesa (Ubuntu)
     Assignee: (unassigned) => Alexandros Frantzis (afrantzis)

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to mesa in Ubuntu.
https://bugs.launchpad.net/bugs/1418486

Title:
  Mir Mesa EGL platform leaks memory with every frame

Status in Mir:
  New
Status in mesa package in Ubuntu:
  New

Bug description:
  The Mir Mesa EGL platform leaks memory with every frame. This is the
  result of incorrect handling of the buffer objects we create from the
  incoming Mir buffers.

  The problem is exhibited in the attached 'massif.out.leak' file.  An
  excerpt from the result of ms_print for this file:

  --------------------------------------------------------------------------------
    n       time(ms)         total(B)   useful-heap(B) extra-heap(B)    stacks(B)
  -------------------------------------------------------------------------------- 
   23         11,593        4,965,432        4,503,871       461,561            0
   24         12,058        4,971,256        4,509,023       462,233            0
   25         12,640        4,978,536        4,515,463       463,073            0
   26         13,222        4,985,832        4,521,903       463,929            0
   27         13,804        4,993,112        4,528,343       464,769            0
   28         14,386        5,000,392        4,534,783       465,609            0
   29         14,735        5,004,760        4,538,647       466,113            0
   30         15,435        5,013,384        4,546,326       467,058            0
   31         16,016        5,020,776        4,552,815       467,961            0
   32         16,780        5,030,360        4,561,279       469,081            0
   33         17,163        5,035,144        4,565,511       469,633            0

  I have attached a patch (fix-mir-mesa-egl-gbm-bo-leak.diff) to get
  some wider review before creating a final patch for the ubuntu
  package. With this patch the leak is fixed as shown in the attached
  'massif.out.fixed' file:

  --------------------------------------------------------------------------------
    n       time(ms)         total(B)   useful-heap(B) extra-heap(B)    stacks(B)
  --------------------------------------------------------------------------------
   56         11,787        4,859,080        4,410,063       449,017            0
   57         11,920        4,859,080        4,410,063       449,017            0
   58         12,053        4,859,080        4,410,063       449,017            0
   59         12,186        4,859,080        4,410,063       449,017            0
   60         12,320        4,859,080        4,410,063       449,017            0
   61         12,453        4,859,080        4,410,063       449,017            0
   62         12,586        4,859,080        4,410,063       449,017            0
   63         12,719        4,859,080        4,410,063       449,017            0
   64         12,852        4,859,080        4,410,063       449,017            0
   65         12,985        4,859,080        4,410,063       449,017            0

To manage notifications about this bug go to:
https://bugs.launchpad.net/mir/+bug/1418486/+subscriptions