Sorry if I missed your mail. Your patch looks good, I'll work on it as soon as possible. Thanks, ?Benny On Sun, Oct 2, 2011 at 4:11 AM, Amir Pakdel <pakdel at gmail.com> wrote: > > > ---------- Forwarded message ---------- > From: Amir Pakdel <pakdel@xxxxxxxxx> > Date: Fri, Aug 5, 2011 at 1:20 PM > Subject: Re: Contribution > To: Perry Ismangil <perry at teluu.com>, pjsip at lists.pjsip.org > > > Hi Perry, > > ? I am testing pjproject-trunk (from SVN) on a Linux x64 (Slackware64 > 13.37.0) and when I ran the vid_streamutil for the first time, > I encountered lots and lots of the following message: > libv4l2: error dequeuing buf: Resource temporarily unavailable > > According to the documentation of V4L2 > (http://v4l2spec.bytesex.org/spec/r12878.htm), it is because 'Non-blocking > I/O has been selected using O_NONBLOCK and no buffer was in the outgoing > queue.' > Therefore, in v4l2_dev.c, I removed ' | O_NONBLOCK' from v4l2_open call. > I hope the stream uses a separate thread and this would not affect its > performance. > > > Another change was calling pjmedia_vid_dev_default_param (in > vid_streamutil.c) with PJMEDIA_VID_DEFAULT_RENDER_DEV (instead of 1) when > 'dir' is PJMEDIA_DIR_DECODING, so that 'pjmedia_vid_port_create(pool, &vpp, > &renderer);' would not return PJMEDIA_EVID_INVDEV. > > Moreover, the pjmedia_vid_codec_h263_fmtp of H263 pjmedia_vid_codec_param is > empty; therefore, pjmedia_vid_codec_h263_apply_fmtp will reset the size to > QCIF: > ??????? /* Negotiate size & MPI setting */ > ??????? if (fmtp_rem.mpi_cnt == 0) { > ??????????? /* Remote doesn't specify MPI setting, send QCIF=1 */ > ??????????? size.w = 176; > ??????????? size.h = 144; > ??????????? mpi??? = 1; > ??????????? printf("Remote doesn't specify MPI setting, send QCIF=1\n"); > ??????? } else if (fmtp_loc.mpi_cnt == 0) { > > Consequently, the previously created capture port will generate fames with a > different size wich will result in a scrambled video frames! > Therefore, I moved create_stream(...) which creates the codec to a few lines > before creation of the capture port. > > > The patch is attached. I hope it would be worth something :) > > > > Regards, > Amir > > > On Mon, Jul 25, 2011 at 3:03 PM, Amir Pakdel <pakdel at gmail.com> wrote: >> >> Hi Perry, >> >> Thanks for your prompt reply. >> >> I have not developed anything special yet. I was wondering if the >> development team needs help with anything particular or maybe there is a bug >> that I can try to fix? >> I am mostly interested in video streaming and perhaps I can help with >> something like development of a Simple GUI for video API testing >> (http://trac.pjsip.org/repos/ticket/1327). >> >> >> Regards, >> Amir >> >> >> On Mon, Jul 25, 2011 at 2:51 PM, Perry Ismangil <perry at teluu.com> wrote: >>> >>> Hi Amir, >>> >>> We're glad you're excited with PJSIP. >>> >>> You can describe your contribution here on the list, and we can >>> consider it for inclusion. >>> >>> If it's substantial, then as mentioned you need to send the >>> Contributor agreement to me. >>> >>> Thanks. >>> >>> Regards, >>> >>> On Mon, Jul 25, 2011 at 06:57, Amir Pakdel <pakdel at gmail.com> wrote: >>> > Dear Developers, >>> > >>> > >>> > I am really enthusiast about PJSIP version 2.0 and I want to help with >>> > development. >>> > I have read almost all of the documentation in >>> > http://www.pjsip.org/docs.htm >>> > and I have written a few sample applications using pjproject-1.10. >>> > I would like to know how could I contribute to the development of the >>> > project? Where should I start (apart from PJSIP Contributor Agreement)? >>> > >>> > >>> > Best Regards, >>> > Amir Pakdel >>> > >>> > >>> > _______________________________________________ >>> > Visit our blog: http://blog.pjsip.org >>> > >>> > pjsip mailing list >>> > pjsip at lists.pjsip.org >>> > http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org >>> > >>> > >>> >>> >>> >>> -- >>> Perry Ismangil >> > > >
