From: Thomas Hellstrom <thellstrom@vmware.com>
To: Ilija Hadzic <ihadzic@research.bell-labs.com>
Cc: "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
Subject: Re: KMS cursor BO semantics
Date: Fri, 04 Nov 2011 15:38:59 +0100 [thread overview]
Message-ID: <4EB3F903.40604@vmware.com> (raw)
In-Reply-To: <Pine.GSO.4.62.1111040931150.11051@umail>
On 11/04/2011 03:36 PM, Ilija Hadzic wrote:
>
>
> On Fri, 4 Nov 2011, Thomas Hellstrom wrote:
>
>> Hi.
>>
>> I have a question about the semantics of the DRM_IOCTL_MODE_CURSOR
>> iotcl:
>>
>> Some hardware (vmware's virtual in particular) may not be able to
>> pick up the changes from a bo directly, since the cursor data is sent
>> though the command stream. Hence we need a notification when the
>> cursor image has changed.
>>
>> Could we *require* that a cursor image change needs to be followed by
>> an ioctl call with the flag
>> DRM_MODE_CURSOR_BO?
>>
>> Thanks,
>> Thomas
>>
>
> FWIW, Acked-by: Ilija Hadzic <ihadzic@research.bell-labs.com>
> I have a few places where I could use such an ioctl.
>
Well, the idea was to reuse the DRM_IOCTL_MODE_CURSOR ictl that sets the
cursor BO, and require that it is re-emitted once the cursor image has
changed, since I guess that's what many X servers do anyway.
> BTW, Thomas, in the above "since the cursor data is sent though the
> command stream", did you mean "since the cursor data is *not* sent
> though the command stream". If it was sent, through command stream,
> then CS ioctl would know when the cursor changes.
I meant the device interface, not the drm interface. So when we have a
new cursor image, we need to collect the scanline data an send it
through the device command stream. We don't export that possibility in
the DRM interface.
>
> My understanding is that the cursor data are mmap-ed letting userland
> poke it at will (so the case when an "hourglass" changes into "arrow"
> is particularly hard to know that it happened).
Yes. Hardware that scan out directly from the cursor BO don't really
need to know, but we do.
>
> -- Ilija
Thanks,
/Thomas
next prev parent reply other threads:[~2011-11-04 14:41 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-04 11:59 KMS cursor BO semantics Thomas Hellstrom
2011-11-04 14:36 ` Ilija Hadzic
2011-11-04 14:38 ` Thomas Hellstrom [this message]
2011-11-04 15:34 ` Daniel Vetter
2011-11-04 22:30 ` Thomas Hellstrom
2011-11-04 22:49 ` Maarten Maathuis
2011-11-04 22:59 ` Thomas Hellstrom
2011-11-04 23:20 ` Maarten Maathuis
2011-11-05 16:10 ` Daniel Vetter
2011-11-07 10:26 ` Thomas Hellstrom
2011-11-11 18:17 ` James Simmons
2011-11-11 19:10 ` Maarten Maathuis
2011-11-11 19:14 ` Maarten Maathuis
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=4EB3F903.40604@vmware.com \
--to=thellstrom@vmware.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=ihadzic@research.bell-labs.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.