Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Commits

Ticket #6734: Memory leak in EIT scanner (UK DVB-T)

  

 
MythTV commits RSS feed   Index | Next | Previous | View Threaded


mythtv at cvs

Jul 17, 2009, 10:49 PM

Post #1 of 25 (2485 views)
Permalink
Ticket #6734: Memory leak in EIT scanner (UK DVB-T)
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
There have been reports from some users of constant mythbackend memory
growth on the stable release when EIT scanning is enabled. My production
backend is no exception and I see VSZ memory growth of ~8MiB/day when
active EIT scanning is enabled.

I've finally gotten around to running valgrind on latest 0.21-fixes, and
have attached the valgrind log to this ticket, having been run with
options:

{{{
valgrind --leak-check=full --show-reachable=yes --verbose [[BR]]
--log-file=/tmp/valgrind-021-fixes-0453.log [[BR]]
/usr/local/bin/mythbackend --noupnp -v important,general,eit,channel
[[BR]]
> /tmp/mythbackend-valgrind-021-fixes.log
}}}

UPNP was disabled on the backend, and only one of the 3 cards (all KWorld
DVB Xperts) has active EIT scanning enabled. Build details (source was
exported from a checkout at r20947) are:

{{{
MythTV Version : exported
MythTV Branch : branches/release-0-21-fixes
Library API : 0.21.20080304-1
Network Protocol : 40
Options compiled in:
linux debug use_hidesyms using_oss using_alsa using_backend [[BR]]
using_dvb using_frontend using_iptv using_ivtv using_lirc [[BR]]
using_opengl_vsync using_opengl_video using_v4l using_valgrind [[BR]]
using_x11 using_xrandr using_xv using_xvmc using_xvmcw [[BR]]
using_xvmc_vld using_bindings_perl using_bindings_python [[BR]]
using_opengl using_ffmpeg_threads using_live
}}}


The summary of the leaktest (valgrind was run for approx 1 hour) was:

{{{
==20475== LEAK SUMMARY:
==20475== definitely lost: 2,276 bytes in 31 blocks.
==20475== indirectly lost: 302,980 bytes in 8,410 blocks.
==20475== possibly lost: 2,829,856 bytes in 40 blocks.
==20475== still reachable: 3,387,744 bytes in 56,158 blocks.
}}}

Multiplying up the ~300KiB/hr lost memory over a 24hr period would seem to
agree well with my observed memory growth of ~8MiB/day. I don't know if
this is an allowable 'leap' to make.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Nov 14, 2009, 1:43 PM

Post #2 of 25 (2313 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by stuarta):

* status: new => infoneeded_new


Comment:

Does this still happen with the eit scanner in 0.22?

Stuart

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:1>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Nov 14, 2009, 4:07 PM

Post #3 of 25 (2312 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

I'm going to have to monitor my DVB-S/T equipped dev machine (running
0.22-fixes) for several days to check - my production systems are still on
0.21-fixes until I find the time for OS-upgrades.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:2>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 13, 2010, 11:50 AM

Post #4 of 25 (2206 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by robertm):

Nick,

Any update on this? Ready to close it no info.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:3>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 13, 2010, 12:05 PM

Post #5 of 25 (2201 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

Replying to [comment:3 robertm]:
> Nick,
>
> Any update on this? Ready to close it no info.

Apologies for the lack of update on this ticket. I've recently had the
test machine on 24/7 (purely EIT collection from DVB-T and DVB-S) but
hadn't actually cron'ed the monitoring...

I will post any useful findings once I've left it running for some days (a
week seems sensible?) with monitoring enabled - but early indications
suggest it is not growing like it was on 0.21-fixes (if at all). If I do
see some growth, I'll try and get a valgrind trace as before.

Please don't close for the time being :)

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:4>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 19, 2010, 9:36 AM

Post #6 of 25 (2079 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

Replying to [comment:3 robertm]:
> Nick,
>
> Any update on this? Ready to close it no info.

I've now been monitoring my 0.22-fixes-based DVB-T/S "test" machine for
the past 5 days to monitor VSZ growth. I am observing a steady increase in
VSZ usage, and the machine has only been used lightly for a handful of
scheduled recordings from DVB-S. Memory growth has increased almost
linearly, from approx 350MiB to almost 390MiB in those 5 days.

I have some more DVB-S recordings scheduled for the remainder of the week,
but the weekend is free and I will endeavour to run valgrind to see if I
see the same issue as with 0.21-fixes. I will also get a 7-day graph of
the increase and post a link later today/tomorrow.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:5>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 20, 2010, 2:34 AM

Post #7 of 25 (2060 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by stuarta):

I've been graphing my backend memory use for over 1yr.
What i have noticed is that the memory usage will increase
for a few days (anything up to a week) before reaching a
steady state.

Can you monitor for longer than a week without restarting
the backend? I expect it won't grow any more after a week.


Stuart

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:6>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 20, 2010, 8:51 AM

Post #8 of 25 (2047 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

Replying to [comment:6 stuarta]:
> I've been graphing my backend memory use for over 1yr.
> What i have noticed is that the memory usage will increase
> for a few days (anything up to a week) before reaching a
> steady state.
>
> Can you monitor for longer than a week without restarting
> the backend? I expect it won't grow any more after a week.

Sure. The current monitoring session started on 2010-01-15, so I'll keep
it going over the weekend before it resumes its scheduled recordings next
week to get more than a week's stats.

I have noticed occasional plateaus during the memory growth on my
0.21-fixes backend (original reason for ticket), but memory growth always
resumes and appears to continue to rise until I restart the mythbackend
service. It never plateaus permanently.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:7>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Jan 26, 2010, 5:29 PM

Post #9 of 25 (1961 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

Replying to [comment:6 stuarta]:
> I've been graphing my backend memory use for over 1yr.
> What i have noticed is that the memory usage will increase
> for a few days (anything up to a week) before reaching a
> steady state.
>
> Can you monitor for longer than a week without restarting
> the backend? I expect it won't grow any more after a week.

12 days and still growing (0.22-fixes, not trunk). VSZ graph here:

[http://www.insidethex.co.uk/mythtv/mbe-mem-usage-2010-01-26-2w.png]

Will update to trunk and/or run valgrind at the weekend if required.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:8>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Feb 26, 2010, 4:33 AM

Post #10 of 25 (1853 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by stuartm):

* status: infoneeded_new => new


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:9>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 8, 2010, 5:09 AM

Post #11 of 25 (1705 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

@stuarta

Just wondering whether:

i) the recent commits to fix memory leaks will have touched upon the
issues valgrind seems to have highlighted w.r.t. this ticket?; and

ii) whether you need any more information?

(I've not been well recently and have not yet updated to 0.23RCx)

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:10>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 9, 2010, 6:43 AM

Post #12 of 25 (1690 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - EIT | Version: 0.21-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by janne):

[24004] needs backporting to 0.22-fixes,

but I would hope it's at least fixed in the sense that a 30 hour run with
eit scanning and a couple of recordings in trunk reported 0 bytes
"definitely lost"

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:11>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 10, 2010, 3:42 PM

Post #13 of 25 (1671 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: 0.23
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by stuartm):

* status: new => infoneeded_new
* version: 0.21-fixes => 0.22-fixes
* milestone: unknown => 0.23


Comment:

Last report was against 0.22-fixes.

Waiting to find out if this occurs with 0.23/trunk

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:12>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 17, 2010, 8:48 AM

Post #14 of 25 (1613 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: 0.23
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

Have updated to trunk and will try to test in next few days

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:13>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 19, 2010, 4:13 PM

Post #15 of 25 (1575 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: infoneeded_new
Priority: minor | Milestone: 0.23
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------

Comment(by Nick Morrott <knowledgejunkie (at) gmail (dot) com>):

This evening I left valgrind running on a fresh checkout of trunk at
r24200 for about 3 hours. This is on a Fedora 11 box with 1 DVB-T card and
1 DVB-S card. EIT is enabled for both cards.

mythbackend details:

{{{
MythTV Version : exported
MythTV Branch : trunk
Network Protocol : 56
Library API : 0.23.20100417-1
QT Version : 4.6.2
Options compiled in:
linux debug use_hidesyms using_oss using_alsa using_pulse
using_pulseoutput using_backend using_dvb using_frontend using_hdpvr
using_iptv using_ivtv using_lirc using_mheg using_opengl_video
using_opengl_vsync using_qtdbus using_qtwebkit using_v4l using_valgrind
using_x11 using_xrandr using_xv using_xvmc using_xvmc_vld using_xvmcw
using_bindings_perl using_bindings_python using_opengl
using_ffmpeg_threads using_live using_mheg
}}}

The summary of the leaktest (valgrind was run for approx 3 hours) was:

{{{
==30155== LEAK SUMMARY:
==30155== definitely lost: 116 bytes in 1 blocks.
==30155== possibly lost: 962,816 bytes in 46,889 blocks.
==30155== still reachable: 4,914,033 bytes in 23,232 blocks.
==30155== suppressed: 0 bytes in 0 blocks.
}}}

The definitely lost record was:

{{{
==30155== 116 bytes in 1 blocks are definitely lost in loss record 362 of
475
==30155== at 0x4004E5C: calloc (vg_replace_malloc.c:397)
==30155== by 0x650D783: my_thread_init (in
/usr/lib/mysql/libmysqlclient_r.so.16.0.0)
==30155== by 0x6506BA4: mysql_server_init (in
/usr/lib/mysql/libmysqlclient_r.so.16.0.0)
==30155== by 0x6532438: mysql_init (in
/usr/lib/mysql/libmysqlclient_r.so.16.0.0)
==30155== by 0x401FDCD: (within
/usr/lib/qt4/plugins/sqldrivers/libqsqlmysql.so)
==30155== by 0x5C8D190: QSqlDatabase::open() (in
/usr/lib/libQtSql.so.4.6.2)
==30155== by 0x5482386: MSqlDatabase::OpenDatabase() (mythdbcon.cpp:87)
==30155== by 0xCBFFCD: clone (in /lib/libc-2.10.2.so)
}}}

Most of the "probably lost" memory was:

{{{
==30155== 958,124 bytes in 46,863 blocks are possibly lost in loss record
474 of 475
==30155== at 0x4006F3D: malloc (vg_replace_malloc.c:207)
==30155== by 0x5E393EC: qMalloc(unsigned int) (in
/usr/lib/libQtCore.so.4.6.2)
==30155== by 0x5E6C2C6: QMapData::node_create(QMapData::Node**, int,
int) (in /usr/lib/libQtCore.so.4.6.2)
==30155== by 0x46FA6A7: QMap<unsigned int, unsigned long
long>::node_create(QMapData*, QMapData::Node**, unsigned int const&,
unsigned long long const&) (qmap.h:428)
==30155== by 0xCBFFCD: clone (in /lib/libc-2.10.2.so)
}}}


The full valgrind log is attached.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:14>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Apr 28, 2010, 11:08 AM

Post #16 of 25 (1419 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: 0.23
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by robertm):

* status: infoneeded_new => new


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:15>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

May 10, 2010, 9:12 AM

Post #17 of 25 (1337 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: minor | Milestone: 0.23-fixes
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by stuarta):

* milestone: 0.23 => 0.23-fixes


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:16>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Aug 3, 2010, 8:30 AM

Post #18 of 25 (975 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: new
Priority: critical | Milestone: 0.23-fixes
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by robertm):

* priority: minor => critical


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:17>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Aug 3, 2010, 8:30 AM

Post #19 of 25 (977 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: critical | Milestone: 0.23-fixes
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by robertm):

* status: new => assigned


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:18>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Aug 4, 2010, 12:59 AM

Post #20 of 25 (972 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: critical | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Mlocked: 0
-----------------------------------------------------------------+----------
Changes (by stuarta):

* milestone: 0.23-fixes => 0.24


--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:19>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Sep 10, 2010, 5:00 AM

Post #21 of 25 (740 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: critical | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Resolution:
Keywords: memory leak eit scanner | Ticket locked: 0
-----------------------------------------------------------------+----------

Comment (by Nick Morrott <knowledgejunkie <at> gmail <dot> com>):

I left trunk @ r26195 running for a few hours collecting EIT data from a
single DVB-T card. Valgrind is still reporting leaks:

{{{
==2566== LEAK SUMMARY:
==2566== definitely lost: 4,372 bytes in 82 blocks.
==2566== indirectly lost: 2,063,760 bytes in 6,349 blocks.
==2566== possibly lost: 1,284,804 bytes in 13,092 blocks.
==2566== still reachable: 4,591,551 bytes in 9,648 blocks.
==2566== suppressed: 0 bytes in 0 blocks.
}}}

Full valgrind log is attached.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:20>
MythTV <http://www.mythtv.org/>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Sep 11, 2010, 6:53 AM

Post #22 of 25 (732 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: critical | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Resolution:
Keywords: memory leak eit scanner | Ticket locked: 0
-----------------------------------------------------------------+----------

Comment (by stuarta):

(In [26228]) Refs #6734. Create RecordingProfile destructor.
Cleans up the objects new'd in the constructor.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:21>
MythTV <http://www.mythtv.org/>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Sep 11, 2010, 7:46 AM

Post #23 of 25 (736 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: minor | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Resolution:
Keywords: memory leak eit scanner | Ticket locked: 0
-----------------------------------------------------------------+----------
Changes (by stuarta):

* priority: critical => minor


Comment:

The leak is now more like a small dribble. Downgrading severity.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:22>
MythTV <http://www.mythtv.org/>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Sep 11, 2010, 8:03 AM

Post #24 of 25 (738 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: assigned
Priority: minor | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Resolution:
Keywords: memory leak eit scanner | Ticket locked: 0
-----------------------------------------------------------------+----------

Comment (by stuarta):

(In [26231]) Refs #6734. Backports [26228]. Add RecordingProfile
destructor.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:23>
MythTV <http://www.mythtv.org/>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits


mythtv at cvs

Sep 13, 2010, 1:57 PM

Post #25 of 25 (694 views)
Permalink
Re: Ticket #6734: Memory leak in EIT scanner (UK DVB-T) [In reply to]
#6734: Memory leak in EIT scanner (UK DVB-T)
-----------------------------------------------------------------+----------
Reporter: Nick Morrott <knowledgejunkie (at) gmail (dot) com> | Owner: stuarta
Type: defect | Status: closed
Priority: minor | Milestone: 0.24
Component: MythTV - EIT | Version: 0.22-fixes
Severity: medium | Resolution: fixed
Keywords: memory leak eit scanner | Ticket locked: 0
-----------------------------------------------------------------+----------
Changes (by stuarta):

* status: assigned => closed
* resolution: => fixed


Comment:

The changes already made against this ticket fix the majority
of the leak as triggered by the EIT scanner. The problems lie
primarily within the RecordingProfile class which is being
used both within the backend and the frontend.

Fixing the leaks further would cause segfaults in the frontend
due to double free's. To fix it properly requires a complete
refactor of the RecordingProfile classes to separate out the
UI components from the data components.

As this lies outside of the EIT scanner, i'm closing this ticket.
I'll open a separate one for what remains.


Stuart

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6734#comment:24>
MythTV <http://www.mythtv.org/>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

MythTV commits RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.