[mythtv] Patch for "fixme" in NuppelVideoRecorder.cpp
rob
mythtv-dev@snowman.net
Fri, 15 Nov 2002 09:53:32 -0600
This is a multi-part message in MIME format.
--------------010105070407090702040307
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Here's a patch to fix the block of code in NuppelVideoRecorder.cpp that
has the comment "count free buffers -- FIXME this can be done with less
CPU time!!" It just adds a member variable to the class that keeps
track of the free buffers, instead of counting them in the loop every
time in the function.
Also, in what case(s) is it necessary to have the thread for writing
data to the disk (ThreadedFileWriter)? The implementation is a ring
buffer that has data added when Write() is called, then sending that
data to the disk in another thread. Doesn't the write() system call
just dump the data to kernel buffers, where it is written when it can
be? The extra thread and memcpy's to implement the ring buffer stuff
may not be necessary. I didn't notice any difference in performance
when I removed all of the ThreadedFileWriter code and just had system
write() calls.
--------------010105070407090702040307
Content-Type: application/gzip;
name="patch_nvr_fixme.gz"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
filename="patch_nvr_fixme.gz"
H4sICBQW1T0CA3BhdGNoX252cl9maXhtZQCtV21z2kYQ/mx+xSYfGrAkLCEwGILHTounnknS
1knaTjMeRogDaSok9XSy63rc3969F4lDKDbGvQ/cod199vVu7+bhYgFWTmF1xwJ2Y9nt/lEU
zjL+8zFPUxL9Gs5JcqStr4if0DmhbT9NNbF2TG6fI9qwLGtPrQcd2+5YjmPZfbDdoTMY2r22
XQyw7GPbbhiG8QLrShVOD+yTYbcz7LlbKs7OwDo5No/BwN8BnJ01gI80TGEMCy/KyKgB8tsN
V3ARRozQ92HGkP769ahhNAxFnOWLBSWEJXxBKNJtlH3g4jUWDof/1nxt3iThvCWscrsDbpbb
s82+sKsBnFiPdRmH7J1QmykIuH/CMD4WCYVmGDMIxUec3ko3p5J16id5zPC7YbSkxL2cNGRC
2V1K4FD+RRxM0yZtJP3p9aQ/A+XPJo51mnmrNCKadTqRO/A5mcR+Mn+c5V3ho4O5Odj23jDW
ohsYRDG00zwLpjPP/7MppVtK4EG4Mei63I1Bt1+mhY8VWfnpXVPD+er57No6lX9MwNncjG0W
/kM49jfzZI1hG0/3ctSoGl/DpEULiVPdhDIW4QKaVSKcjmtqoQUiCo7d7fIw4HyySzoPiq8s
XJEnc6in2aiUmx4gYwxNB6z6CmjtVCIqs8qrjkwuzlWvakYl7Cp4RNh+vfZUt2NH0dqdsKPs
IxHcLZJPg9cE+FnGbcVfH1Upfb+WY7NeJeM36lUltt/BhGJi+0Vi1UHGDz+2Sk0Q+8SEJGfT
iMT456cvn6fvJx+V9ozR3GdA2YJ6KxIQD49d0NajhlXA/e15JnBPhQES1k9WKSVZxoIwUxnh
rI9ylfZR71arBWEx1lZKqNCPpG5By5E4mDI4TCMvJtlX91qevY5r26bTwQC4dnFuHQjpZLFM
Ug2dbwXhydERSMu4jSBDmgH2+4vL3z9MgJsIvhfDjMA8iQnchiyACI2H73/+Igx89UoiiSaD
QZEthi/qmwxSsM0o9fdyKpKt1xXyVUqpECpGGVZePJZ0y9ggwLhmF5SdnmvUeN8q/Rv2whG4
LUDNcKCnTTqJwctSQuaQpxgpAqpCU5r4yCdT0us4Jt5MDFzwQ2fdSrSqaou16K9jeHP+RiB7
+TxMJFeRLG5wYYUg14RRh+W8ApXDujosz6oCIvMidmIieBd6BqpdQc3jKm5VPMWWSxhuviVW
0lhK6r2yTOWWpNZUiuVGLhchzRjzYTzGuBcdrHcsO1iv19cufsUOW3opL2aut3IOc2qEuilh
irI+YfeJsbGGXsPylTrXihCgXkIj4t2QeXOJd4lmFiSUCXMOW/x28Wizqh9bMcaq7u4D1Mzj
LFzGWPF+4FE0aJW6wiYZannDxtkxT8pQP6gMycJSt0vjZYVl7FdYWg5oGC/lqWKd/kZDRprf
aVgmXFydf5j8ODn/YXL16fKPid4Et0XFpW9Lny4jiLM7RjLeg7dN464mcXSn/I0TkIbg4Tp/
wbsv2PddFez75gvUc8y2XAccd9gZDDvd//PFF2y/95xu7Xuv44idz6f1JY/vo79yL+JRVfn5
5RPjKQW+C/1yn3PG2t7BxyxJIv54LI+fAqPyeBw1/gOrHYHwtg8AAA==
--------------010105070407090702040307--