Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1391257

Re: [PATCH v4 3/7] drm/fb-helper: Add fb_deferred_io support

Show all headers | View raw

Adding David, forgot that before hitting send.
-Daniel

On Fri, Apr 29, 2016 at 5:36 PM, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Fri, Apr 29, 2016 at 06:00:18PM +0300, Tomi Valkeinen wrote:
>>
>> 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 =).
>
> The big plan that has been floated years back already (before David
> Herrmann disappeared into the rh systemd gang) was to have a pure
> userspace kmscon for console (including vt-switching in userspace like X
> does), plus a super-minimal panic console on top of kms in the kernel.
> That looked like a really solid design, unfornutately it didn't go
> anywhere. But basic patches are still around, and it really just boils
> down to that some distro would need to integrate kmscon properly, and
> polish of the kernel side too for integration.
> -Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

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