Hi
On Tue, Sep 23, 2014 at 11:38 PM, Jerome Glisse <j.glisse@xxxxxxxxx> wrote:
> On Tue, Sep 23, 2014 at 03:46:47PM +0200, Daniel Vetter wrote:
>> Really, the legacy buffer api should be dead, especially for all these
>> newfangled drivers. I suspect this is copypasta from the transitioning
>> days, which probably originated in radeon.
>>
>> Cc: David Airlie <airlied@xxxxxxxx>
>> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
>> Cc: "Christian König" <christian.koenig@xxxxxxx>
>> Cc: David Herrmann <dh.herrmann@xxxxxxxxx>
>> Cc: Rashika <rashika.kheria@xxxxxxxxx>
>> Cc: Josh Triplett <josh@xxxxxxxxxxxxxxxx>
>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
>> Cc: Fabian Frederick <fabf@xxxxxxxxx>
>> Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
>> Cc: Ben Skeggs <bskeggs@xxxxxxxxxx>
>> Cc: Alexandre Courbot <acourbot@xxxxxxxxxx>
>> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxx>
>> Cc: Christian Engelmayer <cengelma@xxxxxx>
>> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx>
>
> I would say NAK as i am pretty sure this break radeon UMS code path.
> Of course if we have the go ahead to nuke radeon UMS i am more than
> happy.
radeon_drv.c says:
driver_old.fops.mmap == drm_mmap;
..so I don't understand why that would break UMS? Do you use KMS mixed
with UMS? radeon_mmap() is only used by radeon_driver_kms_fops, which
is only used by kms_driver. UMS (which I understand as non-KMS in
radeon case) should not be affected by this.
Thanks
David
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
