Wednesday, January 25th, 2012, 00:16 UTC | ||
[00:16:23] | vallor (vallor!~Ponzo@pdpc/supporter/monthlygold/vallor) has joined #mythtv | |
[00:17:26] | dissent (dissent!~Ponzo@cto.vpn.sonic.net) has quit (Ping timeout: 248 seconds) | |
[00:22:44] | Seeker`: | skd5aner: On my dev test system, if you are watching TV then starting RAOP playback fails because the sound card is already in use |
[00:26:26] | Seeker`: | skd5aner: although that does mean someone cna prevent you getting sound from anything else by keeping a connection open |
[00:33:30] | wagnerrp: | Captain_Murdoch: i figure you're probably well versed in these kinds of issues, if youve got an opinion to add... |
[00:34:02] | wagnerrp: | in the autoexpirer code, we're casting a long long to a size_t, and then comparing it to a uint64_t |
[00:34:15] | wagnerrp: | prior to the manipulation, the long long is the correct value |
[00:35:00] | wagnerrp: | afterwards, according to problems exhibited by a user running 0.23 (although the code is the same in master), dropping below 8TB of free space causes it to incorrectly expire content |
[00:36:51] | stichnot (stichnot!chatzilla@nat/intel/x-qvuajasijithiwva) has quit (Quit: ChatZilla 0.9.88 [Firefox 9.0.1/20111220165912]) | |
[00:43:09] | stichnot (stichnot!~chatzilla@192.55.54.38) has joined #mythtv | |
[00:45:42] | stichnot: | jpabq_: regarding missing text, when running with "-v gui" I get a number of messages like this: |
[00:45:59] | stichnot: | (Layout) – 'channel': height overflow. line height 21 paragraph height 21, area height 20 |
[00:46:10] | stichnot: | also for the 'timedate' field |
[00:46:12] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (Quit: Leaving.) | |
[00:46:43] | stichnot: | but that doesn't seem related to the truncation of the description field (or whatever it's called) |
[00:47:12] | neufeld: | wagnerrp: size_t is 32-bits long on 32-bit wordsize systems |
[00:47:40] | jpabq_: | stichnot: if they are NOT multiline textareas, it should still draw the text, even though the theme author did not specify enough area height. However, if <multiline> is yes, then the text line would not be drawn. |
[00:48:22] | wagnerrp: | neufeld: so it seems... meaning our comparison is just outright broken for freespaces > 4TB |
[00:49:36] | neufeld: | wagnerrp: a C compiler can be compliant and allow only 16 bits unsigned for SIZE_MAX. On gcc, it is 32-bits for 32-bit wordsizes, you can see it in /usr/include/stdint.h |
[00:56:33] | ** neufeld tends not to trust "long long", "size_t", "<un-sign-qualified> char". Prefers to go straight for uint32_t, uint64_t as dictated by context ** | |
[00:57:30] | wagnerrp: | yeah, thats the current plan for a fix |
[00:58:20] | stichnot: | jpabq_: if I'm reading the theme file correctly, this title area has multiline=no and cutdown=yes, so it seems it should be drawing the text. The text is missing at 1280x720, but fully present and not cutdown at 640x360. |
[00:59:24] | wagnerrp: | neufeld: thanks |
[01:00:02] | jpabq_: | stichnot: that is very odd. Your screenshot makes not sense. The line that is not drawn seems to be for an episode with a missing subtitle, but the next line down also would appear to be for a missing subtitle (it shows the title instead, right?) |
[01:00:06] | neufeld: | wagnerrp: no worries, you did the hard bit |
[01:01:33] | stichnot: | jpabq_: have to head out now, but will respond here in ~2 hours. |
[01:01:54] | jpabq_: | sticknot, I tried downloading Blue Abstract from the theme server, but I seem to have a misconfig on my machine — it is not listing ANY of the themes from the server, just the locally installed themes. I have had that problem before, and it turned out to be a permission problem, but I can't remember on what directory…. |
[01:04:40] | sphery: | jpabq_: did you try MENU and tell it to show the remote themes? I /think/ (but haven't actually looked, so this is anecdotal evidence) that themes decide whether it shows remote themes initial or not |
[01:05:40] | jpabq_: | sphery: on the theme chooser screen? I don't see that option. |
[01:05:44] | stichnot (stichnot!~chatzilla@192.55.54.38) has quit (Ping timeout: 244 seconds) | |
[01:06:17] | jpabq_: | sphery: Weird, I just told it to download the available themes for the 10th time — this time it worked! |
[01:08:54] | sphery: | weird... glad you got it working, though |
[01:21:32] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[01:29:33] | jpabq_: | stuartm: I can reproduce sticknot's problem. Since the cause is probably the same as yours, that is a good thing. |
[01:33:58] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
[01:34:03] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Remote host closed the connection) | |
[01:37:51] | Seeker`: | markk: don't know if you're interested, but here is a patch to lookup the hostname of the connecting device (in anticipation of there being a UI for this in the future) http://paste.ubuntu.com/816033/ |
[01:39:42] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[01:42:45] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has quit (Quit: jpabq_) | |
[01:43:34] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[01:43:40] | iamlindoro: | Wouldn't that only work if the host was in local DNS or hosts? |
[01:44:34] | iamlindoro: | The hosts are already resolving the Frontend via bonjour/zeroconf, why wouldn't we just use that? |
[01:45:10] | Seeker`: | iamlindoro: I believe so. It will fall back to the ip address of the device if it doesn't manage to look it up, and after a quick search I couldn't find another way of getting the device name |
[01:45:43] | Seeker`: | I'm not overly familiar with bonjour/zeroconf |
[01:46:05] | iamlindoro: | it's the discovery protocol used by the RAOP code |
[01:47:42] | kth (kth!~kth@unaffiliated/kth) has quit (Quit: Leaving.) | |
[02:03:12] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
[02:05:52] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
[02:06:12] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
[02:12:35] | Seeker`: | iamlindoro: I'll have a look and try to work out how to do it with Bonjour. DNS might be good enough for a first pass though. |
[02:20:20] | jpabq_: | stuartm: Does not look like https://github.com/MythTV/mythtv/commit/a257f6915 is to blame. I reverted that commit, and it still happens. I was really wondering how that commit could cause this behavior. |
[02:21:00] | jpabq_: | stuartm: you are correct, in that it seems to be caused by the "text" being exactly the right size. If I add 5 pixels to the width, the problem goes away. |
[02:22:32] | beata (beata!beata@108.12.181.220) has quit (Ping timeout: 276 seconds) | |
[02:40:15] | Seeker`: | iamlindoro: Are you sure you can get the device's name without DNS using Bonjour? |
[02:40:27] | iamlindoro: | yes |
[02:40:35] | iamlindoro: | That's the whole point of Bonjour |
[02:41:42] | Seeker`: | Isn't the point of Bonjour the ability to broadcast 'There is service X available on port Y of IP address A.B.C.D' (i.e. you don't need to know the hostname in advance to set up the connection) |
[02:42:05] | iamlindoro: | No, that's just one capability of Zeroconf, name resolution is built in too |
[02:43:15] | iamlindoro: | Specifically, section 3.2 of the specification |
[02:54:42] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[02:56:19] | jflatt (jflatt!~jflatt@ip70-173-253-248.lv.lv.cox.net) has joined #mythtv | |
[03:04:57] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[03:27:31] | Captain_Murdoch: | stuartm, there's no limit to the # of threads that can be queued up, but only a certain number can run at one time and that's determined by the Qt maxthreads value. we could set a counter and if we exceed X number of background loads we could force load in the foreground. |
[03:31:14] | stichnot (stichnot!~chatzilla@192.55.55.41) has joined #mythtv | |
[03:42:00] | stichnot: | jpabq_: since you reproduced my problem, I won't bother trying the clarify the screenshot... |
[03:46:28] | jpabq_: | stichnot, stuartm: Pure luck that I was able to reproduce the problem. I just happened to have a title – subtitle that rendered at exactly the right number of pixels for it to show up. |
[03:46:42] | jpabq_: | stichnot: Please update, and let me know if it is fixed, for you. |
[03:48:02] | jpabq_: | Unfortunately, while tracking that down, I lost the work I had been doing for Shadow/Outline. Oh well, I know what needs to be done, just have to re-code it. |
[03:49:44] | jpabq_ is now known as abqjp | |
[03:52:15] | Computer_Czar (Computer_Czar!~Unknown@69.4.155.83) has joined #mythtv | |
[03:52:24] | Computer_Czar (Computer_Czar!~Unknown@69.4.155.83) has quit (Client Quit) | |
[03:55:28] | Czar_Away (Czar_Away!~Unknown@69.4.155.83) has quit (Ping timeout: 276 seconds) | |
[03:55:37] | stichnot: | Unfortunately, my instance of the problem is still there. MythTV Version : v0.25pre-4143-gdaa9f1d-dirty |
[03:57:37] | abqjp: | bummer. That commit fixed it for me, so I can no longer "reproduce" it. |
[04:18:45] | abqjp: | stichnot: I sent you an email. |
[04:27:20] | stichnot: | abqjp: thanks, resulting logs emailed. |
[04:31:27] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
[04:34:04] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 240 seconds) | |
[04:38:30] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[04:52:27] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[04:54:07] | abqjp: | stichnot: Sent you a new debug patch. |
[04:55:13] | abqjp: | I think there is a bug in Qt. When told to elide the text, the result does not always fit inside the requested area. |
[05:05:07] | stichnot: | abqjp: That did the trick. The text is properly cutdown now. |
[05:17:25] | stichnot: | abqjp: markk: if there is a Qt bug, I wouldn't be surprised if it's the same problem as in #8954, which led to PAD_WIDTH and PAD_HEIGHT in subtitlescreen.cpp. |
[05:23:59] | jpabq: | stichnot, When Qt is told to elide the text down to a given width, and the resulting width is checked, it is correct. However, when Qt is then told to take that elided text and fit it into a QTextLine of that same width, it does not fit. |
[05:26:50] | stichnot: | OK. That sounds a little different from #8954, where (iirc) the computed dimensions of a text string are simply wrong. |
[05:29:05] | jpabq: | It actually looks to me, like it does not fit by the size of the "..." . |
[05:36:30] | xris: | stuartm: I was talking to you about channel icon stuff about half a day ago, right? (long day, job stability mess/stress). |
[05:38:14] | xris: | Captain_Murdoch: any interest in adding your theme files and such to the ftp sync with osu? |
[05:41:31] | xris: | ok, I just moved the osuosl sync files from /opt/www/.…./htdocs/mc to a simpler /opt/ftp |
[05:41:51] | xris: | and no more confusion with the old/outdated hostname directory that only held ftp release files |
[05:41:56] | xris: | Beirdo: ^^^^ |
[05:42:58] | Beirdo: | nice |
[05:49:23] | jpabq: | stuartm, stichnot, please test latest master to make sure it is working for you. |
[05:50:51] | jflatt: | -v all should give me LOG(VB_GENERAL, LOG_DEBUG... right? |
[05:51:38] | jpabq: | jflatt, you would also need --loglevel debug |
[05:53:36] | jflatt: | thanks, that was key |
[05:55:09] | len__ (len__!~quassel@184-97-147-75.mpls.qwest.net) has joined #mythtv | |
[05:59:16] | abqjp (abqjp!~jpabq@mythtv/developer/jpabq) has quit (Ping timeout: 240 seconds) | |
[06:00:58] | stichnot: | I just scrolled through my list of 2000+ recordings. Plenty of cutdown, but no blank entries. |
[06:12:09] | sphery: | jpabq: Is this the case that was mentioned before where we're computing a width without using the exact string, so that when it's actually drawn kerning and such makes the actual width different from the value we computed? |
[06:13:48] | jpabq: | stichnot, thanks. Not how I really wanted to spend my evening, but at least something was accomplished. |
[06:14:51] | stichnot: | I hear you... thanks for the fix. |
[06:16:36] | jpabq: | sphery, I don't know. How recently was this discussion? Was it more than four months ago? If so, just about all the code was re-written. Before the conversion of MythUIText to use QTextLayout, "cutdown" was being done my out own code. Now we are using Qt's elided function to do it. |
[06:16:52] | jpabq: | s/out/our/ |
[06:17:58] | jpabq: | sphery, I believe what you are talking about pre-dates the conversion to QTextLayout. It was the issue that cause the cursor to be positioned wrong in textedits that used variable width fonts. |
[06:18:44] | sphery: | ah, yeah, it was before your changes |
[06:19:00] | sphery: | didn't know you had changed that, too |
[06:19:45] | jpabq: | Yeah. "fixing" multi-line text edits turned out to be a VERY invasive change to the text rendering system. |
[06:19:58] | cattelan is now known as cattelan_away | |
[06:21:13] | sphery: | hehe, sounds like for the better, though |
[06:22:27] | jpabq: | All of a sudden, stuartm has found a few "fringe" bugs in that code. It always amazes me how long bugs can sit before they are discovered. |
[06:45:12] | MythBuild: | build #1382 of master-debian-stable-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1382 blamelist: John Poet <jpoet@mythtv.org > |
[06:49:09] | jpabq: | ? |
[06:49:52] | iamlindoro: | It choked on tv_play.cpp |
[06:50:04] | iamlindoro: | which is the biggest memory/time hog in the whole compilation |
[06:50:06] | iamlindoro: | not your fault |
[06:52:06] | jpabq: | Good, because I am exhausted, and am going to bed. Thanks iamlindoro. |
[07:14:42] | stoffel (stoffel!~quassel@pD9E42CD1.dip.t-dialin.net) has joined #mythtv | |
[07:16:00] | len__ (len__!~quassel@184-97-147-75.mpls.qwest.net) has quit (Remote host closed the connection) | |
[07:17:04] | len_ (len_!~quassel@184-97-147-75.mpls.qwest.net) has joined #mythtv | |
[07:19:39] | davez0r (davez0r!c7687d39@gateway/web/freenode/ip.199.104.125.57) has quit (Quit: Page closed) | |
[07:28:38] | stoffel (stoffel!~quassel@pD9E42CD1.dip.t-dialin.net) has quit (Remote host closed the connection) | |
[07:38:23] | jflatt (jflatt!~jflatt@ip70-173-253-248.lv.lv.cox.net) has quit (Quit: Konversation terminated!) | |
[07:40:36] | Beirdo: | gah, speaking of exhausted... stupid me... spent all evening rationalizing nagios configs for 300ish servers... and I'm not done yet. |
[07:40:57] | Beirdo: | but tomorrow's another day. |
[07:57:15] | wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has quit (Quit: Leaving) | |
[08:05:18] | len_ (len_!~quassel@184-97-147-75.mpls.qwest.net) has quit (Remote host closed the connection) | |
[08:25:35] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
[08:28:51] | wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has joined #mythtv | |
[09:01:53] | JoeJulian (JoeJulian!~JoeJulian@gaealink.net) has quit (Read error: Operation timed out) | |
[09:07:43] | stuarta: | Beirdo: you heard of centreon? |
[09:37:39] | Peitolm (Peitolm!mlqU8V@mandlebrot.random-chaos.org.uk) has quit (Ping timeout: 245 seconds) | |
[09:38:33] | Peitolm (Peitolm!LUG1@mandlebrot.random-chaos.org.uk) has joined #mythtv | |
[09:50:28] | stuartm: | Captain_Murdoch: that might be easier than the solution I was considering – having a worker thread maintaining a queue of requests and farming them out to a fixed number of pooled threads |
[10:01:25] | knightr (knightr!~Nicolas@mythtv/developer/knightr) has quit (Ping timeout: 276 seconds) | |
[10:01:55] | knightr (knightr!~Nicolas@mythtv/developer/knightr) has joined #mythtv | |
[10:17:02] | danielk22 (danielk22!~danielk@121.246.154.93) has joined #mythtv | |
[10:17:45] | danielk22: | Any idea how to find deleted trac wiki pages? This is what I'm looking for: http://code.mythtv.org/trac/wiki/SimplifyingBackendConfiguration |
[10:18:03] | Cougar (Cougar!~cougar@kkk.version6.net) has quit (Ping timeout: 244 seconds) | |
[10:18:20] | danielk22: | I believe those were my thoughts on the HTTP backend config and I wanted to refresh my memory by following the link.. but it's dead. |
[10:19:30] | Cougar (Cougar!~cougar@kkk.version6.net) has joined #mythtv | |
[10:21:13] | danielk22: | Hmm, found this: http://code.mythtv.org/trac/wiki/TaskBrowserBasedSetup |
[10:37:10] | aloril (aloril!~aloril@84.249.126.153) has quit (Ping timeout: 244 seconds) | |
[10:39:01] | _davidjw_ (_davidjw_!~davidjw@gatea.kw.bbc.co.uk) has joined #mythtv | |
[10:49:34] | aloril (aloril!~aloril@dsl-tkubrasgw3-fe7ef900-153.dhcp.inet.fi) has joined #mythtv | |
[10:58:44] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (Ping timeout: 240 seconds) | |
[11:05:02] | mike|2 (mike|2!~mike@c-76-115-119-121.hsd1.or.comcast.net) has quit (Remote host closed the connection) | |
[11:05:55] | mike (mike!~mike@c-76-115-119-121.hsd1.or.comcast.net) has joined #mythtv | |
[11:06:21] | mike is now known as Guest90452 | |
[11:11:17] | sraue (sraue!~stephan@xbmc/staff/sraue) has joined #mythtv | |
[11:14:22] | stuarta: | danielk22: that was the original set of thoughts from before coding started. |
[11:14:30] | stuarta: | dunno if any of it is still valid |
[11:31:23] | stuartm: | danielk22: http://code.mythtv.org/trac/wiki/TaskSimplify . . . onfiguration |
[11:31:44] | stuartm: | danielk22: index of pages can be found here: http://code.mythtv.org/trac/wiki/TitleIndex |
[12:03:17] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has quit (Read error: Connection reset by peer) | |
[12:04:36] | danielk22: | stuartm: Thanks. I guess it got renamed without a forward link. I know it's out of date; I'm just trying to remind myself of some of the background. |
[12:04:43] | MythBuild: | build #1383 of master-debian-stable-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1383 |
[12:20:02] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has joined #mythtv | |
[12:21:37] | stuartm: | Captain_Murdoch: I think part of the problem is that we're using the ideal system thread count for just one instance of MThreadPool, is there any reason not to use the global instance of MThreadPool instead? Nowhere else do we create a new thread pool and it seems obvious that with multiple instances you can no longer judge the correct number of threads to run at a given time |
[12:22:26] | stuartm: | alternatively we limit both the global and image thread pools to half of IdealThreadCount() (for the frontend at least) |
[12:24:33] | stuartm: | the latter leaves us with a dedicated pool of threads to keep the UI snappy, although it seems to me that we have priorities for that very reason |
[12:33:59] | danielk22: | stuartm: If you use the global thread pool you don't have any guarantee it will start running in the next X minutes if you put it on the queue with start(). Someone else may have started some process in idealThreadPool() threads that blocks until it's finished copying 10 TB of data to a remote store. |
[12:34:33] | danielk22: | That doesn't use any CPU but it blocks another QRunnable from using the thread. |
[12:36:55] | stuartm: | true, so two thread pools but carve up the idealThreadCount between them? The goal being not to greatly exceed idealThreadCount by having two pools running at near full capacity |
[12:38:39] | danielk22: | stuartm: what exactly is the problem we're seeing? |
[12:38:46] | stuartm: | or better still we reserve some threads in the global pool for UI work – a minimum number dedicated for UI image loading and the rest we can call upon if available |
[12:40:22] | danielk22: | that could be implemented by just doing a tryStart on the global pool and then iff that fails do a start on the UI thread pool which has the reserved # of threads (with idealThreadCount() – reserved threads in the global pool.) |
[12:40:27] | stuartm: | danielk22: segfaults, all sorts of weird GL errors that are probably memory related if we fire off lots of http image loads e.g. in the icon downloader |
[12:41:06] | stuartm: | happens when using the following patch and doing a search for something with lots of results e.g. BBC – http://pastebin.com/KgJMhZgZ |
[12:42:50] | danielk22: | stuartm: I doubt that has anything to do with the number of threads.. It's probably just because that code is using methods like qApp->processEvents(); which is known to cause trouble. |
[12:42:57] | stuartm: | the backtraces are too big for pastebin, hundreds of image loading threads |
[12:44:46] | danielk22: | stuartm: There shouldn't be hundreds of threads unless we're using startReserved() with a time out.. so that's a problem, but 10,000 threads shouldn't cause segfaults, maybe just a slow system. |
[12:45:03] | stuartm: | danielk22: yeah, that's possible, but there still the theoretical problem of using IdealThreadCount for multiple instances of MThreadPool, that's supposed to be the ideal for system not each pool |
[12:46:23] | danielk22: | idealThreadCount() isn't magic it's just 2 * # cores. Even if we have two pools 4 * cores isn't hundreds unless you have a supercomputer. |
[12:48:06] | danielk22: | Most likely the icon loading thread is hogging the threadpool and using startReserved() rather than a normal start() so when it can't get all the threads it needs from the pool it's asking the threadpool to create more threads.. |
[12:48:40] | stuartm: | 310 threads in total, first 30 of which are nothing to do with image loading |
[12:50:16] | stuartm: | each thread looks pretty much like this – http://pastebin.com/b4KnKv84 |
[12:51:09] | stuartm: | well exactly like that, except for a change in addresses |
[12:53:51] | stuartm: | http://pastebin.com/cki7yukL < lower threads are further along |
[12:55:26] | stuartm: | this is a dual core system, so something has clearly gone wrong |
[12:59:50] | stuartm: | I'll be back in a bit, grabbing lunch |
[13:01:42] | bradd (bradd!~bradd@shaolin.ameri.ca) has joined #mythtv | |
[13:05:09] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
[13:05:30] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
[13:10:17] | stuartm: | I'll stick in some debugging to MThreadPool, see what it thinks the various total/running/available count is at any given moment |
[13:19:49] | jams: | stuartm, any idea what happened to HostImageSelect in setting.h? it seems to have been removed but can't find anything reference it in commit. It whas the combobox that had an image associated with each selection. |
[13:20:28] | jams: | one of my programs used it |
[13:20:43] | stuartm: | jams: it was only used for theme selection in mythfrontend (originally added for that purpose), it was removed when the theme selector was added |
[13:22:18] | jams: | thats what i thought. |
[13:22:34] | jams: | no direct replacment? |
[13:22:49] | stuartm: | no with the old ui/settings stuff |
[13:22:54] | stuartm: | s/no/not/ |
[13:23:14] | stuartm: | at the time it wasn't imagined that we'd still be using the old stuff :/ |
[13:23:37] | jams: | i could see that |
[13:24:22] | jams: | on that note xavierh does plan to continue work on migrating the settings. He had some personal stuff come up that needed his attention. |
[13:24:34] | jams: | hopefully his work will be used. |
[14:05:29] | Captain_Murdoch: | stuartm, I think we used our own thread pool so that image loads weren't held up by other threads potentially. |
[14:06:27] | stuartm: | yeah, I'm now thinking that something is just broken in the thread pool |
[14:07:03] | Captain_Murdoch: | I don't see anything in my ImageLoadThread that releases the thread, so it shouldn't be actually running more than 2xCores. |
[14:07:25] | Captain_Murdoch: | yeah, sounds like that might be a possibility |
[14:10:24] | danielk22: | stuartm: That's bizarre. The only place I see starting that runnable is this GetMythUI()->GetImageThreadPool()->start(bImgThread); That shouldn't allow for more threads to be created than idealThreadCount() |
[14:13:24] | stuartm: | like every bug I've run into lately I was trying to fix/improve something and I'm sidetracked into finding/fixing something else, it's getting a little annoying :) However if the end result is less buggy code I guess that's not so bad |
[14:15:26] | stuartm: | danielk22: hopefully the debugging I'm sticking into MThreadPool will shed some light |
[14:15:36] | danielk22: | stuartm: I'd look at MThreadPool::TryStartInternal().. It has three parts, deleting old threads, using available threads, and creating new threads. The last part (beginning at "if (reserved ||" would be where I'd suspect something is going wrong. |
[14:17:00] | danielk22: | It looks like we're incrementing m_reserve_thread event if reserved, I think that might be the issue... |
[14:18:00] | danielk22: | The "m_priv->m_reserve_thread++;" should probably be proceeded by "if (reserved)" |
[14:19:00] | danielk22: | otherwise "(m_priv->m_running_threads.size() < m_priv->GetRealMaxThread())" will always be true.. |
[14:20:31] | Captain_Murdoch: | makes sense from my reading of the code. |
[14:22:51] | stuartm: | yup, changed that locally and we'll see what happens |
[14:24:30] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[14:29:05] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (Read error: Connection reset by peer) | |
[14:29:24] | sraue (sraue!~stephan@xbmc/staff/sraue) has joined #mythtv | |
[14:42:21] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
[14:45:20] | jya (jya!~jyavenard@mythtv/developer/jya) has quit (Quit: jya) | |
[14:48:07] | stuartm: | danielk22: good catch, that's fixed the crazy number of threads, sadly not the segfault and now the UI is locking up for a minute before it finally dies |
[14:48:55] | stuartm: | heh, it's running out of threads, the ImageLoading thread spawns a MythDownloadManger thread, and it deadlocks |
[14:49:42] | stuartm: | well, not strictly deadlocks, but cannot continue since thread A cannot spawn thread B since maxthreads has been reached |
[14:59:17] | Seeker`: | does anyone have an appleTV? |
[14:59:57] | stuartm: | Captain_Murdoch: does it make sense for mythdownloadmanager to be asynchronous when it's being spawned from an image loading thread? |
[15:04:07] | stuartm: | nevermind, that's not the cause of the bottleneck, since MDM doesn't use the threadpool |
[15:19:36] | rsiebert (rsiebert!~quassel@g231187115.adsl.alicedsl.de) has joined #mythtv | |
[15:20:04] | rsiebert_ (rsiebert_!~quassel@e179130195.adsl.alicedsl.de) has quit (Ping timeout: 240 seconds) | |
[15:34:11] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has quit (*.net *.split) | |
[15:34:13] | highzeth (highzeth!~hz@hoiseth.no) has quit (*.net *.split) | |
[15:34:14] | iamlindoro (iamlindoro!~iamlindor@mythtv/developer/iamlindoro) has quit (*.net *.split) | |
[15:34:14] | brtb (brtb!~brtb@linode.brtb.net) has quit (*.net *.split) | |
[15:34:15] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (*.net *.split) | |
[15:34:17] | davide (davide!~david@host70.16.intrusion.com) has quit (*.net *.split) | |
[15:34:17] | bradd (bradd!~bradd@shaolin.ameri.ca) has quit (*.net *.split) | |
[15:34:17] | knightr (knightr!~Nicolas@mythtv/developer/knightr) has quit (*.net *.split) | |
[15:34:19] | notlistening (notlistening!~tom@94-195-105-95.zone9.bethere.co.uk) has quit (*.net *.split) | |
[15:34:19] | tris (tris!tristan@2001:1868:a00a::4) has quit (*.net *.split) | |
[15:34:19] | markk (markk!~mark@host86-182-165-130.range86-182.btcentralplus.com) has quit (*.net *.split) | |
[15:34:19] | cesman (cesman!~cesman@pdpc/supporter/professional/cesman) has quit (*.net *.split) | |
[15:34:19] | J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@unaffiliated/j-e-f-f-a) has quit (*.net *.split) | |
[15:34:20] | Unhelpful (Unhelpful!~quassel@rockbox/developer/Unhelpful) has quit (*.net *.split) | |
[15:34:21] | skd5aner (skd5aner!~skd5aner@cpe-071-071-242-134.carolina.res.rr.com) has quit (*.net *.split) | |
[15:34:22] | joe____ (joe____!~jmk@64.73.32.135) has quit (*.net *.split) | |
[15:34:22] | foxbuntu (foxbuntu!~foxbuntu@ubuntu/member/foxbuntu) has quit (*.net *.split) | |
[15:34:23] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has quit (*.net *.split) | |
[15:34:27] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (*.net *.split) | |
[15:34:27] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (*.net *.split) | |
[15:34:28] | stichnot (stichnot!~chatzilla@192.55.55.41) has quit (*.net *.split) | |
[15:34:28] | superm1 (superm1!u4318@ubuntu/member/superm1) has quit (*.net *.split) | |
[15:34:28] | jams (jams!~jams@cpe-184-58-217-97.wi.res.rr.com) has quit (*.net *.split) | |
[15:34:29] | clever (clever!~clever@142.167.216.200) has quit (*.net *.split) | |
[15:34:30] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (*.net *.split) | |
[15:34:30] | tgm4883 (tgm4883!~tgm4883@ubuntu/member/tgm4883) has quit (*.net *.split) | |
[15:34:32] | mirage335 (mirage335!~mirage335@mirage335.dyndns.org) has quit (*.net *.split) | |
[15:34:33] | Peitolm (Peitolm!LUG1@mandlebrot.random-chaos.org.uk) has quit (*.net *.split) | |
[15:34:33] | danielk22 (danielk22!~danielk@121.246.154.93) has quit (*.net *.split) | |
[15:34:33] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (*.net *.split) | |
[15:34:33] | MythBuild (MythBuild!~MythBuild@184-106-209-209.static.cloud-ips.com) has quit (*.net *.split) | |
[15:34:37] | brfransen (brfransen!~brfransen@64.179.142.146) has quit (*.net *.split) | |
[15:34:37] | ybot_ (ybot_!quassel@nat/nokia/x-cfbxppevayusvyir) has quit (*.net *.split) | |
[15:34:40] | joki (joki!~joki@p54862B3A.dip.t-dialin.net) has quit (*.net *.split) | |
[15:34:40] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has quit (*.net *.split) | |
[15:34:40] | foobum (foobum!~foobum@78-105-15-213.zone3.bethere.co.uk) has quit (*.net *.split) | |
[15:34:40] | mrand (mrand!~mrand@ubuntu/member/mrand) has quit (*.net *.split) | |
[15:34:40] | gigem (gigem!~gigem@mythtv/developer/gigem) has quit (*.net *.split) | |
[15:34:40] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has quit (*.net *.split) | |
[15:34:40] | wahrhaft (wahrhaft!~quassel@cpe-24-210-71-26.columbus.res.rr.com) has quit (*.net *.split) | |
[15:34:42] | tomimo (tomimo!~kurre@xdsl-83-150-88-111.nebulazone.fi) has quit (*.net *.split) | |
[15:34:43] | ElmerFudd (ElmerFudd!~le@0x5737a682.cpe.ge-0-1-0-1101.hsnqu1.customer.tele.dk) has quit (*.net *.split) | |
[15:34:44] | Slasher` (Slasher`!~Slasher@188.165.164.15) has quit (*.net *.split) | |
[15:34:44] | _davidjw_ (_davidjw_!~davidjw@gatea.kw.bbc.co.uk) has quit (*.net *.split) | |
[15:34:45] | wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has quit (*.net *.split) | |
[15:34:45] | zCougar (zCougar!~cougar@2001:67c:32c:600:fc5b:69b5:7872:dd22) has quit (*.net *.split) | |
[15:34:46] | Anssi (Anssi!hannulaa@mandriva/developer/anssi) has quit (*.net *.split) | |
[15:34:46] | rhpot1991 (rhpot1991!~rhpot1991@ubuntu/member/rhpot1991) has quit (*.net *.split) | |
[15:34:46] | stuarta (stuarta!~stuarta@mythtv/developer/stuarta) has quit (*.net *.split) | |
[15:34:46] | damaltor (damaltor!sbnc@h1889977.stratoserver.net) has quit (*.net *.split) | |
[15:34:46] | dblain (dblain!~dblain@mythtv/developer/dblain) has quit (*.net *.split) | |
[15:34:46] | coling (coling!~colin@cpc1-sgyl30-2-0-cust727.sgyl.cable.virginmedia.com) has quit (*.net *.split) | |
[15:34:46] | jstenback (jstenback!~jstenback@2620:101:8003:200:224:e8ff:fe39:34c2) has quit (*.net *.split) | |
[15:34:47] | GreyFoxx (GreyFoxx!~greg@mythtv/developer/GreyFoxx) has quit (*.net *.split) | |
[15:34:47] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit (*.net *.split) | |
[15:34:50] | justinh (justinh!~justin@cpc10-salf5-2-0-cust50.10-2.cable.virginmedia.com) has quit (*.net *.split) | |
[15:34:50] | ThisNewGuy (ThisNewGuy!~doug@pool-74-102-15-78.nwrknj.fios.verizon.net) has quit (*.net *.split) | |
[15:34:50] | ghoti (ghoti!~paul@scratch.it.ca) has quit (*.net *.split) | |
[15:34:50] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has quit (*.net *.split) | |
[15:34:50] | taylorr (taylorr!~taylorr@unaffiliated/elmojo) has quit (*.net *.split) | |
[15:34:50] | jwhite (jwhite!~jwhite@75-146-153-89-minnesota.hfc.comcastbusiness.net) has quit (*.net *.split) | |
[15:34:51] | jpabq- (jpabq-!~jpabq@174-28-149-119.albq.qwest.net) has quit (*.net *.split) | |
[15:34:51] | kurre2 (kurre2!~tomimo@xdsl-83-150-88-111.nebulazone.fi) has quit (*.net *.split) | |
[15:34:52] | laga (laga!~laga@h1626373.stratoserver.net) has quit (*.net *.split) | |
[15:34:52] | _charly_ (_charly_!kroseneg@sunrise.schmidham.net) has quit (*.net *.split) | |
[15:34:53] | dlblog (dlblog!~dlblog@c-76-127-227-175.hsd1.ma.comcast.net) has quit (*.net *.split) | |
[15:34:54] | TazzNZ (TazzNZ!~pieter@mail.insync.za.net) has quit (*.net *.split) | |
[15:34:54] | rsiebert (rsiebert!~quassel@g231187115.adsl.alicedsl.de) has quit (*.net *.split) | |
[15:34:55] | vallor (vallor!~Ponzo@pdpc/supporter/monthlygold/vallor) has quit (*.net *.split) | |
[15:34:56] | dekarl_zZz (dekarl_zZz!~dekarl@dslb-084-058-159-001.pools.arcor-ip.net) has quit (*.net *.split) | |
[15:34:56] | Captain_Murdoch (Captain_Murdoch!~cpinkham@mythtv/developer/CaptainMurdoch) has quit (*.net *.split) | |
[15:34:56] | purserj (purserj!~purserj@hosting.collaborynth.com.au) has quit (*.net *.split) | |
[15:34:56] | neufeld (neufeld!~user@ottawa-hs-206-191-28-140.s-ip.magma.ca) has quit (*.net *.split) | |
[15:34:57] | anykey_ (anykey_!~guedel@46-126-245-147.dynamic.hispeed.ch) has quit (*.net *.split) | |
[15:34:57] | poptix- (poptix-!poptix@poptix.net) has quit (*.net *.split) | |
[15:34:57] | mag0o (mag0o!20001@slackhost.lynchmv.com) has quit (*.net *.split) | |
[15:34:57] | jpabq (jpabq!~jpabq@mythtv/developer/jpabq) has quit (*.net *.split) | |
[15:34:58] | ChanServ (ChanServ!ChanServ@services.) has quit (*.net *.split) | |
[15:34:59] | aloril (aloril!~aloril@dsl-tkubrasgw3-fe7ef900-153.dhcp.inet.fi) has quit (*.net *.split) | |
[15:35:00] | sphery (sphery!~mdean@mythtv/developer/sphery) has quit (*.net *.split) | |
[15:35:00] | sailerboy (sailerboy!~sailerboy@sailerboy.net) has quit (*.net *.split) | |
[15:35:01] | cattelan_away (cattelan_away!~cattelan@c-66-41-26-220.hsd1.mn.comcast.net) has quit (*.net *.split) | |
[15:35:01] | fluvvell (fluvvell!~barryc@port166-72.ubs.maxnet.net.nz) has quit (*.net *.split) | |
[15:35:02] | Chutt (Chutt!~ijr@cpe-24-29-225-175.neo.res.rr.com) has quit (*.net *.split) | |
[15:35:02] | eharris (eharris!~eharris@99-179-7-82.lightspeed.austtx.sbcglobal.net) has quit (*.net *.split) | |
[15:50:41] | aloril (aloril!~aloril@dsl-tkubrasgw3-fe7ef900-153.dhcp.inet.fi) has joined #mythtv | |
[15:51:55] | Chutt (Chutt!~ijr@cpe-24-29-225-175.neo.res.rr.com) has joined #mythtv | |
[15:55:44] | jpabq (jpabq!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
[15:55:44] | mag0o (mag0o!20001@slackhost.lynchmv.com) has joined #mythtv | |
[15:55:44] | poptix- (poptix-!poptix@poptix.net) has joined #mythtv | |
[15:55:44] | anykey_ (anykey_!~guedel@46-126-245-147.dynamic.hispeed.ch) has joined #mythtv | |
[15:55:44] | neufeld (neufeld!~user@ottawa-hs-206-191-28-140.s-ip.magma.ca) has joined #mythtv | |
[15:55:44] | purserj (purserj!~purserj@hosting.collaborynth.com.au) has joined #mythtv | |
[15:55:44] | Captain_Murdoch (Captain_Murdoch!~cpinkham@mythtv/developer/CaptainMurdoch) has joined #mythtv | |
[15:55:44] | dekarl_zZz (dekarl_zZz!~dekarl@dslb-084-058-159-001.pools.arcor-ip.net) has joined #mythtv | |
[15:55:44] | vallor (vallor!~Ponzo@pdpc/supporter/monthlygold/vallor) has joined #mythtv | |
[15:55:44] | rsiebert (rsiebert!~quassel@g231187115.adsl.alicedsl.de) has joined #mythtv | |
[15:55:44] | jstenback (jstenback!~jstenback@2620:101:8003:200:224:e8ff:fe39:34c2) has joined #mythtv | |
[15:55:44] | jwhite (jwhite!~jwhite@75-146-153-89-minnesota.hfc.comcastbusiness.net) has joined #mythtv | |
[15:55:44] | laga (laga!~laga@h1626373.stratoserver.net) has joined #mythtv | |
[15:55:44] | _charly_ (_charly_!kroseneg@sunrise.schmidham.net) has joined #mythtv | |
[15:55:44] | tomimo (tomimo!~kurre@xdsl-83-150-88-111.nebulazone.fi) has joined #mythtv | |
[15:55:44] | GreyFoxx (GreyFoxx!~greg@mythtv/developer/GreyFoxx) has joined #mythtv | |
[15:55:44] | dlblog (dlblog!~dlblog@c-76-127-227-175.hsd1.ma.comcast.net) has joined #mythtv | |
[15:55:44] | Slasher` (Slasher`!~Slasher@188.165.164.15) has joined #mythtv | |
[15:55:44] | ElmerFudd (ElmerFudd!~le@0x5737a682.cpe.ge-0-1-0-1101.hsnqu1.customer.tele.dk) has joined #mythtv | |
[15:55:44] | taylorr (taylorr!~taylorr@unaffiliated/elmojo) has joined #mythtv | |
[15:55:44] | coling (coling!~colin@cpc1-sgyl30-2-0-cust727.sgyl.cable.virginmedia.com) has joined #mythtv | |
[15:55:44] | dblain (dblain!~dblain@mythtv/developer/dblain) has joined #mythtv | |
[15:55:44] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has joined #mythtv | |
[15:55:44] | foxbuntu (foxbuntu!~foxbuntu@ubuntu/member/foxbuntu) has joined #mythtv | |
[15:55:44] | ghoti (ghoti!~paul@scratch.it.ca) has joined #mythtv | |
[15:55:44] | damaltor (damaltor!sbnc@h1889977.stratoserver.net) has joined #mythtv | |
[15:55:44] | joe____ (joe____!~jmk@64.73.32.135) has joined #mythtv | |
[15:55:44] | wahrhaft (wahrhaft!~quassel@cpe-24-210-71-26.columbus.res.rr.com) has joined #mythtv | |
[15:55:44] | mirage335 (mirage335!~mirage335@mirage335.dyndns.org) has joined #mythtv | |
[15:55:44] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
[15:55:44] | stuarta (stuarta!~stuarta@mythtv/developer/stuarta) has joined #mythtv | |
[15:55:44] | ThisNewGuy (ThisNewGuy!~doug@pool-74-102-15-78.nwrknj.fios.verizon.net) has joined #mythtv | |
[15:55:44] | skd5aner (skd5aner!~skd5aner@cpe-071-071-242-134.carolina.res.rr.com) has joined #mythtv | |
[15:55:44] | rhpot1991 (rhpot1991!~rhpot1991@ubuntu/member/rhpot1991) has joined #mythtv | |
[15:55:44] | Anssi (Anssi!hannulaa@mandriva/developer/anssi) has joined #mythtv | |
[15:55:44] | justinh (justinh!~justin@cpc10-salf5-2-0-cust50.10-2.cable.virginmedia.com) has joined #mythtv | |
[15:55:44] | tgm4883 (tgm4883!~tgm4883@ubuntu/member/tgm4883) has joined #mythtv | |
[15:55:44] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
[15:55:44] | gigem (gigem!~gigem@mythtv/developer/gigem) has joined #mythtv | |
[15:55:44] | mrand (mrand!~mrand@ubuntu/member/mrand) has joined #mythtv | |
[15:55:44] | clever (clever!~clever@142.167.216.200) has joined #mythtv | |
[15:55:44] | foobum (foobum!~foobum@78-105-15-213.zone3.bethere.co.uk) has joined #mythtv | |
[15:55:44] | kurre2 (kurre2!~tomimo@xdsl-83-150-88-111.nebulazone.fi) has joined #mythtv | |
[15:55:44] | Unhelpful (Unhelpful!~quassel@rockbox/developer/Unhelpful) has joined #mythtv | |
[15:55:44] | ybot_ (ybot_!quassel@nat/nokia/x-cfbxppevayusvyir) has joined #mythtv | |
[15:55:44] | J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@unaffiliated/j-e-f-f-a) has joined #mythtv | |
[15:55:44] | MythBuild (MythBuild!~MythBuild@184-106-209-209.static.cloud-ips.com) has joined #mythtv | |
[15:55:44] | brfransen (brfransen!~brfransen@64.179.142.146) has joined #mythtv | |
[15:55:44] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has joined #mythtv | |
[15:55:44] | zCougar (zCougar!~cougar@2001:67c:32c:600:fc5b:69b5:7872:dd22) has joined #mythtv | |
[15:55:44] | cesman (cesman!~cesman@pdpc/supporter/professional/cesman) has joined #mythtv | |
[15:55:44] | jpabq- (jpabq-!~jpabq@174-28-149-119.albq.qwest.net) has joined #mythtv | |
[15:55:44] | jams (jams!~jams@cpe-184-58-217-97.wi.res.rr.com) has joined #mythtv | |
[15:55:44] | markk (markk!~mark@host86-182-165-130.range86-182.btcentralplus.com) has joined #mythtv | |
[15:55:44] | tris (tris!tristan@2001:1868:a00a::4) has joined #mythtv | |
[15:55:44] | notlistening (notlistening!~tom@94-195-105-95.zone9.bethere.co.uk) has joined #mythtv | |
[15:55:44] | joki (joki!~joki@p54862B3A.dip.t-dialin.net) has joined #mythtv | |
[15:55:44] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has joined #mythtv | |
[15:55:44] | superm1 (superm1!u4318@ubuntu/member/superm1) has joined #mythtv | |
[15:55:44] | stichnot (stichnot!~chatzilla@192.55.55.41) has joined #mythtv | |
[15:55:45] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
[15:55:45] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
[15:55:45] | wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has joined #mythtv | |
[15:55:45] | Peitolm (Peitolm!LUG1@mandlebrot.random-chaos.org.uk) has joined #mythtv | |
[15:55:45] | knightr (knightr!~Nicolas@mythtv/developer/knightr) has joined #mythtv | |
[15:55:45] | danielk22 (danielk22!~danielk@121.246.154.93) has joined #mythtv | |
[15:55:45] | _davidjw_ (_davidjw_!~davidjw@gatea.kw.bbc.co.uk) has joined #mythtv | |
[15:55:45] | bradd (bradd!~bradd@shaolin.ameri.ca) has joined #mythtv | |
[15:55:45] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
[15:55:45] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[15:55:45] | sraue (sraue!~stephan@xbmc/staff/sraue) has joined #mythtv | |
[15:55:45] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
[15:55:45] | sailerboy_ (sailerboy_!~sailerboy@sailerboy.net) has joined #mythtv | |
[15:55:45] | eharris (eharris!~eharris@99-179-7-82.lightspeed.austtx.sbcglobal.net) has joined #mythtv | |
[15:55:45] | sphery (sphery!~mdean@user-0c6s50s.cable.mindspring.com) has joined #mythtv | |
[15:55:45] | iamlindoro (iamlindoro!~iamlindor@c-71-202-67-27.hsd1.ca.comcast.net) has joined #mythtv | |
[15:55:45] | fluvvell (fluvvell!~barryc@port166-72.ubs.maxnet.net.nz) has joined #mythtv | |
[15:55:45] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has joined #mythtv | |
[15:55:45] | brtb (brtb!~brtb@linode.brtb.net) has joined #mythtv | |
[15:56:49] | stuarta: | sounds like the MDM needs a queue of download requests, if it doesn't already have one |
[15:57:00] | stuarta: | rather than firing off lots of threads |
[15:59:11] | TazzNZ (TazzNZ!~pieter@mail.insync.za.net) has joined #mythtv | |
[16:04:30] | MythBuild (MythBuild!~MythBuild@184-106-209-209.static.cloud-ips.com) has quit (Quit: buildmaster reconfigured: bot disconnecting) | |
[16:04:53] | MythBuild (MythBuild!~MythBuild@184-106-209-209.static.cloud-ips.com) has joined #mythtv | |
[16:05:40] | stichnot (stichnot!~chatzilla@192.55.55.41) has quit (Ping timeout: 248 seconds) | |
[16:06:30] | MythBuild: | build #501 of master-vista-mingw-32bit is complete: Exception [exception git] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/501 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[16:06:48] | stuarta: | please excuse me on that one |
[16:06:56] | stuartm: | it actually does, when I looked closer it seems we're just requesting each download 4 times (that's expected) and it doesn't appear to be using the cache |
[16:06:57] | stuarta: | i twatted the buildmaster |
[16:07:13] | sailerboy_ is now known as sailerboy | |
[16:07:26] | stuartm: | there's a little more going on, but I'm still working through it |
[16:10:25] | stuarta: | right, so thats Qt install on the mac as a framework |
[16:10:53] | stuarta: | tweaked the buildbot so it uses the buildbot local install paths before the usual ones, so hopefully it will still build |
[16:30:16] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
[16:33:16] | Captain_Murdoch: | stuartm, isn't ImageLoadThread using MythImageReader which uses MDM::download() which is sync vs MDM::queueDownload() which is async |
[16:34:04] | Captain_Murdoch: | stuartm, you mean it's not using the QNetworkDiskCache so it's hitting the net 4 times for the same image? |
[16:34:16] | Captain_Murdoch: | or not using our image cache? |
[16:34:40] | stuartm: | Captain_Murdoch: ignore the former, I was getting confused, it's the latter, it's not using the QNetworkDiskCache |
[16:35:34] | stuartm: | I've stuck debugging in and it's getting as far as looking for 'Date' in the headers but not apparently finding it |
[16:36:09] | stuartm: | Captain_Murdoch: http://pastebin.com/Euv90fCt |
[16:41:57] | cattelan_away (cattelan_away!~cattelan@c-66-41-26-220.hsd1.mn.comcast.net) has joined #mythtv | |
[16:46:04] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[16:46:06] | Captain_Murdoch: | do you know that's actually hitting the net 4 times? tcpdump/wireshark? |
[16:46:17] | Captain_Murdoch: | s/^/how / |
[16:47:37] | stuartm: | Captain_Murdoch: I stuck debugging in and it's not setting m_preferCache? Plus it's taking way too long to be operating from the cache? |
[16:48:01] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Read error: Operation timed out) | |
[16:48:43] | stuartm: | Captain_Murdoch: while you're here, why is the cache window just 60 seconds? for the channel icons I think we'd definitely benefit from a bigger window, 5 minutes at a minimum although most browsers cache for at least an hour |
[16:51:05] | Captain_Murdoch: | prefercache says "load from cache if it's there, don't even check lastmodified" so if we just loaded within 60 seconds, we don't bother going out over the net again to do another lastmodified check against the webserver. |
[16:51:24] | Captain_Murdoch: | so it's a speedup, otherwise we'd be doing multiple lastmodified checks against the webserver. |
[16:52:24] | Captain_Murdoch: | it was put in specifically for buttonlists with images when I was testing ideas with the theme downloader. the buttonlist has the theem preview in it and when I was testing remote previews, the buttonlist was quite laggy when scrolling. |
[16:52:38] | Captain_Murdoch: | I think that's when I added that code. :| |
[16:53:26] | Captain_Murdoch: | so the whole 'Date' and 60-second shortcut to use prefercache just lets us tell Qt to not bother with the lastmodified check against the webserver, just use whatever is in the disk cache if it's there. |
[16:53:56] | stuartm: | ok that makes sense, we do hit the 60 second limit for a page full of icons but mostly because it's taking so long – that log snippet shows two full minutes passing and that's only a fraction of the whole log |
[16:55:28] | stuartm: | iirc there are 30–36 icons on display for Terra, I don't know where the holdup is but it shouldn't take so long to grab that many images and it's freezing the UI while it does it which I can't see to explain :( |
[16:55:37] | ** stuartm has a headache ** | |
[16:55:47] | Captain_Murdoch: | you don't see the lastmodified checks in Qt code in the logs. that's why I was curious how you saw it hitting the net multiple times. those subsequent loads should be coming from the disk cache, but there will be a lastmodified check against the webserver for loads outside the 60-second window. you might try bumping that to 5 minutes as a test to see if subsequent loads are quick without the webserver lastmodified check slowi |
[16:55:48] | Captain_Murdoch: | ng things down. |
[16:57:46] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[16:58:03] | Captain_Murdoch: | and that 'Date' is date served, so I don't think it gets updated in the cache ever, so once it has been 60 seconds from the first download, all subsequent requests will trigger a lastmodified check against the webserver, even if you exit and reenter the screen. |
[16:58:12] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[16:58:17] | ** stuarta larts self for commit message ** | |
[16:59:55] | Captain_Murdoch: | stuartm, how long does a 'wget' take of one of those images? |
[17:00:09] | stuartm: | yeah, I've got debugging in the if (header == "date") section and it's not printing, suggesting that it's never getting to that point :( |
[17:00:13] | Captain_Murdoch: | about 0.5 seconds here using 'time wget' |
[17:00:43] | stuartm: | 0.03s |
[17:01:12] | Captain_Murdoch: | you could print out the (*it).first() string there as a test to see what headers it is finding. |
[17:01:42] | stuartm: | admittedly opera's is working lots of downloads in parallel but to load 3000 icons in the services web interface it's taking less than 3 minutes |
[17:01:48] | Captain_Murdoch: | if you run tcpdump or wireshark I think you'll see it sending lastmodified checks to the webserver but getting the actual data from disk cache. |
[17:03:34] | Captain_Murdoch: | might be nice if there was a 'lastaccessed' header we could check to bypass the lastmodified check against the webserver if we'd used the cached image within the last X seconds, but that could cause problems elsewhere. |
[17:05:14] | Seeker`: | stuartm: "date" or "Date"? |
[17:06:27] | Captain_Murdoch: | DAte |
[17:06:28] | Captain_Murdoch: | Date |
[17:06:41] | Captain_Murdoch: | if ((*it).first == "Date") |
[17:07:56] | Seeker`: | Captain_Murdoch: i was asking . ecause he said he had a debug statement in if (header == "date"), didnt know if that was why itisnt reaching it |
[17:08:28] | danielk22 (danielk22!~danielk@121.246.154.93) has quit (Ping timeout: 276 seconds) | |
[17:08:48] | Seeker`: | also, apologies for typing, phoneirc |
[17:11:29] | Captain_Murdoch: | no problem. |
[17:12:59] | stuartm: | touching libmythbase means long recompiles :( |
[17:14:33] | stuartm: | Captain_Murdoch: no date header :/ http://pastebin.com/CVzBUrmh |
[17:14:35] | Captain_Murdoch: | did you change the API? I just make install inside that lib dir if not changing the API. |
[17:15:28] | stuartm: | Captain_Murdoch: I'm running make install inside the lib dir, I probably should be running make inside the lib too |
[17:15:47] | Captain_Murdoch: | odd, not even Last-Modified listed. |
[17:16:49] | stichnot (stichnot!~chatzilla@192.55.55.39) has joined #mythtv | |
[17:16:52] | Captain_Murdoch: | I have a bunch of aliases to cd into the various lib dirs to make it easier so I can edit and make inside each dir if not modifying the API. |
[17:20:46] | Captain_Murdoch: | wait, that's the redirect's headers, not the images's headers. redirect has location instead of last-modified. |
[17:20:59] | Captain_Murdoch: | so that's probably what's messing it up. |
[17:21:26] | Captain_Murdoch: | s/probably/possibly/ :) |
[17:22:25] | Captain_Murdoch: | if we encounter a 'Location' header, it means it's a redirect, so we should re-call m_manager->cache()->metaData(QUrl(url)); with the 'Location' URL to get the real object's headers. |
[17:23:38] | stuartm: | I'll add that |
[17:24:06] | stuartm: | unless you want to? |
[17:26:07] | Captain_Murdoch: | I can't mess with it right now. a quick and dirty test could be right after the initial call before the m_infoLock->unlock(). just loop through the headers and reset urlData to be the metadata for the Location header URL if we find one. |
[17:26:42] | Captain_Murdoch: | I could look at it later tonight possibly, but can't promise. |
[17:27:05] | danielk22 (danielk22!~danielk@183.83.207.164) has joined #mythtv | |
[17:28:34] | jpabq_: | stuartm: did you have a chance to test Terra, to make sure it is now showing that text? |
[17:28:42] | jpabq_ is now known as abqjp | |
[17:29:53] | stuartm: | jpabq: it's working, thanks |
[17:32:14] | abqjp: | Good. Did you notice any cutdown places where it is not showing all three dots? I am not 100% happy with the fix, but short of putting a loop in there to reduce the size until it fits, it is the best I could come up with. |
[17:34:26] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has quit (Remote host closed the connection) | |
[17:36:32] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
[17:36:37] | stuartm: | I haven't yet, but I'll keep a look out |
[17:42:59] | highzeth (highzeth!~hz@hoiseth.no) has joined #mythtv | |
[18:05:50] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
[18:06:12] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
[18:17:17] | notlistening (notlistening!~tom@94-195-105-95.zone9.bethere.co.uk) has quit (Quit: Leaving) | |
[18:26:13] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[18:27:35] | joki (joki!~joki@p54862B3A.dip.t-dialin.net) has quit (Ping timeout: 252 seconds) | |
[18:27:45] | joki- (joki-!~joki@p54863372.dip.t-dialin.net) has joined #mythtv | |
[18:27:54] | joki- is now known as joki | |
[18:28:22] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 244 seconds) | |
[18:57:49] | sutula (sutula!sutula@nat/hp/x-aojgsubnygspunng) has joined #mythtv | |
[19:01:57] | gast (gast!bcc294ec@gateway/web/freenode/ip.188.194.148.236) has joined #mythtv | |
[19:02:56] | danielk22 (danielk22!~danielk@183.83.207.164) has quit (Quit: Leaving.) | |
[19:03:28] | stuartm: | Captain_Murdoch: so I've fixed it so that we're looking at the right metadata but there's still no Date header http://pastebin.com/cHzzaMAB |
[19:05:13] | mike (mike!~mike@c-76-115-119-121.hsd1.or.comcast.net) has joined #mythtv | |
[19:05:39] | mike is now known as Guest81167 | |
[19:07:58] | Guest90452 (Guest90452!~mike@c-76-115-119-121.hsd1.or.comcast.net) has quit (Ping timeout: 255 seconds) | |
[19:10:34] | stuartm: | we could inject it when first loading via setRawHeaders()? |
[19:13:53] | stuartm: | Here's the patch – http://pastebin.com/DiCvbjTW |
[19:13:57] | gast (gast!bcc294ec@gateway/web/freenode/ip.188.194.148.236) has quit (Ping timeout: 264 seconds) | |
[19:16:19] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
[19:22:44] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
[19:24:49] | rockhound (rockhound!~rockhound@d041048.adsl.hansenet.de) has joined #mythtv | |
[19:27:00] | rockhound (rockhound!~rockhound@d041048.adsl.hansenet.de) has quit (Client Quit) | |
[19:28:22] | stuartm: | can we not use QNetworkCacheMetaData::lastModified() instead? According to the docs that's not the "Last Modified" header but the modification time of the metadata |
[19:28:39] | rockhound (rockhound!~rockhound@d041048.adsl.hansenet.de) has joined #mythtv | |
[19:34:06] | Beirdo: | stuarta: nope |
[19:34:53] | Beirdo: | the problem is.. this is a setup from a company we took over... trying to bring it up to our current (and still sub-par) standards |
[19:37:04] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
[19:38:06] | rockhound (rockhound!~rockhound@d041048.adsl.hansenet.de) has quit (Quit: Leaving) | |
[19:40:37] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[19:41:03] | stuartm: | the docs lied |
[19:41:14] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
[19:45:03] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 245 seconds) | |
[19:46:08] | allesmueller (allesmueller!~allesmuel@unixboard/users/allesmueller) has joined #mythtv | |
[20:04:51] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
[20:17:00] | gast (gast!bcc294ec@gateway/web/freenode/ip.188.194.148.236) has joined #mythtv | |
[20:39:24] | stuartm: | Captain_Murdoch: working patch – accounts for redirects and inserts a Date header into the cached metadata if one doesn't exist, a little hacky maybe – http://pastebin.com/QTU2Tj7Z |
[20:39:39] | stuartm: | if you don't object I'll commit it |
[20:48:51] | mzanetti_ is now known as mzanetti | |
[20:50:44] | kth (kth!~kth@unaffiliated/kth) has quit (Quit: Leaving.) | |
[21:02:12] | stichnot_ (stichnot_!~chatzilla@192.55.55.41) has joined #mythtv | |
[21:03:29] | stichnot (stichnot!~chatzilla@192.55.55.39) has quit (Ping timeout: 245 seconds) | |
[21:03:30] | Captain_Murdoch: | stuartm, fine by me. I was wondering about using setRawHeaders() to set a 'lastAccessedTime' as well, perhaps for later implementing our own cache cleanup. |
[21:03:39] | stichnot_ is now known as stichnot | |
[21:04:24] | Captain_Murdoch: | ah, missed your lastmodified comment. it's possible, I don't recall all the QNetwork* info I read when writing MDM. |
[21:05:58] | stuartm: | turns out the lastmodified documentation was badly worded, it is the Last Modified header and not the modification timestamp of the cached entry |
[21:07:05] | stuartm: | inserting our own header into the cached metadata seems to be the only way of doing this that I could see |
[21:17:08] | zombor_ is now known as zombor | |
[21:35:09] | stuarta: | right, now to see if i can get things to build on OSX against Qt as a framework |
[21:36:55] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit () | |
[21:39:38] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 240 seconds) | |
[21:56:12] | poptix- (poptix-!poptix@poptix.net) has quit (Quit: Reconnecting) | |
[21:56:18] | poptix (poptix!poptix@poptix.net) has joined #mythtv | |
[21:59:46] | xris: | iamlindoro: I actually made some time last night to figure out how to get the icon scanner to pull in high res icons. |
[22:00:09] | xris: | didn't deploy it because it's been so long since I touched it that I don't know what their addition will do to the rest of the data. |
[22:07:16] | gast (gast!bcc294ec@gateway/web/freenode/ip.188.194.148.236) has quit () | |
[22:07:34] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 245 seconds) | |
[22:08:37] | stuartm: | Beirdo: segfault in logging code, but it looks to be cause by a corrupt stack – http://pastebin.com/r6pw7Gbn |
[22:14:09] | stuartm: | does the url http://www.lyngsat-logo.com/logo/tv/bb/bbc_kids_ca.jpg work for anyone? I'm wondering if they've just made radical changes to their site or if I'm now being blocked for making too many requests |
[22:14:37] | dekarl_zZz: | I get "forbidden" |
[22:14:41] | laga: | error 403 forbidden |
[22:14:47] | stuartm: | ok, so it's not just me then |
[22:15:16] | stuartm: | hmm, the main site is also giving the same response so looks like an issue on their end |
[22:15:22] | dekarl_zZz: | I get it for the whole lyngsat-logo site |
[22:15:39] | stuartm: | guess I'm done working on the icon downloader tonight :) |
[22:28:06] | Beirdo: | stuartm: that's a problem with some other code tromping the stack, most likely (or a double-free situatuon) |
[22:28:41] | Beirdo: | it has been every time I've seen it so far :) |
[22:33:19] | stichnot_ (stichnot_!~chatzilla@192.55.55.41) has joined #mythtv | |
[22:33:19] | stichnot (stichnot!~chatzilla@192.55.55.41) has quit (Remote host closed the connection) | |
[22:33:20] | stichnot_ is now known as stichnot | |
[22:41:36] | allesmueller (allesmueller!~allesmuel@unixboard/users/allesmueller) has quit (Ping timeout: 240 seconds) | |
[22:46:47] | beata (beata!beata@108.12.181.220) has joined #mythtv | |
[23:08:15] | beata- (beata-!beata@108.12.181.220) has joined #mythtv | |
[23:09:20] | NightMonkey (NightMonkey!~NightMonk@pdpc/supporter/professional/nightmonkey) has joined #mythtv | |
[23:09:30] | beata (beata!beata@108.12.181.220) has quit (Ping timeout: 252 seconds) | |
[23:26:55] | skd5aner (skd5aner!~skd5aner@cpe-071-071-242-134.carolina.res.rr.com) has quit (Read error: Connection reset by peer) | |
[23:27:11] | skd5aner (skd5aner!~skd5aner@cpe-071-071-242-134.carolina.res.rr.com) has joined #mythtv | |
[23:46:50] | highzeth (highzeth!~hz@hoiseth.no) has quit (*.net *.split) | |
[23:46:51] | cattelan_away (cattelan_away!~cattelan@c-66-41-26-220.hsd1.mn.comcast.net) has quit (*.net *.split) | |
[23:56:20] | jya (jya!~jyavenard@120.147.14.211) has joined #mythtv | |
[23:59:45] | jya (jya!~jyavenard@120.147.14.211) has quit (Changing host) | |
[23:59:45] | jya (jya!~jyavenard@mythtv/developer/jya) has joined #mythtv |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.