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;
Received-SPF: TempError (protection.outlook.com: error in processing during
 lookup of arm.com: DNS Timeout)
From:   Mihail Atanassov <Mihail.Atanassov@arm.com>
To:     "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
CC:     Mihail Atanassov <Mihail.Atanassov@arm.com>, nd <nd@arm.com>,
        Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Sean Paul <sean@poorly.run>, Liviu Dudau <Liviu.Dudau@arm.com>,
        Brian Starkey <Brian.Starkey@arm.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        "james qian wang (Arm Technology China)" <james.qian.wang@arm.com>,
        "Lowry Li (Arm Technology China)" <Lowry.Li@arm.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: [PATCH 1/5] drm/komeda: Add debugfs node to control error verbosity
Thread-Topic: [PATCH 1/5] drm/komeda: Add debugfs node to control error
 verbosity
Thread-Index: AQHViC8xuPMk/kmsbUCVoX21Aq4jyg==
Date:   Mon, 21 Oct 2019 16:47:14 +0000
Message-ID: <20191021164654.9642-2-mihail.atanassov@arm.com>
References: <20191021164654.9642-1-mihail.atanassov@arm.com>
In-Reply-To: <20191021164654.9642-1-mihail.atanassov@arm.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
Authentication-Results-Original: spf=none (sender IP is )
 smtp.mailfrom=Mihail.Atanassov@arm.com; 
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-publictraffictype: Email
X-MS-Office365-Filtering-Correlation-Id: b5f6b1e0-5bc3-49b2-124a-08d756465d12
X-MS-Office365-Filtering-HT: Tenant
X-MS-TrafficTypeDiagnostic: VI1PR08MB3231:|VI1PR08MB3231:|VI1PR08MB3677:
x-ms-exchange-transport-forked: True
X-Microsoft-Antispam-PRVS: <VI1PR08MB3677610B61331B1C8F95B9C58F690@VI1PR08MB3677.eurprd08.prod.outlook.com>
x-checkrecipientrouted: true
x-ms-oob-tlc-oobclassifiers: OLM:1850;OLM:1850;
x-forefront-prvs: 0197AFBD92
X-Forefront-Antispam-Report-Untrusted: SFV:NSPM;SFS:(10009020)(4636009)(39860400002)(346002)(136003)(376002)(366004)(396003)(189003)(199004)(66476007)(8936002)(2616005)(81166006)(81156014)(386003)(50226002)(44832011)(76176011)(6506007)(2501003)(102836004)(1076003)(25786009)(71200400001)(6486002)(3846002)(6116002)(476003)(11346002)(316002)(71190400001)(256004)(446003)(99286004)(486006)(54906003)(14454004)(26005)(5640700003)(86362001)(305945005)(7736002)(8676002)(6436002)(6916009)(5660300002)(4326008)(6512007)(66446008)(64756008)(66946007)(66556008)(478600001)(36756003)(52116002)(66066001)(2906002)(2351001)(186003);DIR:OUT;SFP:1101;SCL:1;SRVR:VI1PR08MB3231;H:VI1PR08MB4078.eurprd08.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1;
received-spf: None (protection.outlook.com: arm.com does not designate
 permitted sender hosts)
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam-Untrusted: BCL:0;
X-Microsoft-Antispam-Message-Info-Original: 2VO7Tqx8aoVFFuhlMZWvq3Ph9VmYVldu8iY4dT0oxoV/uGfZkejSq9tmU8ER9/GMJLDzycAzgZ6PpCk3SsFpRwJStC+XsqlohhX7YpP/fhPZobAfQaWTqU63qzwkR1tN4S6lxWFT1Qu1KrMKc+6cuULSe390LxXKveDNzih230o+FNaHu3sc+2UxYZDgivXELbZkIAkP9hiy6uVCmJx87ZS+ZA4/0uYS3smndEzRlHCNPJ91WOgUAPz1nhi6euLd8Zv6YN3i5aTtcZ75OZXqYFrQsYLIFYxH0oBfEkujsZZHfhfvI7OLZP2I5B9YYY+c3xHPe92vfWZDEr8BcevVqUMbxyxw27nel7y+7YPY2sw7mfOEGorJiCX41H9ZXJFn1Ouo5vlozzRHDWBvOQOXAI6NCVFUgP2KSbFziUyG8SvAbl8HuV3AhFxwe/S+CpS2
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3231
Original-Authentication-Results: spf=none (sender IP is )
 smtp.mailfrom=Mihail.Atanassov@arm.com; 
X-EOPAttributedMessage: 0
X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT028.eop-EUR03.prod.protection.outlook.com
X-Forefront-Antispam-Report: CIP:63.35.35.123;IPV:CAL;SCL:-1;CTRY:IE;EFV:NLI;SFV:NSPM;SFS:(10009020)(4636009)(39860400002)(136003)(346002)(396003)(376002)(189003)(199004)(7736002)(305945005)(47776003)(26826003)(22756006)(6862004)(4326008)(23756003)(86362001)(50466002)(6486002)(2906002)(478600001)(2351001)(14454004)(1076003)(316002)(2501003)(70586007)(70206006)(50226002)(54906003)(76130400001)(486006)(336012)(3846002)(63350400001)(446003)(126002)(8746002)(36906005)(11346002)(99286004)(356004)(102836004)(2616005)(66066001)(186003)(81156014)(81166006)(5640700003)(6512007)(25786009)(6116002)(6506007)(8936002)(76176011)(8676002)(5660300002)(386003)(476003)(26005)(36756003);DIR:OUT;SFP:1101;SCL:1;SRVR:VI1PR08MB3677;H:64aa7808-outbound-1.mta.getcheckrecipient.com;FPR:;SPF:TempError;LANG:en;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com;MX:1;A:1;
X-MS-Office365-Filtering-Correlation-Id-Prvs: f0cabc26-5c2c-4b5e-39ce-08d756465375
NoDisclaimer: True
X-Forefront-PRVS: 0197AFBD92
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: jF3NqNwo0Y0EpQwrZ7GrGXEUmNLabyIl7N8o8734necETB15nueAU4Oy4dqBERag5i/hrXGL0+2yhO1QBqqGuVz3+eYVNOQ2HIlFjEJ/a8QfRVixj50w6TEDBJ/gOUE0+orslYXGCKieqAGREXYYSz9I630SSeEfuWs7j9OJChtNZuaEG5IW8eeykXPi002Q9n77XS2aBxKLXeYQhL5Ch1kJeoy7f/Ps0q3zNKHfy0/JviNUPn9RnXz92yRIza3X/aTQavmoycYNBEfEDg1WwT/v9aGRs9vBhNIjZifczL0lMjsKYSV9HtjwGFy+MUNe9hthnJVQh3FoYWiswEGop6AObFqXEB/pp62fN+Fhr65ltAOIxGxaU0JB4yWgImtazZcHQS8NaTcDCBgrj07+VcERr6pwe9/oVKIj9RHRf6E=
X-OriginatorOrg: arm.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Oct 2019 16:47:30.2927
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: b5f6b1e0-5bc3-49b2-124a-08d756465d12
X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3677
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Named 'err_verbosity', currently with only 1 active bit in that
replicates the existing level - print error events once per flip.

Signed-off-by: Mihail Atanassov <mihail.atanassov@arm.com>
---
 drivers/gpu/drm/arm/display/komeda/komeda_dev.c   |  4 ++++
 drivers/gpu/drm/arm/display/komeda/komeda_dev.h   | 14 ++++++++++++--
 drivers/gpu/drm/arm/display/komeda/komeda_event.c |  9 +++++++--
 drivers/gpu/drm/arm/display/komeda/komeda_kms.c   |  2 +-
 4 files changed, 24 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/=
drm/arm/display/komeda/komeda_dev.c
index 937a6d4c4865..82230c0ddec3 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
@@ -58,6 +58,8 @@ static void komeda_debugfs_init(struct komeda_dev *mdev)
 	mdev->debugfs_root =3D debugfs_create_dir("komeda", NULL);
 	debugfs_create_file("register", 0444, mdev->debugfs_root,
 			    mdev, &komeda_register_fops);
+	debugfs_create_x16("err_verbosity", 0664, mdev->debugfs_root,
+			   &mdev->err_verbosity);
 }
 #endif
=20
@@ -280,6 +282,8 @@ struct komeda_dev *komeda_dev_create(struct device *dev=
)
 		goto err_cleanup;
 	}
=20
+	mdev->err_verbosity =3D KOMEDA_DEV_PRINT_ERR_EVENTS;
+
 #ifdef CONFIG_DEBUG_FS
 	komeda_debugfs_init(mdev);
 #endif
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.h b/drivers/gpu/=
drm/arm/display/komeda/komeda_dev.h
index 414200233b64..b5bd3d5898ee 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.h
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.h
@@ -202,6 +202,14 @@ struct komeda_dev {
=20
 	/** @debugfs_root: root directory of komeda debugfs */
 	struct dentry *debugfs_root;
+	/**
+	 * @err_verbosity: bitmask for how much extra info to print on error
+	 *
+	 * See KOMEDA_DEV_* macros for details.
+	 */
+	u16 err_verbosity;
+	/* Print a single line per error per frame with error events. */
+#define KOMEDA_DEV_PRINT_ERR_EVENTS BIT(0)
 };
=20
 static inline bool
@@ -219,9 +227,11 @@ void komeda_dev_destroy(struct komeda_dev *mdev);
 struct komeda_dev *dev_to_mdev(struct device *dev);
=20
 #ifdef CONFIG_DRM_KOMEDA_ERROR_PRINT
-void komeda_print_events(struct komeda_events *evts);
+void komeda_print_events(struct komeda_events *evts, struct drm_device *de=
v);
 #else
-static inline void komeda_print_events(struct komeda_events *evts) {}
+static inline void komeda_print_events(struct komeda_events *evts,
+				       struct drm_device *dev)
+{}
 #endif
=20
 int komeda_dev_resume(struct komeda_dev *mdev);
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_event.c b/drivers/gp=
u/drm/arm/display/komeda/komeda_event.c
index a36fb86cc054..575ed4df74ed 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_event.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_event.c
@@ -107,10 +107,12 @@ static bool is_new_frame(struct komeda_events *a)
 	       (KOMEDA_EVENT_FLIP | KOMEDA_EVENT_EOW);
 }
=20
-void komeda_print_events(struct komeda_events *evts)
+void komeda_print_events(struct komeda_events *evts, struct drm_device *de=
v)
 {
-	u64 print_evts =3D KOMEDA_ERR_EVENTS;
+	u64 print_evts =3D 0;
 	static bool en_print =3D true;
+	struct komeda_dev *mdev =3D dev->dev_private;
+	u16 const err_verbosity =3D mdev->err_verbosity;
=20
 	/* reduce the same msg print, only print the first evt for one frame */
 	if (evts->global || is_new_frame(evts))
@@ -118,6 +120,9 @@ void komeda_print_events(struct komeda_events *evts)
 	if (!en_print)
 		return;
=20
+	if (err_verbosity & KOMEDA_DEV_PRINT_ERR_EVENTS)
+		print_evts |=3D KOMEDA_ERR_EVENTS;
+
 	if ((evts->global | evts->pipes[0] | evts->pipes[1]) & print_evts) {
 		char msg[256];
 		struct komeda_str str;
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/=
drm/arm/display/komeda/komeda_kms.c
index d49772de93e0..e30a5b43caa9 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c
@@ -48,7 +48,7 @@ static irqreturn_t komeda_kms_irq_handler(int irq, void *=
data)
 	memset(&evts, 0, sizeof(evts));
 	status =3D mdev->funcs->irq_handler(mdev, &evts);
=20
-	komeda_print_events(&evts);
+	komeda_print_events(&evts, drm);
=20
 	/* Notify the crtc to handle the events */
 	for (i =3D 0; i < kms->n_crtcs; i++)
--=20
2.23.0