Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1391232
| From | Tomi Valkeinen <tomi.valkeinen@ti.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support |
| Date | 2016-04-29 17:10 +0200 |
| Message-ID | <rtidl-8c3-35@gated-at.bofh.it> (permalink) |
| References | <rsVTr-5A5-3@gated-at.bofh.it> <rsW39-5Ij-37@gated-at.bofh.it> <rtgbx-69i-49@gated-at.bofh.it> <rti3D-7N6-1@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
On 29/04/16 17:55, Daniel Vetter wrote:
>> Who's calling {write,fillrect,copyarea,imageblit} in an atomic context?
>> That sounds like a very bad idea to me...
>>
>> If this is only for accumulating changes, I think it may be better to
>> leave that to the driver as it may have better idea of how to accumulate.
>>
>> But, of course, this is a helper, so if all the drivers use this kind of
>> accumulation, it makes sense =).
>
> Apparently panic context and cursor timer and stuff like that. I think
> this started with udl, and just to make sure (it's fbdev after all, no one
> wants to read the code itself) we've put those checks onto all entry
> points that draw stuff. It could be overkill, but this is what udl/qxl
> already do, so better to keep imo for now.
>
> I guess if it's really not needed we could later on change that, but not
> sure that's worth the effort. And we can't get rid of it entirely.
Yep... Sounds fine to me.
Someone should implement (minimal) fbdev userspace API support to DRM
without using the current fbdev, and implement fbcon on top of that. I
don't like how DRM and fbdev gets mixed...
I offer a beer to anyone who does that =).
Tomi
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v4 0/7] drm: Add fbdev deferred io support to helpers Noralf Trønnes <noralf@tronnes.org> - 2016-04-28 17:30 +0200
[PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Noralf Trønnes <noralf@tronnes.org> - 2016-04-28 17:30 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-04-29 15:00 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Noralf Trønnes <noralf@tronnes.org> - 2016-04-29 16:50 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Daniel Vetter <daniel@ffwll.ch> - 2016-05-02 16:30 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Daniel Vetter <daniel@ffwll.ch> - 2016-04-29 17:00 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-04-29 17:10 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Daniel Vetter <daniel@ffwll.ch> - 2016-04-29 17:40 +0200
Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support Daniel Vetter <daniel@ffwll.ch> - 2016-04-29 17:40 +0200
[PATCH v4 4/7] fbdev: fb_defio: Export fb_deferred_io_mmap Noralf Trønnes <noralf@tronnes.org> - 2016-04-28 17:30 +0200
Re: [PATCH v4 4/7] fbdev: fb_defio: Export fb_deferred_io_mmap Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-04-29 16:10 +0200
Re: [PATCH v4 0/7] drm: Add fbdev deferred io support to helpers Daniel Vetter <daniel@ffwll.ch> - 2016-04-29 17:00 +0200
Re: [PATCH v4 0/7] drm: Add fbdev deferred io support to helpers Daniel Vetter <daniel@ffwll.ch> - 2016-05-02 16:30 +0200
csiph-web