Mailing List Archive: Cherokee: dev

Issue 1362 in cherokee: Stream handler needs mutex locking.

Index | Next | Previous | View Threaded

cherokee at googlecode

Jun 9, 2012, 9:02 PM

Post #1 of 7 (495 views)
Permalink

Issue 1362 in cherokee: Stream handler needs mutex locking.

Status: New
Owner: ----

New issue 1362 by mehmetal...@gmail.com: Stream handler needs mutex locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

What steps will reproduce the problem?
1. Comment out stream handler built-in cache
2. Get same file repeatedly
3. Memory usage will rise up to 100mb

What version of the product are you using? On what operating system?
Cherokee 1.2.101

Please provide any additional information below.
av_find_stream_info needs mutex locking so i found an example for it but it
is directly using pthread library's function. Cherokee has built-in
mutex-locking functions but i can't understand how to add it...

i attached sample code with some comments.

Attachments:
handler_streaming.c.diff 1.3 KB

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Jun 9, 2012, 10:01 PM

Post #2 of 7 (458 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Comment #1 on issue 1362 by mehmetal...@gmail.com: Stream handler needs
mutex locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

We should not free hdl->start :) i notice it is different thing.

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Jun 15, 2012, 4:11 PM

Post #3 of 7 (447 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Updates:
Status: Started
Owner: ste...@konink.de
Labels: Type-Defect Priority-Medium Component-Logic Usability
Milestone-Stable

Comment #2 on issue 1362 by ste...@konink.de: Stream handler needs mutex
locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

We should indeed do this using the native cherokee code for it, I'll
comment :)

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Jun 18, 2012, 8:41 AM

Post #4 of 7 (445 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Comment #3 on issue 1362 by 246...@gmail.com: Stream handler needs mutex
locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

Add a patch that includes a "cherokized" lock manager ...

Note that the "hdl->avformat = NULL;" chunk of the original patch is not
included in this diff and should be included as a separate diff.

Attachments:
add-locks-to-handler-streaming.diff 1.1 KB

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Aug 26, 2012, 10:48 AM

Post #5 of 7 (370 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Comment #4 on issue 1362 by mehmetal...@gmail.com: Stream handler needs
mutex locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

so, av_find_stream_info is never fail when getting file info.

but memory usage is raising. normally, AVFormatContext is keeping in hdl
struct so i move it into function.

i comment out some part of function and i add a variable to struct. it is
in attachment.

for now i tested this code on production server and memory usage is not
raising same as old one's.

Attachments:
func.c 2.1 KB

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Sep 15, 2012, 7:39 AM

Post #6 of 7 (347 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Comment #5 on issue 1362 by mehmetal...@gmail.com: Stream handler needs
mutex locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

with comment #4's functions memory usage is still raising. so it isn't
solution.

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

cherokee at googlecode

Jan 17, 2013, 3:31 PM

Post #7 of 7 (139 views)
Permalink

Re: Issue 1362 in cherokee: Stream handler needs mutex locking. [In reply to]

Updates:
Status: Fixed

Comment #6 on issue 1362 by ste...@konink.de: Stream handler needs mutex
locking.
http://code.google.com/p/cherokee/issues/detail?id=1362

Mutex has been implemented thanks to 246tnt, lets track the leaking
somewhere else.

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads