Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp3365947ybl; Mon, 19 Aug 2019 17:34:35 -0700 (PDT) X-Google-Smtp-Source: APXvYqwbvPEq83dWwoTQPEHzQ1AIw9z4IQYp96+SGrMWb7VolgB+5cFosr90Nkk7MrtrwCcpu//7 X-Received: by 2002:a63:5a0a:: with SMTP id o10mr22645582pgb.282.1566261275358; Mon, 19 Aug 2019 17:34:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1566261275; cv=none; d=google.com; s=arc-20160816; b=Rt5iKWCJ7ovRFibLKWMUZNBmBICc78VDRcAH8IYQUiESDtrisbzlZWacpHuN/vdeaY cBkMZTejVeA7Zay4Lr5mZnV0chrUyJ/K38534bpBiGSALRa6hA/5OkJOiFBFxOtYuS// 38UEvHYGtkMy25Ef2Y5B3PL+wIIAOTbnrpx9WmFH0g3t4S+4OEYgCLljR59fb4ugg7ED w8Rlj6EYEK37bMzdzQ0O0C56zPWNL3lkTm6krTOKXK9VFOpe6drDPrO6fZ0KAfU5501I pbxFm9ctGtChApiS9j2+AVARlO/3hUuZYhzSW1I110k6jPXhof/8NlXd1NeiDC6GuxC1 Pxow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=wyMT0jwhb6wCowCFW/o5BpgRu7EcFHVU+3LZ1Fjgnh8=; b=YAFeoUjUVju4zePt4Ofnrn5EEfYfJKimEQOlPHUeShufL4qsvdZcHD4cujBjnQu3wM t1SFF7PGMY/yY1xbOqYtWU19mcjAeI8pxtlTKwslW/MT7SL0biMlDFXJwtIbF4Nr78Hm gfMiySORl4NDQ3OtFJE7PYwR5dtUsj9h7CyflFZZ3xFA8Y/IYyNmdPZfWO7oZ5gEEaHX UltBESc2mhiMQEjddge32a5n/t402AjO/QHQEFJROfTOClFYa1/ikJYhZnE8jUk6FAOq S5qSux4hP3rfFWJT9/k9AJ/4RsI6+3MVHETsO0hOLTZ+UzgupV+jKUyFFXigIqiHWUZo 04Kg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Bwf1VA32; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y3si11110295pge.429.2019.08.19.17.34.19; Mon, 19 Aug 2019 17:34:35 -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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Bwf1VA32; 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=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728782AbfHTAda (ORCPT + 99 others); Mon, 19 Aug 2019 20:33:30 -0400 Received: from mail-yb1-f196.google.com ([209.85.219.196]:46934 "EHLO mail-yb1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728580AbfHTAd3 (ORCPT ); Mon, 19 Aug 2019 20:33:29 -0400 Received: by mail-yb1-f196.google.com with SMTP id x10so1253097ybs.13; Mon, 19 Aug 2019 17:33:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=wyMT0jwhb6wCowCFW/o5BpgRu7EcFHVU+3LZ1Fjgnh8=; b=Bwf1VA32s3fDTHNmDZ0aPk7evhArYfMfbe6KbqijJ60jU6wBmb+ee6hqfQHNY8FqOb 3nxlWu1cW4m4juHjPuG9B/HrhdgbpZlw2lXk5ChuR9+F4X/scMZTdC+kQrjv5lB6jWFN 5Ppl1Wyxldz15g3MEAfqGm2zOCQ8SxwVba19XyWWwFlv/j3BCkLvTd7a8X+I8bJAhSF9 EGFX/Yu/vjJ74CwYn2BddqgK+FHwQTuyERpSsc7vGdvX4w9+/LIagx1V4T0AEWBeoBFL SDwwEB2gXZqCdeQrxSNW3qB9iFSpLujlQT9mgsSHLQOJv8XmV5GKY5ZH59owdKQRAr8P BTZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=wyMT0jwhb6wCowCFW/o5BpgRu7EcFHVU+3LZ1Fjgnh8=; b=mYjnQbh9lgcyAR1hgbRseUD9sPBaITHnxdbso3ebhk8Bi2qLLAsydS3Y4r990FuBN7 6+1gqn4RDCH4z8IavTQFcbQWKdWc9bpVxiskHt6tBuMXeld19UQMR0/1kzJD4mSGx2M8 h0HTmZa+pxNBbkrRiuvZ+I6V3QT+aVIvC9Tn2VXTl6oPk6aNeGB23c2VEMrJBLd/9OKq yN6Js2bQkfga8ozkGU/WsebmT7SIkQ4M4b65w68aUUYM6M3RitbvWdxLc4eas4FplQIF SuisS6SfKQETLs3ZXRM0hE017QZ5tUm/Tfxkay/OwnB2aMbzgyhaEebK4pKmdnBNGLqm 4W7Q== X-Gm-Message-State: APjAAAUig0JPcCL7LdA9YUD4ChUCDCvcpmlTx6w0jx9mYzpOPwzpxMoD 3/RlHgtLocOlw0F8zdDYOFRWufqy X-Received: by 2002:a25:aca:: with SMTP id 193mr16921407ybk.521.1566261208611; Mon, 19 Aug 2019 17:33:28 -0700 (PDT) Received: from theseus.lan ([2604:2d80:b386:1f00::780]) by smtp.gmail.com with ESMTPSA id 193sm3658853ywh.89.2019.08.19.17.33.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Aug 2019 17:33:28 -0700 (PDT) From: Clark Williams To: bigeasy@linutronix.com Cc: tglx@linutronix.com, linux-rt-users@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PREEMPT_RT PATCH 0/3] i915 fixups for lockdep/lockdebugging Date: Mon, 19 Aug 2019 19:33:16 -0500 Message-Id: <20190820003319.24135-1-clark.williams@gmail.com> X-Mailer: git-send-email 2.21.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Clark Williams The i915 driver was throwing splats on my home test box running v5.2-rt3 when I turned on lockdep and lock debugging configs. This was mainly due to the non-side effects of the spin*_irq*() macros which do nothing to IRQs on PREEMPT_RT. Converting the various irq_lock spinlocks and the uncore lock to be raw_spinlock_t fixes this, although I'm not sure of the performance implications for latency spikes. Testing was done on an Intel NUC with the following graphics: 00:02.0 VGA compatible controller: Intel Corporation Iris Plus Graphics 650 (rev 06) Testing on other variants of i915 would be appreciated. Clark Williams (3): i915: do not call lockdep_assert_irqs_disabled() on PREEMPT_RT i915: convert all irq_locks spinlocks to raw spinlocks i915: convert uncore lock to raw spinlock drivers/gpu/drm/i915/i915_debugfs.c | 8 +- drivers/gpu/drm/i915/i915_drv.c | 6 +- drivers/gpu/drm/i915/i915_drv.h | 2 +- drivers/gpu/drm/i915/i915_gem.c | 4 +- drivers/gpu/drm/i915/i915_irq.c | 154 ++++++++++----------- drivers/gpu/drm/i915/i915_pmu.c | 4 +- drivers/gpu/drm/i915/intel_breadcrumbs.c | 51 +++---- drivers/gpu/drm/i915/intel_display.c | 20 +-- drivers/gpu/drm/i915/intel_engine_cs.c | 4 +- drivers/gpu/drm/i915/intel_engine_types.h | 2 +- drivers/gpu/drm/i915/intel_fifo_underrun.c | 16 +-- drivers/gpu/drm/i915/intel_guc.c | 6 +- drivers/gpu/drm/i915/intel_guc.h | 10 +- drivers/gpu/drm/i915/intel_hotplug.c | 36 ++--- drivers/gpu/drm/i915/intel_pm.c | 8 +- drivers/gpu/drm/i915/intel_runtime_pm.c | 8 +- drivers/gpu/drm/i915/intel_sprite.c | 32 ++--- drivers/gpu/drm/i915/intel_tv.c | 8 +- drivers/gpu/drm/i915/intel_uncore.c | 52 +++---- drivers/gpu/drm/i915/intel_uncore.h | 2 +- drivers/gpu/drm/i915/intel_workarounds.c | 4 +- 21 files changed, 220 insertions(+), 217 deletions(-) -- 2.21.0