From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60672)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <fred.konrad@greensocs.com>) id 1YuJeb-0007H9-Ee
	for qemu-devel@nongnu.org; Mon, 18 May 2015 07:56:18 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <fred.konrad@greensocs.com>) id 1YuJeY-00055z-1g
	for qemu-devel@nongnu.org; Mon, 18 May 2015 07:56:17 -0400
Received: from greensocs.com ([193.104.36.180]:34441)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <fred.konrad@greensocs.com>) id 1YuJeX-00055p-QY
	for qemu-devel@nongnu.org; Mon, 18 May 2015 07:56:13 -0400
Message-ID: <5559D357.1010103@greensocs.com>
Date: Mon, 18 May 2015 13:56:07 +0200
From: Frederic Konrad <fred.konrad@greensocs.com>
MIME-Version: 1.0
References: <1431544326-13372-1-git-send-email-fred.konrad@greensocs.com>		<1431544326-13372-4-git-send-email-fred.konrad@greensocs.com>	
	<1431934462.10826.1.camel@nilsson.home.kraxel.org>	
	<555999F8.80609@greensocs.com>
	<1431947865.16446.29.camel@nilsson.home.kraxel.org>
In-Reply-To: <1431947865.16446.29.camel@nilsson.home.kraxel.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 3/8] console: add qemu_alloc_display_format.
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: peter.crosthwaite@xilinx.com, mark.burton@greensocs.com, qemu-devel@nongnu.org, hyunk@xilinx.com

On 18/05/2015 13:17, Gerd Hoffmann wrote:
> On Mo, 2015-05-18 at 09:51 +0200, Frederic Konrad wrote:
>> On 18/05/2015 09:34, Gerd Hoffmann wrote:
>>> On Mi, 2015-05-13 at 21:12 +0200, fred.konrad@greensocs.com wrote:
>>>> From: KONRAD Frederic <fred.konrad@greensocs.com>
>>>>
>>>> This allows to create a surface with a different format than xrgb8888.
>>> What is the use case for this?
>>>
>>> cheers,
>>>     Gerd
>>>
>>>
>>>
>> Hi,
>>
>> The Display Port introduced in patch 7 and the driver use differents
>> pixel format.
>> eg: rgb565, RGB888, etc. see xilinx_dp_change_graphic_fmt in patch 7:
> Ah, and dpdma scatterlists (patch 6) suggest the guest framebuffer is
> not contiguous in guest phyiscal memory, correct?  So using guest memory
> as surface storage using qemu_create_displaysurface_from() doesn't work.
Exactly, the framebuffer is not necessarilly contiguous, so I need to 
put the
fragment into the pixman storage.

> But I think you can simply call qemu_create_displaysurface_from() with
> linesize=0 and data=NULL to make pixman allocate storage for you.
Ah, maybe! I'll check that.

Thanks,
Fred
> cheers,
>    Gerd
>
>