Received: by 2002:ac0:a5b6:0:0:0:0:0 with SMTP id m51-v6csp3293016imm; Mon, 4 Jun 2018 00:38:38 -0700 (PDT) X-Google-Smtp-Source: ADUXVKJAtVFkb4/2y7QfaLlZzlevL+P3vCy3B0PCyJFYTbuq0vABUQbyP8rARP8L/ZuTNSNK0Xax X-Received: by 2002:a62:8a83:: with SMTP id o3-v6mr3573634pfk.12.1528097918668; Mon, 04 Jun 2018 00:38:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1528097918; cv=none; d=google.com; s=arc-20160816; b=myO5lbq7Twgsq9gjH6jpY9y3EH7MI5t/7qliOl9HgpCoQmfWDlubmAz+J+5xxjkg9I 6OG+j3ehOw0YbrcmJ0xpZBvrsMHBanQLZXnmlRKWj8o/MhHJYGoq5fOQllSo+7fO68Gh V4jIeNUQyTOePaKPYyDS3EnDYPINnrHKDjfCwMpfQJdpG2AlgDve3WD+fCjwRaDjWyG4 410dENAOdZdwHgSV9326KWBw3zVhJOSUQkFsWj2F97qsgxmWjICdoadE5BeZB7P6vw5w C/0MVniBDdQn9sohbiTHNfYq/Msqwe99YXj2OggJmXnafDoOMobxpiaptun5/uAtXUFp CHIA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=WDxe9R5I7xjptG45aT2xIMyiQazZkJFP9zmK0mQeJsQ=; b=etTshKmoXZXUxoEBvaNF7zvI/Dk/xbAbebkqmCjCLxFvB3HnwbtnF6I0mOlxKcO4ae 73SR/M41hCczhBoKjnMlUc3uxjEJI7QfcrhdhGzBHNr2jHGznnBCRL5tIic2gdc186P6 W8XImtV1xS+At4ICFMsYVBkzdBQcMjTqSsBumLFAxjuLctijzkSOkjixdS/TcsYAPrhv M3HJU7oZFIHGkfdhYjdqrr7Pw4tAQEpf4NFwWRwah/HEuhUZepBnRSjSlXL+vEeQmmLU smcH17l5E3DDgDBV6XGobI0fFlr7Qfyarxq3szyxTNNlxDv13Kh9RykikSd0pCVb6Hpe +hlQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n5-v6si295953pfi.360.2018.06.04.00.38.21; Mon, 04 Jun 2018 00:38:38 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751937AbeFDHh0 (ORCPT + 99 others); Mon, 4 Jun 2018 03:37:26 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:37814 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750819AbeFDHhZ (ORCPT ); Mon, 4 Jun 2018 03:37:25 -0400 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 1F3E2814F0B8; Mon, 4 Jun 2018 07:37:25 +0000 (UTC) Received: from sirius.home.kraxel.org (ovpn-116-24.ams2.redhat.com [10.36.116.24]) by smtp.corp.redhat.com (Postfix) with ESMTP id C72522144B20; Mon, 4 Jun 2018 07:37:23 +0000 (UTC) Received: by sirius.home.kraxel.org (Postfix, from userid 1000) id 3C66B1750E; Mon, 4 Jun 2018 09:37:23 +0200 (CEST) Date: Mon, 4 Jun 2018 09:37:23 +0200 From: Gerd Hoffmann To: Jeremy Cline Cc: Dave Airlie , Ray Strode , virtualization@lists.linux-foundation.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH] drm/qxl: Call qxl_bo_unref outside atomic context Message-ID: <20180604073723.ld7b2h5veqsxoun6@sirius.home.kraxel.org> References: <20180601200532.13619-1-jcline@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180601200532.13619-1-jcline@redhat.com> User-Agent: NeoMutt/20180512 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.6 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Mon, 04 Jun 2018 07:37:25 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.8]); Mon, 04 Jun 2018 07:37:25 +0000 (UTC) for IP:'10.11.54.6' DOMAIN:'int-mx06.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'kraxel@redhat.com' RCPT:'' Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jun 01, 2018 at 04:05:32PM -0400, Jeremy Cline wrote: > "qxl_bo_unref" may sleep, but calling "qxl_release_map" causes > "preempt_disable()" to be called and "preempt_enable()" isn't called > until "qxl_release_unmap" is used. Move the call to "qxl_bo_unref" out > from in between the two to avoid sleeping from an atomic context. > > This issue can be demonstrated on a kernel with CONFIG_LOCKDEP=y by > creating a VM using QXL, using a desktop environment using Xorg, then > moving the cursor on or off a window. > > Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1571128 > Fixes: 9428088c90b6 ("drm/qxl: reapply cursor after resetting primary") > Cc: stable@vger.kernel.org > Signed-off-by: Jeremy Cline Pushed to drm-misc-fixes. thanks, Gerd