July 2010 - DVDnav-discuss - lists.mplayerhq.hu

[PATCH] consuming all memory when scanning dvd "Up"
by John Stebbins 09 Dec '12

09 Dec '12

First a disclaimer. This is a big invasive patch. If you don't like it, my feelings won't be hurt. I will be using it in HandBrake. So if you see problems with it, please let me know. Many dvd's are showing up that have titles that have numerous repeated language unit tables and pgc's. The new release "Up" is an example. I believe "Dark Knight" also exhibits this. When scanning such discs with lsdvd (and other programs that scan all titles), libdvdread will consume many GB of data for these repeated elements and gets very slow (e.g hours to read all titles) on OS X due to reading the duplicate data from uncached raw devices. This patch detects duplicates and reference counts pgc's and pgcit's. When a duplicate is detected, a reference count is incremented instead of allocating new memory and re-reading the data. I opted for reference counting instead of simply returning a failure on duplicate detection because it is perfectly valid to have a title with such duplicates. I could easily imagine titles adding duplicates, then navigating around them.

3 5

[PATCH 0/6] catch up to 0.9.6 and a couple of extras
by Erik Hovland 11 Oct '11

11 Oct '11

In an attempt to work towards getting Jorgen's patches in, I have taken on the task of catching up the mplayer tree of libdvdread to 0.9.6. Next will be 0.9.7. But I figured that I can submit that in its own chunk. This patch set also includes some cosmetic fixes as well as a potential memory leak fix and a check of a few system calls. Enjoy E --- Erik Hovland (6): 0.9.6 included a few items to improve the sofware on the *BSD Add in DVDFileStat Add the version function DVDVersion(). The version should be bumped to at least 5. But I am going to go to 6 on this one. Make cosmetic changes to files. Some system calls that report errors are not being checked. msvc/include/pthreads/pthread.h | 6 - msvc/include/sys/time.h | 2 msvc/include/unistd.h | 4 - src/bitreader.c | 2 src/dvd_input.c | 2 src/dvd_reader.c | 258 ++++++++++++++++++++++++++++++++++++--- src/dvd_udf.c | 4 - src/dvdread/dvd_reader.h | 47 ++++++- src/dvdread/ifo_types.h | 2 src/dvdread/nav_types.h | 8 + src/ifo_print.c | 22 ++- src/md5.c | 4 - src/nav_print.c | 2 13 files changed, 302 insertions(+), 61 deletions(-) -- Signature

7 34

[PATCH] UDF2.50 part 1
by Jorgen Lundman 06 Oct '11

06 Oct '11

Separated RAR support, as it is the most questionble change in the old patch. Told emacs to never use tabs too, so it should be cleaner. Add UDF2.50 logic, which includes BD5ISO, and HD-DVD support. Support non-DVD style access. (File iteration, and reading). Diffed against svn 1153. This patch includes: 1. Add ExtFinfo (262) support 2. Add FileType (250) re-direct support 3. DVDOpendir/DVDReaddir/DVDClosedir/DVDOpenFile for UDF access. 4. dvdread-1.9.7 stat methods for obtaining VOB data and sizes 5. AD chaining, instead of assuming files will always fit in 1 AD group. 6. dvd_udf API uses "UDF_FILE" type instead of "starting block number". 7. Separate struct AD and struct FileAD instead of ugly 64->32 bit work. 8. Minor memory clearing problems, and memory leaks. Lund -- Jorgen Lundman | <lundman(a)lundman.net> Unix Administrator | +81 (0)3 -5456-2687 ext 1017 (work) Shibuya-ku, Tokyo | +81 (0)90-5578-8500 (cell) Japan | +81 (0)3 -3375-1767 (home) -- Jorgen Lundman | <lundman(a)lundman.net> Unix Administrator | +81 (0)3 -5456-2687 ext 1017 (work) Shibuya-ku, Tokyo | +81 (0)90-5578-8500 (cell) Japan | +81 (0)3 -3375-1767 (home) -- Jorgen Lundman | <lundman(a)lundman.net> Unix Administrator | +81 (0)3 -5456-2687 ext 1017 (work) Shibuya-ku, Tokyo | +81 (0)90-5578-8500 (cell) Japan | +81 (0)3 -3375-1767 (home)

6 35

Re: [DVDnav-discuss] dvdnav patches from handbrake project
by Erik Hovland 06 Oct '11

06 Oct '11

>> Jean - Do VLC folk have anything to say about the patches that the >> handbrake folk submitted to this list? If VLC is running w/ them it would >> help the patches case in getting committed. > Sorry, but do you have a link to them? I do! Huge invasive patch to consolidate memory: http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2009-November/001097.html Bug fix for crashing: http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2009-November/001102.html Follow up patch for same crashing issue: http://lists.mplayerhq.hu/pipermail/dvdnav-discuss/2009-December/001114.html E -- Erik Hovland erik(a)hovland.org http://hovland.org/

8 47

[PATCH] crash while processing mac the ripper "feature title extract" images
by John Stebbins 06 Oct '11

06 Oct '11

Mac the ripper's feature title extraction removes menus from the resulting image, but does not remove navigation instructions that attempt to jump to those menus. This patch checks that a menu exists before acting on such instructions. If the menu does not exist, the it puts the vm into the stopped state.

3 6

Assertion failure in src/vm/vm.c:1102
by Dominik 'Rathann' Mierzejewski 24 Nov '10

24 Nov '10

Hi. I got a crash report from Fedora's new automated bugreporter. It happened in totem, but the crash came from a failed assertion: "totem: /builddir/build/BUILD/libdvdnav-4.1.4/src/vm/vm.c:1102: play_Cell: La declaracin `0' no se cumple.\n" I don't have any information on how to reproduce it, but maybe one of you has an idea why this might happen and how to fix it. The original bugreport is here: https://bugzilla.redhat.com/show_bug.cgi?id=539446 -- Fedora http://fedoraproject.org/wiki/User:Rathann RPMFusion http://rpmfusion.org | MPlayer http://mplayerhq.hu "Faith manages." -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations"

2 1

[PATCH] Use 'path' instead of 'dev_name'
by KO Myung-Hun 25 Sep '10

25 Sep '10

Hi/2. The device is indicated by 'path' not a 'dev_name' on Win32 and OS/2. -- KO Myung-Hun Using Mozilla SeaMonkey 2.0.3 Under OS/2 Warp 4 for Korean with FixPak #15 On AMD ThunderBird 1GHz with 512 MB RAM Korean OS/2 User Community : http://www.ecomstation.co.kr

2 2

[PATCH] Check \video_ts as well as /video_ts on OS/2
by KO Myung-Hun 25 Sep '10

25 Sep '10

Hi/2. This patch check \video_ts as well as /video_ts on OS/2. -- KO Myung-Hun Using Mozilla SeaMonkey 2.0.3 Under OS/2 Warp 4 for Korean with FixPak #15 On AMD ThunderBird 1GHz with 512 MB RAM Korean OS/2 User Community : http://www.ecomstation.co.kr

2 2

[PATCH] Strip off the trailing \ if it is not a drive on OS/2
by KO Myung-Hun 25 Sep '10

25 Sep '10

Hi/2. This patch strips off the trailing \ if it is not a drive on OS/2. -- KO Myung-Hun Using Mozilla SeaMonkey 2.0.3 Under OS/2 Warp 4 for Korean with FixPak #15 On AMD ThunderBird 1GHz with 512 MB RAM Korean OS/2 User Community : http://www.ecomstation.co.kr

2 1

modification of read-raw-device.patch
by Erik Hovland 01 Aug '10

01 Aug '10

This is especially for KO and John. John's patch read-raw-device.patch conflicts w/ one of KO's patches for OS/2. Here is my attempt at it. But since I have neither of these platforms, I have no idea if it means anything. Thanks E -- Erik Hovland erik(a)hovland.org http://hovland.org/

2 3