From: Carl Karsten <carl@personnelware.com>
To: video4linux-list@redhat.com, Andy Walls <awalls@radix.net>
Subject: Re: v4l2 api compliance test
Date: Wed, 05 Nov 2008 00:46:38 -0600 [thread overview]
Message-ID: <4911414E.2050801@personnelware.com> (raw)
In-Reply-To: <1225762470.3198.23.camel@palomino.walls.org>
Andy Walls wrote:
> On Sun, 2008-11-02 at 21:23 -0600, Carl Karsten wrote:
>> Andy Walls wrote:
>>> On Sun, 2008-11-02 at 18:32 -0600, Carl Karsten wrote:
>>>> Mauro Carvalho Chehab wrote:
>>>>> On Sat, 01 Mar 2008 01:45:24 -0600
>>>>> Carl Karsten <carl@personnelware.com> wrote:
>>>>>
>>>>>
>>>>> Please, feel free to improve the tools. Unfortunately, nobody yet had time to
>>>>> dedicate on improving the testing tools.
>>>> These 2 issues are thwarting my efforts to write my tester:
>>>>
>>>> 1. memory leak:
>>>> valgrind ./capture --userp -d /dev/video1
>>>> ==17153== malloc/free: in use at exit: 2,457,632 bytes in 5 blocks.
>>>>
>>>> 2. capabilities mismatch:
>>>> ./capture --userp -d /dev/video1
>>>> VIDIOC_QBUF error 22, Invalid argument
>>>>
>>>> details: http://linuxtv.org/v4lwiki/index.php/Test_Suite#Bugs_in_Examples
>>> I'm not sure why a memory leak on abnormal termination is worrisome for
>>> you. It looks like init_userp() allocated a bunch of "buffers", which
>>> has to happen for a program to use user pointer mode of v4l2. The
>>> function errno_exit() doesn't bother to clean up when the VIDIOC_QBUF
>>> ioctl() call fails. free() is only called by uninit_device(). Since
>>> the alternate flow of the program through errno_exit() to termination
>>> doesn't call free() on "buffers", you should have a process heap memory
>>> leak on error exit.
>>>
>>> Since this is userspace, a memory leak from the process heap doesn't
>>> hang around when the process terminates - no big deal.
>> Are you sure about that?
>
> About the process heap, yes.
>
>> if I run
>> ./capture --userp -d /dev/video1
>> VIDIOC_QBUF error 22, Invalid argument
>>
>> enough I can't run the valid modes:
>>
>> juser@dhcp186:~/vga2usb/v4l.org/examples$ ./capture --read -d /dev/video1
>> read error 12, Cannot allocate memory
>
> The capture app would output "Out of memory" if the calloc() call for
> the --read option buffers failed. This is some global/kernel resource
> that has been exhausted.
>
>
>> juser@dhcp186:~/vga2usb/v4l.org/examples$ ./capture --mmap -d /dev/video1
>> mmap error 12, Cannot allocate memory
>
> Ditto for this. This message can only happen at the end of init_mmap()
> when the mmap call fails. Thus an allocation of some sort of kernel
> global resource/space failed.
>
>
> I don't know what could be exhausting those kernel resources when using
> the userp option. The failed ioctl()'s calls to the vivi driver would
> be a place to start looking.
>
>
>> although free still shows lots:
>>
>> juser@dhcp186:~/vga2usb/v4l.org/examples$ free
>> total used free shared buffers cached
>> Mem: 1033388 282340 751048 0 31012 98208
>> -/+ buffers/cache: 153120 880268
>> Swap: 859436 0 859436
>>
>
> Perhaps you could look at /proc/meminfo between runs and see if
> something is gradually being exhausted. Vmalloc address space
> exhaustion is what I'd look for.
>
I forgot to sort it - now I see:
HighFree go from 72608 to 252
VmallocUsed from 6440 to 109724
VmallocChunk from 103620 to 336
I hope this sheds some light on where to look.
Carl K
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2008-11-05 6:47 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-01 7:45 v4l2 api compliance test Carl Karsten
2008-03-04 14:38 ` Mauro Carvalho Chehab
2008-11-03 0:32 ` Carl Karsten
2008-11-03 1:20 ` Andy Walls
2008-11-03 3:23 ` Carl Karsten
2008-11-04 1:34 ` Andy Walls
2008-11-05 1:15 ` Carl Karsten
2008-11-07 20:56 ` Mauro Carvalho Chehab
2008-11-05 6:46 ` Carl Karsten [this message]
2008-11-05 12:17 ` Andy Walls
2008-11-07 5:43 ` Carl Karsten
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4911414E.2050801@personnelware.com \
--to=carl@personnelware.com \
--cc=awalls@radix.net \
--cc=video4linux-list@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox