From mboxrd@z Thu Jan  1 00:00:00 1970
From: Cornelia Huck <cornelia.huck@de.ibm.com>
Subject: Re: [PATCH 5/5] KVM: s390: Split out early console code.
Date: Tue, 30 Oct 2012 16:35:16 +0100
Message-ID: <20121030163516.74b5b4d0@BR9GNB5Z>
References: <1351516046-30037-1-git-send-email-cornelia.huck@de.ibm.com>
	<1351516046-30037-6-git-send-email-cornelia.huck@de.ibm.com>
	<1440F0DC-0DC5-40FB-88E2-EE203A7C5F66@suse.de>
	<20121030135926.2db07255@BR9GNB5Z>
	<508FD96E.1010209@suse.de>
	<20121030152949.5030f0dd@BR9GNB5Z>
	<4363A241-FF6B-4867-BB28-34C8277D6A9D@suse.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Cc: Avi Kivity <avi@redhat.com>, Marcelo Tosatti <mtosatti@redhat.com>,
	KVM <kvm@vger.kernel.org>,
	linux-s390 <linux-s390@vger.kernel.org>,
	Christian Borntraeger <borntraeger@de.ibm.com>,
	Carsten Otte <cotte@de.ibm.com>,
	Heiko Carstens <heiko.carstens@de.ibm.com>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>,
	Sebastian Ott <sebott@linux.vnet.ibm.com>
To: Alexander Graf <agraf@suse.de>
Return-path: <kvm-owner@vger.kernel.org>
Received: from e06smtp16.uk.ibm.com ([195.75.94.112]:55567 "EHLO
	e06smtp16.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1758983Ab2J3PfY (ORCPT <rfc822;kvm@vger.kernel.org>);
	Tue, 30 Oct 2012 11:35:24 -0400
Received: from /spool/local
	by e06smtp16.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted
	for <kvm@vger.kernel.org> from <cornelia.huck@de.ibm.com>;
	Tue, 30 Oct 2012 15:35:22 -0000
In-Reply-To: <4363A241-FF6B-4867-BB28-34C8277D6A9D@suse.de>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Tue, 30 Oct 2012 16:12:47 +0100
Alexander Graf <agraf@suse.de> wrote:

> 
> 
> On 30.10.2012, at 15:29, Cornelia Huck <cornelia.huck@de.ibm.com> wrote:
> 
> > On Tue, 30 Oct 2012 14:43:10 +0100
> > Alexander Graf <agraf@suse.de> wrote:
> > 
> >> On 10/30/2012 01:59 PM, Cornelia Huck wrote:
> >>> On Mon, 29 Oct 2012 19:14:19 +0100
> >>> Alexander Graf<agraf@suse.de>  wrote:
> >>> 
> >>>> On 29.10.2012, at 14:07, Cornelia Huck wrote:
> >>>> 
> >>>>> This code is transport agnostic and can be used by both the legacy
> >>>>> virtio code and virtio_ccw.
> >>>> Would it be possible to actually send real virtio or sclp console commands for early printk? That'd make things a lot easier on the user space end. Combining two completely separate character channels (early printk + sclp or early printk + virtio-console) is really tricky.
> >>>> 
> >>> This code is only used if we use a virtio console device. The sclp
> >>> code path is completely different.
> >> 
> >> Ah, so the sclp early console works differently?
> > 
> > sclp consoles are registered via console_initcall(). I'm not aware of
> > 'early' handling there (although I don't really know the code well.)
> > 
> >> 
> >>> What do you mean with "real virtio console commands"? The message is
> >>> just put on the output queue anyway.
> >> 
> >> For virtio-console, yes. For the early printk it is sent to the 
> >> hypervisor using a hypercall, not through the virtio queue. So it 
> >> arrives at a completely different end point that usually has no 
> >> knowledge of the virtio-console output driver.
> > 
> > virtio setup is a core_initcall, which is done way later than the
> > console_initcalls (including s390_virtio_console_init). According to
> > the comments, virtio_cons_early_init is there exactly for that reason.
> 
> It's never worked on upstream QEMU. 

It did on kuli ;)

> Can we just drop kvm specific early printk support then?

I'd prefer to leave this cleanup in the series for now. Whether we want
to do something in qemu or remove this is something we can discuss
later. In the meantime, there's sclp :)