[mythtv] [mythtv-commits] Ticket #1356: DTVRecorder::FindKeyframes in 0.19 causes video gltiches in firewire recordings

[Steven Adeff]

On 2/28/06, Steven Adeff <adeffs.mythtv at gmail.com> wrote:
> On 2/28/06, Jim Westfall <jwestfall at surrealistic.net> wrote:
> > Steven Adeff <adeffs.mythtv at gmail.com> wrote [02.28.06]:
> > > On 2/28/06, Jim Westfall <jwestfall at surrealistic.net> wrote:
> > > >
> > > > I was able to reproduce livetv glitches if my backend gets over loaded.
> > > > Can you verify this isnt happened to you? you do streaming?
> > > >
> > > > also, could you try the following 2 patches and see if they help at all.
> > > > Both should cut down on cpu usage when the backend is recording/livetv.
> > > >
> > > > thanks
> > > > jim
> > >
> > > I don't think my backend is getting overloaded. I only have a single
> > > computer be/fe setup right now, and only the firewire recorder has the
> > > glitch problem, my PVR-150 is fine and my HD3000 is fine, I can record
> > > from those two at the same time with no problems as well. My CPU usage
> > > usually sits around 80% when watching 1080i content, lower, ~50% for
> > > 720p content, and live tv raises that by ~5-10%.
> >
> > ok, just wanted to me a sure.  Have you tested streaming vs non-streaming?
>
> streaming? I would need to build 0.19 on my file server(right now its
> got an older SVN that needs to be updated), but I can do that. Though
> the original firewire problem is in the actual recorded files so when
> I tried playing back on it before I got the same errors in the same
> places.
>
> >
> > >
> > > I'll compile in those two patches, does it matter if I grab the latest
> > > SVN to do so? Should I apply both patches at the same time or test
> > > them seperately?
> >
> > you should be able to stay with the -fixes branch, but will need to grab
> > the newest firewirerecorder.* file again.
>
> I'll use SVN for these tests, then I can have a seperate folder with a
> working copy to revert to so the g/f can watch tv =)
>

oh yea, whats the proper command for using patch with these types of
diff? I normally get

# patch < firewire.diff
can't find file to patch at input line 5
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|Index: libs/libmythtv/firewirerecorder.cpp
|===================================================================
|--- libs/libmythtv/firewirerecorder.cpp        (revision 9175)
|+++ libs/libmythtv/firewirerecorder.cpp        (working copy)
--------------------------


which I can get around and patch properly, but what is the majic
command I'm missing?

--
Steve