linux-kernel - RE: [PATCH] drm/udl: Make page

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <HE1PR0701MB2219375D1A3B61081F239A7AF7A70@HE1PR0701MB2219.eurprd07.prod.outlook.com>
Date:   Thu, 20 Jul 2017 08:54:35 +0000
From:   Michal Lukaszek <michal.lukaszek@...playlink.com>
To:     Daniel Vetter <daniel@...ll.ch>,
        Stéphane Marchesin 
        <stephane.marchesin@...il.com>
CC:     Dave Airlie <airlied@...hat.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        Dawid Kurek <dawid.kurek@...playlink.com>
Subject: RE: [PATCH] drm/udl: Make page_flip asynchronous

On Thursday, July 13, 2017 7:11 PM, Daniel Vetter wrote:

>>> Can't we roll this driver over to the atomic helpers instead? There
>>> you get nonblocking pretty much for free ... I'm not sure extending
>>> the old modeset code has all that much benefit really.

>> This code certainly has value by itself; it makes the driver more
>> efficient. I think the best can sometimes be the enemy of the good --
>> this code is here and written, but I don't think any of us is going to
>> tackle atomic for udl.

> Sure, I guess I should have clarified this with "If you want me to
> review and merge this, then pls look into atomic, since that seems
> actually beneficial for my own interest". I'm not paid by intel to
> review driver patches at random, but to keep overall drm in nice
> shape. Moving drivers to atomic and using more shared infrastructure
> is in that interest, reviewing random driver patches isn't.

So, what's the next step here then? This code makes udl work better, without investing a lot of effort to move everything to atomics.

Btw, this is already causing modesetting not to support page flips (while Chrome OS, having this code uses page flips with udl):
https://cgit.freedesktop.org/xorg/xserver/tree/hw/xfree86/drivers/modesetting/driver.c#n1198 

Regards,
Michal Lukaszek