From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:14737 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933345Ab1LFOmE (ORCPT ); Tue, 6 Dec 2011 09:42:04 -0500 Message-ID: <4EDE29AA.8090203@redhat.com> Date: Tue, 06 Dec 2011 12:41:46 -0200 From: Mauro Carvalho Chehab MIME-Version: 1.0 To: "'Sakari Ailus'" CC: Laurent Pinchart , Kamil Debski , linux-media@vger.kernel.org, =?UTF-8?B?J1NlYmFzdGlhbiBEcsO2Z2Un?= , Sylwester Nawrocki , Marek Szyprowski , Hans Verkuil Subject: Re: [RFC] Resolution change support in video codecs in v4l2 References: <4ED905E0.5020706@redhat.com> <007201ccb118$633ff890$29bfe9b0$%debski@samsung.com> <201112061301.01010.laurent.pinchart@ideasonboard.com> <20111206142821.GC938@valkosipuli.localdomain> In-Reply-To: <20111206142821.GC938@valkosipuli.localdomain> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 06-12-2011 12:28, 'Sakari Ailus' wrote: > Hi all, > > On Tue, Dec 06, 2011 at 01:00:59PM +0100, Laurent Pinchart wrote: > ... >>>>>>> 2) new requirement is for a bigger buffer. DMA transfers need to be >>>>>>> stopped before actually writing inside the buffer (otherwise, memory >>>>>>> will be corrupted). >>>>>>> >>>>>>> In this case, all queued buffers should be marked with an error flag. >>>>>>> So, both V4L2_BUF_FLAG_FORMATCHANGED and V4L2_BUF_FLAG_ERROR should >>>>>>> raise. The new format should be available via G_FMT. >> >> I'd like to reword this as follows: >> >> 1. In all cases, the application needs to be informed that the format has >> changed. >> >> V4L2_BUF_FLAG_FORMATCHANGED (or a similar flag) is all we need. G_FMT will >> report the new format. >> >> 2. In all cases, the application must have the option of reallocating buffers >> if it wishes. >> >> In order to support this, the driver needs to wait until the application >> acknowledged the format change before it starts decoding the stream. >> Otherwise, if the codec started decoding the new stream to the existing >> buffers by itself, applications wouldn't have the option of freeing the >> existing buffers and allocating smaller ones. >> >> STREAMOFF/STREAMON is one way of acknowledging the format change. I'm not >> opposed to other ways of doing that, but I think we need an acknowledgment API >> to tell the driver to proceed. > > Forcing STRAEMOFF/STRAEMON has two major advantages: > > 1) The application will have an ability to free and reallocate buffers if it > wishes so, and > > 2) It will get explicit information on the changed format. Alternative would > require an additional API to query the format of buffers in cases the > information isn't implicitly available. As already said, a simple flag may give this meaning. Alternatively (or complementary, an event may be generated, containing the new format). > > If we do not require STRAEMOFF/STREAMON, the stream would have to be paused > until the application chooses to continue it after dealing with its buffers > and formats. No. STREAMOFF is always used to stop the stream. We can't make it mean otherwise. So, after calling it, application should assume that frames will be lost, while the DMA engine doesn't start again. For things like MPEG decoders, Hans proposed an ioctl, that could use to pause and continue the decoding. > I'd still return a specific error when the size changes since it's more > explicit that something is not right, rather than just a flag. But if I'm > alone in thinking so I won't insist. > > Regards, >