[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 3/4] xen/ppc: Implement early serial printk on pseries


  • To: Shawn Anastasio <sanastasio@xxxxxxxxxxxxxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 24 Jul 2023 13:35:49 +0200
  • 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
  • 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=3LrX9uyBmCeBB48YcEAKjAYONyqob2RuDkUInzMw1MU=; b=Xvs4W44Eb008/nx1gdqCYBr1meX4GGSlY5pBfE9GZmlCMElH2RMLQNHJKmmO1DmBtXvQ7eynEmubudNc3PpSjMvMD5yyDN+1BRCKGwnDv/Yc/WaSfdh6WiG/5qiXoYxPQ7Y9YQaxkEa2cpCl3/yndNvbQ2lIxAoheRKqqPzs/6j/dyUsoUQbE7TftUBeTMt2m0GxQGJ//1hOI/iUy3LO5YJVS9mky1S2gXHrGZ0BieIcIYTRoJea5VhdMkF1/ulgUy+yWAVZPc8Cv7z+V3OmE1S4YpRgV7R7Tl9JOds54QAilrE8DeyLet+5/rZUanxieI94UB54RTCvwB3zL2vrsQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hS+qhRa1z7Wuaz4sbnhHQjnJeJEFnxr4SZLJC1sUP/mFwsAC8hx231j8z8M1fLizvxf56LVrP7u8Lmur8Or1iKfsgOEWGzoXH43hSbVkyvTbIC160cy/UL+vV7S0xzEoYmOeu63Em2LrsTd0V/H1DOKw0pK819bVTBAB6c1YY+dST0IVmswIVXqSyfYTjaDf8rfggzVFfwS7LABNEXF50hBRFD62Dh6R9OxY5E74XjylBzC7gfDZf2CwxG59pfqYbArGYe7OFn+cFg/+jXjuziFHgcj2UwKL2zUF9EY7reYHhoM95ZXGr6RXWMcPQ1BQraAkwSyHpgJ2bfvh9BkJYg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Timothy Pearson <tpearson@xxxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Mon, 24 Jul 2023 11:35:54 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 21.07.2023 19:02, Shawn Anastasio wrote:
> On typical Power VMs (e.g. QEMU's -M pseries), a variety of services
> including an early serial console are provided by Open Firmware.
> Implement the required interfaces to call into Open Firmware and write
> to the serial console.
> 
> Since Open Firmware runs in 32-bit Big Endian mode and Xen runs in
> 64-bit Little Endian mode, a thunk is required to save/restore
> any potentially-clobbered registers as well as to perform the
> required endianness switch. Thankfully, linux already has such
> a routine, which was imported into ppc64/of-call.S.
> 
> Support for bare metal (PowerNV) will be implemented in a future
> patch.
> 
> Signed-off-by: Shawn Anastasio <sanastasio@xxxxxxxxxxxxxxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
with the same remark as for patch 2, plus ...

> --- a/xen/arch/ppc/include/asm/asm-defns.h
> +++ b/xen/arch/ppc/include/asm/asm-defns.h
> @@ -37,4 +37,21 @@
>      .long 0xa6037b7d; /* mtsrr1 r11                         */               
>   \
>      .long 0x2400004c  /* rfid                               */
>  
> +
> +/* Taken from Linux kernel source (arch/powerpc/boot/crt0.S) */

... the introduction of a double blank line here dropped (which I'll
take care of while committing).

Jan



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.