All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Graf <agraf@suse.de>
To: Cornelia Huck <cornelia.huck@de.ibm.com>
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>
Subject: Re: [PATCH 5/5] KVM: s390: Split out early console code.
Date: Tue, 30 Oct 2012 14:43:10 +0100	[thread overview]
Message-ID: <508FD96E.1010209@suse.de> (raw)
In-Reply-To: <20121030135926.2db07255@BR9GNB5Z>

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?

> 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.


Alex

  reply	other threads:[~2012-10-30 13:43 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-29 13:07 [PATCH 0/5] s390: Guest support for virtio-ccw Cornelia Huck
2012-10-29 13:07 ` [PATCH 1/5] KVM: s390: Handle hosts not supporting s390-virtio Cornelia Huck
2012-10-29 13:07 ` [PATCH 2/5] s390: Move css limits from drivers/s390/cio/ to include/asm/ Cornelia Huck
2012-10-29 13:07 ` [PATCH 3/5] s390: Add a mechanism to get the subchannel id Cornelia Huck
2012-10-29 13:07 ` [PATCH 4/5] KVM: s390: Add a channel I/O based virtio transport driver Cornelia Huck
2012-10-29 18:12   ` Alexander Graf
2012-10-29 18:34     ` Cornelia Huck
2012-10-29 18:37       ` Alexander Graf
2012-10-30 13:03         ` Cornelia Huck
2012-10-30 13:41           ` Alexander Graf
2012-10-30 14:00             ` Cornelia Huck
2012-10-30 14:05               ` Alexander Graf
2012-10-30 14:35                 ` Cornelia Huck
2012-10-30 15:11                   ` Alexander Graf
2012-10-29 13:07 ` [PATCH 5/5] KVM: s390: Split out early console code Cornelia Huck
2012-10-29 18:14   ` Alexander Graf
2012-10-30 12:59     ` Cornelia Huck
2012-10-30 13:43       ` Alexander Graf [this message]
2012-10-30 14:29         ` Cornelia Huck
2012-10-30 15:12           ` Alexander Graf
2012-10-30 15:35             ` Cornelia Huck
2012-10-30 15:42               ` Alexander Graf
2012-10-29 17:55 ` [PATCH 0/5] s390: Guest support for virtio-ccw Alexander Graf
2012-10-29 18:15   ` Cornelia Huck
2012-10-29 18:33     ` Alexander Graf
2012-10-30 13:04       ` Cornelia Huck
2012-10-30 13:43         ` Alexander Graf
  -- strict thread matches above, loose matches on Subject: below --
2012-12-07 12:29 [PATCH v3 " Cornelia Huck
2012-12-07 12:29 ` [PATCH 5/5] KVM: s390: Split out early console code Cornelia Huck
2012-12-09 12:45   ` Alexander Graf
2012-12-10  9:06     ` Cornelia Huck
2012-12-11 10:10       ` Alexander Graf
2012-12-11 12:36         ` Cornelia Huck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=508FD96E.1010209@suse.de \
    --to=agraf@suse.de \
    --cc=avi@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=cornelia.huck@de.ibm.com \
    --cc=cotte@de.ibm.com \
    --cc=heiko.carstens@de.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    --cc=schwidefsky@de.ibm.com \
    --cc=sebott@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.