From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8DE63C433DB for ; Thu, 4 Feb 2021 18:53:59 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 57FB864F59 for ; Thu, 4 Feb 2021 18:53:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239517AbhBDSx2 (ORCPT ); Thu, 4 Feb 2021 13:53:28 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54422 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239316AbhBDSwp (ORCPT ); Thu, 4 Feb 2021 13:52:45 -0500 Received: from mail-qv1-xf32.google.com (mail-qv1-xf32.google.com [IPv6:2607:f8b0:4864:20::f32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8D755C0613D6 for ; Thu, 4 Feb 2021 10:52:04 -0800 (PST) Received: by mail-qv1-xf32.google.com with SMTP id es14so2230949qvb.3 for ; Thu, 04 Feb 2021 10:52:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=8ZgAvEajl8OUNniz+1itXG7C5l6k2fF6JkJiBsO4nok=; b=nhMmLKArS2xb9cFp5MMKn5zJt2CZdTGK2ypgPa6MYMSt8duhxNfT5BFkLKjDiJAMXe wj6pNooIJSgf1MhhpBgtcavRJwxmF+xeTFBWe/YiQA5+B10FF4Yqv7dbpsZHSzdPzYYr vUgA3osoftu5awaOEuKhB3pAPpuRtJINsCondAWAr9QcdNXoV9azZWYVjdCMKl50r5AG zhIV9TjRDdVlHyrCbid0aZxIw2GmkSOfQGNWDFarSx2gq9dD/R4gUz5J1X6XVJ+IvvWz y6M3dda45nlYqZ05dheJD89SuSr/3wFCsVTJM3TaA7ZQNg/56Hyfc/1dVPGFufSQrsPw HCoA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=8ZgAvEajl8OUNniz+1itXG7C5l6k2fF6JkJiBsO4nok=; b=H2L0cjC18xczOwTIZKqk1qcD1290t6COpy4XCLD5H7ozFpgcPwzpq2CE7dsCkAMOOv OD5Ye1rgAss9GCaEgUDNg/pjRnI2hI9YM0L+CKcq1T6Hn/W+uyAlza2VvuW8xqMYWeyg mH3fT4Pj88JXzHJwd4MDCZm7pIl1oYVO0Ulq8mQdEUAnmODPj527yYbM3Ga2Cw5+do/B 2R9RF+P18veDgAriH/S/IbZPckx0Tkal+F0uVViec5k7FNnL0PWcZuUSu/C6H2hEWBN8 8/14s9piFLsCcSe5mqrc6J86AH0wsSd76K7bpcMMa8p1zgYxCEgwchFstTi5ifbp7L7T CsGQ== X-Gm-Message-State: AOAM531hE+w2q/RXx7IGiXDTCOQP5C+unS4LfNuVRU3dICIDO2x1DYp5 /hPGd8mYg6qHW1572wSkG58= X-Google-Smtp-Source: ABdhPJyJvT/YDmhSIb/x8r7uRaFvl000p8edHD/mRjYqlce+22wUNtkbzecqkR/f+Bcr9lN5Wu7enA== X-Received: by 2002:a0c:fe0b:: with SMTP id x11mr656852qvr.18.1612464723755; Thu, 04 Feb 2021 10:52:03 -0800 (PST) Received: from ?IPv6:2601:5c0:c200:27c6:cd86:137:1075:f377? ([2601:5c0:c200:27c6:cd86:137:1075:f377]) by smtp.gmail.com with ESMTPSA id 186sm6052286qkh.30.2021.02.04.10.52.02 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 04 Feb 2021 10:52:03 -0800 (PST) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\)) Subject: Re: [PATCH v6 02/10] Revert "drm/qxl: do not run release if qxl failed to init" From: Tong Zhang In-Reply-To: Date: Thu, 4 Feb 2021 13:52:02 -0500 Cc: Gerd Hoffmann , dri-devel@lists.freedesktop.org, David Airlie , open list , "open list:DRM DRIVER FOR QXL VIRTUAL GPU" , "open list:DRM DRIVER FOR QXL VIRTUAL GPU" , Dave Airlie Content-Transfer-Encoding: quoted-printable Message-Id: <60B8023C-78C9-441D-AA21-A13C4445F666@gmail.com> References: <20210204145712.1531203-1-kraxel@redhat.com> <20210204145712.1531203-3-kraxel@redhat.com> To: Thomas Zimmermann X-Mailer: Apple Mail (2.3654.60.0.2.21) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Thomas, The original problem was qxl_device_init() can fail, when it fails there is no need to call=20 qxl_modeset_fini(qdev); qxl_device_fini(qdev); But those two functions are otherwise called in the qxl_drm_release() - I have posted an updated patch. The new patch use the following logic + if (!qdev->ddev.mode_config.funcs) + return; qxl_modeset_fini(qdev); qxl_device_fini(qdev); Thanks, - Tong > On Feb 4, 2021, at 1:34 PM, Thomas Zimmermann = wrote: >=20 > Hi >=20 > Am 04.02.21 um 15:57 schrieb Gerd Hoffmann: >> This reverts commit b91907a6241193465ca92e357adf16822242296d. >=20 > This should be in the correct format, as given by 'dim cite'. >=20 > dim cite b91907a6241193465ca92e357adf16822242296d > b91907a62411 ("drm/qxl: do not run release if qxl failed to init") >=20 >> Patch is broken, it effectively makes qxl_drm_release() a nop >> because on normal driver shutdown qxl_drm_release() is called >> *after* drm_dev_unregister(). >> Cc: Tong Zhang >> Signed-off-by: Gerd Hoffmann >> --- >> drivers/gpu/drm/qxl/qxl_drv.c | 2 -- >> 1 file changed, 2 deletions(-) >> diff --git a/drivers/gpu/drm/qxl/qxl_drv.c = b/drivers/gpu/drm/qxl/qxl_drv.c >> index 34c8b25b5780..fb5f6a5e81d7 100644 >> --- a/drivers/gpu/drm/qxl/qxl_drv.c >> +++ b/drivers/gpu/drm/qxl/qxl_drv.c >> @@ -144,8 +144,6 @@ static void qxl_drm_release(struct drm_device = *dev) >> * reodering qxl_modeset_fini() + qxl_device_fini() calls is >> * non-trivial though. >> */ >> - if (!dev->registered) >> - return; >=20 > I'm not sure what the original problem was, but I'm sure that this = isn't the fix for it. If there's a problem with shutdown, the operations = rather have to be reordered correctly. >=20 > With the citation style address: >=20 > Acked-by: Thomas Zimmermann >=20 >> qxl_modeset_fini(qdev); >> qxl_device_fini(qdev); >> } >=20 > --=20 > Thomas Zimmermann > Graphics Driver Developer > SUSE Software Solutions Germany GmbH > Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany > (HRB 36809, AG N=C3=BCrnberg) > Gesch=C3=A4ftsf=C3=BChrer: Felix Imend=C3=B6rffer >=20 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.5 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 22122C433E0 for ; Thu, 4 Feb 2021 19:35:08 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E351864F6C for ; Thu, 4 Feb 2021 19:35:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E351864F6C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E06D86EE4E; Thu, 4 Feb 2021 19:34:53 +0000 (UTC) Received: from mail-qv1-xf2f.google.com (mail-qv1-xf2f.google.com [IPv6:2607:f8b0:4864:20::f2f]) by gabe.freedesktop.org (Postfix) with ESMTPS id 9E3D66E98B; Thu, 4 Feb 2021 18:52:04 +0000 (UTC) Received: by mail-qv1-xf2f.google.com with SMTP id l11so2243281qvt.1; Thu, 04 Feb 2021 10:52:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=8ZgAvEajl8OUNniz+1itXG7C5l6k2fF6JkJiBsO4nok=; b=nhMmLKArS2xb9cFp5MMKn5zJt2CZdTGK2ypgPa6MYMSt8duhxNfT5BFkLKjDiJAMXe wj6pNooIJSgf1MhhpBgtcavRJwxmF+xeTFBWe/YiQA5+B10FF4Yqv7dbpsZHSzdPzYYr vUgA3osoftu5awaOEuKhB3pAPpuRtJINsCondAWAr9QcdNXoV9azZWYVjdCMKl50r5AG zhIV9TjRDdVlHyrCbid0aZxIw2GmkSOfQGNWDFarSx2gq9dD/R4gUz5J1X6XVJ+IvvWz y6M3dda45nlYqZ05dheJD89SuSr/3wFCsVTJM3TaA7ZQNg/56Hyfc/1dVPGFufSQrsPw HCoA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=8ZgAvEajl8OUNniz+1itXG7C5l6k2fF6JkJiBsO4nok=; b=dfFcoK2iEN0HhdA54vRm9Eh40U1NaWGQ8OtAnP5UX2cEYrkN8pheq5kIHIBXz3OYVf bZjqUUPrAPrT2Xto0vv4SOJkOuQRU7WIFq2XVq7D2EAnA3xT6ak8eeYNP5jAiFx1PRvO Fg8atSie2Y8V0b4/LTwNfAzVu6kHD8HSesPdknY12/EFJ2Osh3uJXHdOfmSXXQyjfgjQ mIAKJeawIjV5rS9hLaY2u8+4wHGCs2e3AU2QS6XthJLSghhGcRsskrs3evXo6iOgsctg Ghd00S4EV/utAnqpZvQb1Kbj7dOZihuflP9vPGKQFRuG209CjGqUMCrebid/IOOoE6Uh hjLQ== X-Gm-Message-State: AOAM530z1VrctkgC+QWBnFHJPQRmKPDGAU33LuqbBjK/kAFjoMZrNPqu 8jjwP5i8ckfbL5TI4u9m7FE= X-Google-Smtp-Source: ABdhPJyJvT/YDmhSIb/x8r7uRaFvl000p8edHD/mRjYqlce+22wUNtkbzecqkR/f+Bcr9lN5Wu7enA== X-Received: by 2002:a0c:fe0b:: with SMTP id x11mr656852qvr.18.1612464723755; Thu, 04 Feb 2021 10:52:03 -0800 (PST) Received: from ?IPv6:2601:5c0:c200:27c6:cd86:137:1075:f377? ([2601:5c0:c200:27c6:cd86:137:1075:f377]) by smtp.gmail.com with ESMTPSA id 186sm6052286qkh.30.2021.02.04.10.52.02 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 04 Feb 2021 10:52:03 -0800 (PST) Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\)) Subject: Re: [PATCH v6 02/10] Revert "drm/qxl: do not run release if qxl failed to init" From: Tong Zhang In-Reply-To: Date: Thu, 4 Feb 2021 13:52:02 -0500 Message-Id: <60B8023C-78C9-441D-AA21-A13C4445F666@gmail.com> References: <20210204145712.1531203-1-kraxel@redhat.com> <20210204145712.1531203-3-kraxel@redhat.com> To: Thomas Zimmermann X-Mailer: Apple Mail (2.3654.60.0.2.21) X-Mailman-Approved-At: Thu, 04 Feb 2021 19:34:49 +0000 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: David Airlie , open list , dri-devel@lists.freedesktop.org, "open list:DRM DRIVER FOR QXL VIRTUAL GPU" , Gerd Hoffmann , Dave Airlie , "open list:DRM DRIVER FOR QXL VIRTUAL GPU" Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" SGkgVGhvbWFzLAoKVGhlIG9yaWdpbmFsIHByb2JsZW0gd2FzIHF4bF9kZXZpY2VfaW5pdCgpIGNh biBmYWlsLAp3aGVuIGl0IGZhaWxzIHRoZXJlIGlzIG5vIG5lZWQgdG8gY2FsbCAKCXF4bF9tb2Rl c2V0X2ZpbmkocWRldik7CglxeGxfZGV2aWNlX2ZpbmkocWRldik7CkJ1dCB0aG9zZSB0d28gZnVu Y3Rpb25zIGFyZSBvdGhlcndpc2UgY2FsbGVkIGluIHRoZSBxeGxfZHJtX3JlbGVhc2UoKSAtCgpJ IGhhdmUgcG9zdGVkIGFuIHVwZGF0ZWQgcGF0Y2guClRoZSBuZXcgcGF0Y2ggdXNlIHRoZSBmb2xs b3dpbmcgbG9naWMKCisJaWYgKCFxZGV2LT5kZGV2Lm1vZGVfY29uZmlnLmZ1bmNzKQorCSAgcmV0 dXJuOwoJcXhsX21vZGVzZXRfZmluaShxZGV2KTsKCXF4bF9kZXZpY2VfZmluaShxZGV2KTsKClRo YW5rcywKLSBUb25nCgoKPiBPbiBGZWIgNCwgMjAyMSwgYXQgMTozNCBQTSwgVGhvbWFzIFppbW1l cm1hbm4gPHR6aW1tZXJtYW5uQHN1c2UuZGU+IHdyb3RlOgo+IAo+IEhpCj4gCj4gQW0gMDQuMDIu MjEgdW0gMTU6NTcgc2NocmllYiBHZXJkIEhvZmZtYW5uOgo+PiBUaGlzIHJldmVydHMgY29tbWl0 IGI5MTkwN2E2MjQxMTkzNDY1Y2E5MmUzNTdhZGYxNjgyMjI0MjI5NmQuCj4gCj4gVGhpcyBzaG91 bGQgYmUgaW4gdGhlIGNvcnJlY3QgZm9ybWF0LCBhcyBnaXZlbiBieSAnZGltIGNpdGUnLgo+IAo+ IGRpbSBjaXRlIGI5MTkwN2E2MjQxMTkzNDY1Y2E5MmUzNTdhZGYxNjgyMjI0MjI5NmQKPiBiOTE5 MDdhNjI0MTEgKCJkcm0vcXhsOiBkbyBub3QgcnVuIHJlbGVhc2UgaWYgcXhsIGZhaWxlZCB0byBp bml0IikKPiAKPj4gUGF0Y2ggaXMgYnJva2VuLCBpdCBlZmZlY3RpdmVseSBtYWtlcyBxeGxfZHJt X3JlbGVhc2UoKSBhIG5vcAo+PiBiZWNhdXNlIG9uIG5vcm1hbCBkcml2ZXIgc2h1dGRvd24gcXhs X2RybV9yZWxlYXNlKCkgaXMgY2FsbGVkCj4+ICphZnRlciogZHJtX2Rldl91bnJlZ2lzdGVyKCku Cj4+IENjOiBUb25nIFpoYW5nIDx6dG9uZzAwMDFAZ21haWwuY29tPgo+PiBTaWduZWQtb2ZmLWJ5 OiBHZXJkIEhvZmZtYW5uIDxrcmF4ZWxAcmVkaGF0LmNvbT4KPj4gLS0tCj4+ICBkcml2ZXJzL2dw dS9kcm0vcXhsL3F4bF9kcnYuYyB8IDIgLS0KPj4gIDEgZmlsZSBjaGFuZ2VkLCAyIGRlbGV0aW9u cygtKQo+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL3F4bC9xeGxfZHJ2LmMgYi9kcml2 ZXJzL2dwdS9kcm0vcXhsL3F4bF9kcnYuYwo+PiBpbmRleCAzNGM4YjI1YjU3ODAuLmZiNWY2YTVl ODFkNyAxMDA2NDQKPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL3F4bC9xeGxfZHJ2LmMKPj4gKysr IGIvZHJpdmVycy9ncHUvZHJtL3F4bC9xeGxfZHJ2LmMKPj4gQEAgLTE0NCw4ICsxNDQsNiBAQCBz dGF0aWMgdm9pZCBxeGxfZHJtX3JlbGVhc2Uoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPj4gIAkg KiByZW9kZXJpbmcgcXhsX21vZGVzZXRfZmluaSgpICsgcXhsX2RldmljZV9maW5pKCkgY2FsbHMg aXMKPj4gIAkgKiBub24tdHJpdmlhbCB0aG91Z2guCj4+ICAJICovCj4+IC0JaWYgKCFkZXYtPnJl Z2lzdGVyZWQpCj4+IC0JCXJldHVybjsKPiAKPiBJJ20gbm90IHN1cmUgd2hhdCB0aGUgb3JpZ2lu YWwgcHJvYmxlbSB3YXMsIGJ1dCBJJ20gc3VyZSB0aGF0IHRoaXMgaXNuJ3QgdGhlIGZpeCBmb3Ig aXQuIElmIHRoZXJlJ3MgYSBwcm9ibGVtIHdpdGggc2h1dGRvd24sIHRoZSBvcGVyYXRpb25zIHJh dGhlciBoYXZlIHRvIGJlIHJlb3JkZXJlZCBjb3JyZWN0bHkuCj4gCj4gV2l0aCB0aGUgY2l0YXRp b24gc3R5bGUgYWRkcmVzczoKPiAKPiBBY2tlZC1ieTogVGhvbWFzIFppbW1lcm1hbm4gPHR6aW1t ZXJtYW5uQHN1c2UuZGU+Cj4gCj4+ICAJcXhsX21vZGVzZXRfZmluaShxZGV2KTsKPj4gIAlxeGxf ZGV2aWNlX2ZpbmkocWRldik7Cj4+ICB9Cj4gCj4gLS0gCj4gVGhvbWFzIFppbW1lcm1hbm4KPiBH cmFwaGljcyBEcml2ZXIgRGV2ZWxvcGVyCj4gU1VTRSBTb2Z0d2FyZSBTb2x1dGlvbnMgR2VybWFu eSBHbWJICj4gTWF4ZmVsZHN0ci4gNSwgOTA0MDkgTsO8cm5iZXJnLCBHZXJtYW55Cj4gKEhSQiAz NjgwOSwgQUcgTsO8cm5iZXJnKQo+IEdlc2Now6RmdHNmw7xocmVyOiBGZWxpeCBJbWVuZMO2cmZm ZXIKPiAKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRy aS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw czovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=