[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v7 1/7] xen/console: promote conring{,_size} to __ro_after_init
- To: xen-devel@xxxxxxxxxxxxxxxxxxxx
- From: dmukhin@xxxxxxxx
- Date: Mon, 13 Jul 2026 11:16:13 -0700
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 205.220.161.53) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=ford.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=ford.com; dkim=pass (signature was verified) header.d=saarlouis.ford.com; dkim=pass (signature was verified) header.d=ford.com; arc=none (0)
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=JIyiTu7QCzAjlfuDW1hzX2VhrdEr2rHhHBlHANAcsfE=; b=pDAO5Ws9eLR4uog8IaCUqAC34NaZUvVnkASs46LjX6wVe6ziDSjXCId+4MnXcE54m8Xl72FefIFiPot2hmZBxBg12a9hGSjKvDIX3c3sLb4bn/bo+JO0hGf3Fip2KiUMrDLZmH9aXcrdDNd5+Si+2zwmQIU3wEX8u9MRu5KbeixsXMKlv1hImTODO7g+8aO/hCEq8V2aG4/5t9xAM9saUcIBnk944y2HVS4Cqff31VefMXGnHl4CRS4rXEbqyrRwq3AjXmveim2d/mP/U2rGzzZaRHhkbupMtcQjHEwzHG/0+COVskoCpFcbWLjLEZPD+FglTJ2KlzhqtuRMUkkwPA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=gVncxdmsDfMW+zhlrtDcOSNF6CIFvQgv7FwiZ/yWDld2x4/CyUSIdPAsvLDAcCEbgBiuYs+YD9HXyIgZ5+AG80YfCZSdGtmQ/9BJqDZVHp2fO8R3WfG/PNg4AzL5r9TlRIUONFhgrI0c5Ffr1Hxcb07Ew3wT+0uiw84FkVEls2v/oPD9wb8ac1OrvpzwH4hLsrhXPzzFNVYkjit2I2BkI9SoilrpMBRv3s4obo6eR4O/NxWaD+LLTvCOcJBCXoW215zH7+c87pU852fuImi/v8tyQUTBPOvEjMi5xKFojjqUizY1Y6FMTaUCqSDEhYu4CF8HryaZGRumT0occ6a9IQ==
- Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=ppford header.d=ford.com header.i="@ford.com" header.h="Cc:Content-Transfer-Encoding:Content-Type:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"; dkim=pass header.s=selector2-azureford-onmicrosoft-com header.d=azureford.onmicrosoft.com header.i="@azureford.onmicrosoft.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck"; dkim=pass header.s=ppserprodsaar header.d=saarlouis.ford.com header.i="@saarlouis.ford.com" header.h="Cc:Content-Transfer-Encoding:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"; dkim=pass header.s=ppfserpocford header.d=ford.com header.i="@ford.com" header.h="Cc:Content-Transfer-Encoding:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"
- Cc: andrew.cooper3@xxxxxxxxxx, anthony.perard@xxxxxxxxxx, jbeulich@xxxxxxxx, julien@xxxxxxx, michal.orzel@xxxxxxx, roger.pau@xxxxxxxxxx, sstabellini@xxxxxxxxxx, dmukhin@xxxxxxxx
- Delivery-date: Mon, 13 Jul 2026 18:16:41 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Pser-m365-app: SER-APP
From: Denis Mukhin <dmukhin@xxxxxxxx>
Both conring{,_size} should be RO after initialization is completed.
Change the conring integer parameters type to `unsigned int` as required
by CODING_STYLE.
Suggested-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Signed-off-by: Denis Mukhin <dmukhin@xxxxxxxx>
---
Changes since v6:
- rebased
---
xen/drivers/char/console.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c
index dbda7f259668..5ebbbf63c092 100644
--- a/xen/drivers/char/console.c
+++ b/xen/drivers/char/console.c
@@ -340,15 +340,15 @@ static void cf_check do_dec_thresh(unsigned char key,
bool unused)
*/
/* conring_size: allows a larger console ring than default (16kB). */
-static uint32_t __initdata opt_conring_size;
+static unsigned int __initdata opt_conring_size;
size_param("conring_size", opt_conring_size);
#define _CONRING_SIZE 16384
#define CONRING_IDX_MASK(i) ((i)&(conring_size-1))
static char __initdata _conring[_CONRING_SIZE];
-static char *__read_mostly conring = _conring;
-static uint32_t __read_mostly conring_size = _CONRING_SIZE;
-static uint32_t conringc, conringp;
+static char *__ro_after_init conring = _conring;
+static unsigned int __ro_after_init conring_size = _CONRING_SIZE;
+static unsigned int conringc, conringp;
static void cf_check conring_notify(void *unused)
{
--
2.54.0
|
