From patchwork Thu Mar 31 09:48:51 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jan Beulich X-Patchwork-Id: 12796959 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D5386C433EF for ; Thu, 31 Mar 2022 09:49:11 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.296851.505438 (Exim 4.92) (envelope-from ) id 1nZrQJ-00061N-IE; Thu, 31 Mar 2022 09:48:59 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 296851.505438; Thu, 31 Mar 2022 09:48:59 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1nZrQJ-00061G-FG; Thu, 31 Mar 2022 09:48:59 +0000 Received: by outflank-mailman (input) for mailman id 296851; Thu, 31 Mar 2022 09:48:58 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1nZrQI-00061A-Am for xen-devel@lists.xenproject.org; Thu, 31 Mar 2022 09:48:58 +0000 Received: from de-smtp-delivery-102.mimecast.com (de-smtp-delivery-102.mimecast.com [194.104.111.102]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id c8b1afa1-b0d7-11ec-8fbc-03012f2f19d4; Thu, 31 Mar 2022 11:48:57 +0200 (CEST) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05lp2108.outbound.protection.outlook.com [104.47.18.108]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-22-vT_QhzW5OQ6fFsj6KKT39g-1; Thu, 31 Mar 2022 11:48:55 +0200 Received: from DU2PR04MB8616.eurprd04.prod.outlook.com (2603:10a6:10:2db::16) by DB8PR04MB5658.eurprd04.prod.outlook.com (2603:10a6:10:b2::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5123.21; Thu, 31 Mar 2022 09:48:53 +0000 Received: from DU2PR04MB8616.eurprd04.prod.outlook.com ([fe80::914d:e08d:7798:8476]) by DU2PR04MB8616.eurprd04.prod.outlook.com ([fe80::914d:e08d:7798:8476%7]) with mapi id 15.20.5123.021; Thu, 31 Mar 2022 09:48:53 +0000 X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: c8b1afa1-b0d7-11ec-8fbc-03012f2f19d4 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=mimecast20200619; t=1648720137; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=+iOZotKkLcJY4CXebIky0sg+x0ejVE7dgfMqo24AelE=; b=mSZ+pWlHMapCjOv8Z5jMUFOZ31LDjO8M+c1k1sGz5DM3c6l1URMMLAM4OaGm/lZYDXL/sx RlaVeNxrerz1JdNARauPanNZ6nGM73MD/0+Mxzz7iHepvnUeuyOUK8hhnRKkd0UQQ8PMB0 k/paZDVLiuQIlGTax664qcdGw+BlYhs= X-MC-Unique: vT_QhzW5OQ6fFsj6KKT39g-1 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cqsBu+6ePMeKPUKtlK+T6vluiV064gyg20OE+23C8O/tqo2sMVI2JTaqiCxoLzZ7uZHHl0khz5kobi+VWWpJzZphHspnbXlhudXNLwltSa8wp4axIPSj3sgMaapjs2Tsvk9qxbwRX4VE/6AZGVaBYjAuM+lsWUDFTtvTdBkpinlUl6aJVl0YIiBYWhCVGyU0i1X1QleG1aX3LgqYT+7uY7iV3x5P2DDfMBs4d/1Zwm3UOVRzBG1612bVGtTDPBjfHrilA7kZ5hmIgk7yN5jaSUuvozkb/RFh8ZXA7aLIklwxbGdTwxm7ieBs+Zfd65WBIM8m2wFsCOrsAdrYoh/g/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=+iOZotKkLcJY4CXebIky0sg+x0ejVE7dgfMqo24AelE=; b=M0BjXn32/8ZRyQH0wWtsIVi6N28EeS6yHYI32DM239fiGn0cXprZqge/BE/aufuRvaX3B0aGYDSIsz3rM/Ot5pgOk34n4l08scdq8bxOXzjkKrVKRLZsfFjgm7rl4q7fNA8l6SxYJiomSqkPnY/ZC9ji3oxbhISSHiFrjCSFz/Nul2YuCdoxGCi/SyOgLsNRM0iD9b5G4ObEEi54OulC9uEh/mCEo5S/yPviqHBtGYGfceDp/nJYlYe6Oepa7EGtG34SuQzjmanJ6MqgVgDg6/N2Om7mpbDLpbqD1p4tHxdo7Km0Ng/GzCLNulnNSotyk9sK67UbEa2ENbLfKwRckQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: Date: Thu, 31 Mar 2022 11:48:51 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: [PATCH v4 4/8] x86/boot: simplify mode_table Content-Language: en-US From: Jan Beulich To: "xen-devel@lists.xenproject.org" Cc: Andrew Cooper , Wei Liu , =?utf-8?q?Roger_Pau_Monn=C3=A9?= References: <819eb552-d49d-e98a-9524-8d30c1b93908@suse.com> In-Reply-To: <819eb552-d49d-e98a-9524-8d30c1b93908@suse.com> X-ClientProxiedBy: AS9PR04CA0066.eurprd04.prod.outlook.com (2603:10a6:20b:48b::19) To DU2PR04MB8616.eurprd04.prod.outlook.com (2603:10a6:10:2db::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a3ebcdac-364f-435a-3742-08da12fbaa48 X-MS-TrafficTypeDiagnostic: DB8PR04MB5658:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: qUfHIdCNJkTPD82OeCwJKceR7ekoUcB7WoLUsRDPY2fskgk8Akw4gOXK8iRF9LPzUlZfLMVv4fR4qYVaUUJwaw7WmLeY3tELTzIj4hPIYM1G/pFHRH/X81MYj0zMRRMrDKFBI13zOsNIi6w4pu20V2uCwTHJkMjeaz2AtNFaIL2rsqKf5bKbUmZXR2BcbvSmG54Q9xLiVoz2PCiFFAfg14m2A/6o8SY05J6lfsxDhr6x9uIs1nrHV6nNEOLF5od9oRl4iQreRGIQsSLxRcbyGpxKHmt5TBFuVkDZGpruGrvJHoZDy+RfHBKcrHCsmFl4uzIoVRiV674nF1cUfJRNuJGBv/dzquJ/fOkP5dRXlgrizXieqgTlsNjUFQjEVf1GDxoIIuIaMyjLQLFCoItWxA+UMVu2/+1bMs/ATnjt72+jIN3blxmX1glBM70rEsEtC+gXof89exBzIG3porE1jXY9Ww5h79hmPPgOr6SjoE7/MHeRJZWZTYby/1yzBbLYU9/Zb1ooQroCTv4SP9jjnoI6wvVKavHNPIpj13ueNxw2PkWR3UzUfOS7aG2DD/FgW8CdiXLebqk/7cpBoHmi4zhG7v7uP3VTUZJbRnRCJVInRrcUDBBOMQ521Ax7pF4EhS6/0f/KBjeDAJWB/ECBjMQPwiZ3ByHqZYzddJNGnpatgvg3e4SqG+gsPKV0sfVUdxw8WuGTfHy9NN5N8XJeqst1gkW2NLyLi5tAT3YdgJF9zYDsJz0LFYvKeZLyjFiF X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU2PR04MB8616.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(366004)(6916009)(54906003)(316002)(66476007)(8676002)(6512007)(6506007)(66946007)(31696002)(86362001)(508600001)(6486002)(66556008)(38100700002)(2616005)(4326008)(5660300002)(26005)(186003)(8936002)(2906002)(36756003)(31686004)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?epzlt0tvjh1BJEL6/odpwIo+hdZ2?= =?utf-8?q?H0jXr52YRb/vpOkI8bSsWRinNa/HoHMrrhBcgNWMfbKdDaUBNpzVJ3b9XRd8z4kSt?= =?utf-8?q?e5uw2LrFxE8jcO6vmdfns7xwFe/HYW9hfdWca8eFs0at9MM3W3It7ufI8HNEkjIHW?= =?utf-8?q?N0QNvTA0NwKbS4iK1bNdag+yeKyAAeQ+D6yJ2HUoXX0zV8U29N4i2vO8JxfnZSnsU?= =?utf-8?q?Ag/l3GA1ptRC4D8xFfcW1/aM/ApDcXihsrSJDjClt5W0wnU9/yIrRIXqf7P16e3O6?= =?utf-8?q?NRVAVBsTp3bPXGgMDvO7VppWLZomfaDQG8baadA1JazB14N0URhoppeO+a8uesV59?= =?utf-8?q?uytUUvRnx+joFt+ICxr7OMCTnS1c5yT53SxLyzEvybJNkWvUTcegb6g9fc4KK5k64?= =?utf-8?q?abLPAw4r208cSpsW69bc4LPWn+pLGysRsNMZm8HemYQXXWQrYudwWg2am8eTM9M2L?= =?utf-8?q?Fl0cfyCHraLZ2ey0eOsObDWC9yyvGR1dV0/gb8gHgyoipF42kRIZzlUvyeEJf9E6x?= =?utf-8?q?DUkXfHGey0lhI4zunqBSJupaRFfa/n9LlLDYqEDlAd009HI50ZnT4pjLa1lZTTAk2?= =?utf-8?q?nMkwWCW2ETLUgBKkorfWP4fAcu0BFcJWCZDA/qP3jDKBSrrUP6GWpORmQueOy6MbL?= =?utf-8?q?ne0jD1HFvRisIosgqNT96sqYplKFoIad4fQtqr9i7+I8l7WjlUPwnPeFC+OiJMcJl?= =?utf-8?q?kEWdENsCLFDTWKaHKCHpMTLKBK7S3g+XouvVJZoyWY8e4AW0m8eFWrwwMIw07RQI6?= =?utf-8?q?/qetr+2NQnf8iUWVKRRoU/BFdy7JgZWJfptrU72nuEUd9/8tA0vnJkTB5jjFUXhCL?= =?utf-8?q?HRqVYCS1Ym3WFEDNfJYvHiXdSmDCIK4k2W2esIcV4g202qVzNCpEml5UqT+/RiDJk?= =?utf-8?q?3YZ+GzckR6cfAawiO2s8gtz71DExI3aHHjGhVUEAPWmzgaul6ep0goyV3ksXnaP2W?= =?utf-8?q?bqXV1xV/ttyvugh6n8DDQ/19s4IjsDZ7jRiS3JJUNNiGDmFdCE3vOAvZpVjH8ZyV+?= =?utf-8?q?knU1PH2tzRFi2ZrBwBCi9xnicBpyCcvRDiTT88CtJR6C9Z8NPdIxpDyV/3Dishl1o?= =?utf-8?q?Pn0AEbfbZCipUC2NTWqDbDKSybczNVGK63Z+zafkWys5YnSYaP5AuaXA4s6IY05Q/?= =?utf-8?q?Q0GFCarQb4uO49NJtVEtBgaxgflQIWp9LgbqXK8EPWjO4+rC3oB8nlAZK7BjEigk+?= =?utf-8?q?JSkMRRGDNwSVv5ACrXfCXlFpyqEUi159UP6CZro3Qs+F5LEp7j/L7KOkWYndYGAG3?= =?utf-8?q?bCbl+rug4RZ/EM3Qo2IhjBSKuP5/mBMoHGd3jv/lpuv8hJoB6E11FnUfmZ7TahjUX?= =?utf-8?q?mqFe5CZw62IetrS2eAm/gz5uzrEpE/rlYupHiCny/3MkqrtmtKIY+885ezsaDsC0S?= =?utf-8?q?WmiUGhxpzG38u7VoBeb51nGmDiu/NNDahvW1KJ3nODmzD8o7cT5J0/4+2duxhMa8q?= =?utf-8?q?bjw/FHiovO5jrdf5OOAF/LsHEr0RgCKbrBOtmTUcvYhAuSf6wg93TPkZqSCrX8BDb?= =?utf-8?q?tfY8exL4IT0G3dddG9sQfuNE4Sxp6qdudK7VI4F/CCWEgptsd3LSrvNfCrowrCGvS?= =?utf-8?q?sjfdXlGWYqlSZFvr5vBxT156pJJ9PgpZ4fWskLnfK9aL0Gz3xr/LtaxL7/axrPmHA?= =?utf-8?q?JGrkGiGlgj0XA5DqWudOlirIpexYk1Rg=3D=3D?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: a3ebcdac-364f-435a-3742-08da12fbaa48 X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8616.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Mar 2022 09:48:53.3840 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: zI9RqP+WY0dJJvIj1GogcqYMv9v7jimkfSPI8TNIF1/EU7r3de2T2f97UsYvPx8AtHOwcdHlLXBrI/TKFjRrGg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB5658 There's no point in writing 80x25 text mode information via multiple insns all storing immediate values. The data can simply be included first thing in the vga_modes table, allowing the already present REP MOVSB to take care of everything in one go. While touching this also correct a related but stale comment. Signed-off-by: Jan Beulich Reviewed-by: Roger Pau Monné --- a/xen/arch/x86/boot/video.S +++ b/xen/arch/x86/boot/video.S @@ -655,8 +655,9 @@ outidx: outb %al, %dx # Build the table of video modes (stored after the setup.S code at the # `modelist' label. Each video mode record looks like: # .word MODE-ID (our special mode ID (see above)) -# .byte rows (number of rows) -# .byte columns (number of columns) +# .word rows (number of rows) +# .word columns (number of columns) +# .word 0 (color depth; gfx modes only) # Returns address of the end of the table in DI, the end is marked # with a ASK_VGA ID. mode_table: @@ -665,12 +666,6 @@ mode_table: jnz mtab1 leaw modelist, %di # Store standard modes: - movw $VIDEO_80x25,(%di) # The 80x25 mode (ALL) - movw $0x50,2(%di) - movw $0x19,4(%di) - movw $0x00,6(%di) - addw $8,%di - leaw bootsym(vga_modes), %si # All modes for std VGA movw $vga_modes_end-vga_modes, %cx rep movsb @@ -684,6 +679,7 @@ ret0: ret # Modes usable on all standard VGAs vga_modes: + .word VIDEO_80x25, 0x50,0x19,0 # 80x25 .word VIDEO_80x50, 0x50,0x32,0 # 80x50 .word VIDEO_80x43, 0x50,0x2b,0 # 80x43 .word VIDEO_80x28, 0x50,0x1c,0 # 80x28