[lkml]   [2021]   [Feb]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v6 02/10] Revert "drm/qxl: do not run release if qxl failed to init"

> On Feb 4, 2021, at 2:06 PM, Thomas Zimmermann <> wrote:
> Hi Tong
>> I have posted an updated patch.
>> The new patch use the following logic
>> + if (!qdev->ddev.mode_config.funcs)
>> + return;
> This is again just papering over the issue. Better don't call qxl_drm_release() in the error path if qxl_device_init() fails.
> I see two solutions: either roll-back manually, or use our new managed DRM interfaces. This is what the other drivers do.
> Best regards
> Thomas

IMHO - qdev->ddev.mode_config.funcs is set only if the initialization is successful,
so using this as an indicator of error case looks ok to me.

The other two options you suggested would require some serious significant amount of work to be done,
which I don’t think I currently have such ability to do.

- Tong

 \ /
  Last update: 2021-02-04 21:27    [W:0.039 / U:9.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site