All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lucas Meneghel Rodrigues <lmr@redhat.com>
To: Jason Wang <jasowang@redhat.com>
Cc: autotest@test.kernel.org, KVM@test.kernel.org,
	mailing list <kvm@vger.kernel.org>
Subject: Re: [PATCH 9/9] KVM test: Redirect the console to serial for all linux guests
Date: Thu, 06 May 2010 12:43:50 -0300	[thread overview]
Message-ID: <1273160630.15652.27.camel@freedom> (raw)
In-Reply-To: <4BE232C5.9010808@redhat.com>

On Thu, 2010-05-06 at 11:08 +0800, Jason Wang wrote:
> Michael Goldish wrote:
> > On 04/26/2010 01:04 PM, Jason Wang wrote:
> >   
> >> As we have the ability to dump the content from serial console or use
> >> a session through it, we need to redirect the console to serial
> >> through unattended files to make use of it.
> >>
> >> Signed-off-by: Jason Wang <jasowang@redhat.com>
> >> ---
> >>  client/tests/kvm/unattended/Fedora-10.ks     |    2 +-
> >>  client/tests/kvm/unattended/Fedora-11.ks     |    2 +-
> >>  client/tests/kvm/unattended/Fedora-12.ks     |    2 +-
> >>  client/tests/kvm/unattended/Fedora-8.ks      |    2 +-
> >>  client/tests/kvm/unattended/Fedora-9.ks      |    2 +-
> >>  client/tests/kvm/unattended/OpenSUSE-11.xml  |    1 +
> >>  client/tests/kvm/unattended/RHEL-3-series.ks |    2 +-
> >>  client/tests/kvm/unattended/RHEL-4-series.ks |    2 +-
> >>  client/tests/kvm/unattended/RHEL-5-series.ks |    2 +-
> >>  client/tests/kvm/unattended/SLES-11.xml      |    1 +
> >>  10 files changed, 10 insertions(+), 8 deletions(-)
> >>
> >> diff --git a/client/tests/kvm/unattended/Fedora-10.ks b/client/tests/kvm/unattended/Fedora-10.ks
> >> index 61e59d7..8628036 100644
> >> --- a/client/tests/kvm/unattended/Fedora-10.ks
> >> +++ b/client/tests/kvm/unattended/Fedora-10.ks
> >> @@ -11,7 +11,7 @@ firewall --enabled --ssh
> >>  selinux --enforcing
> >>  timezone --utc America/New_York
> >>  firstboot --disable
> >> -bootloader --location=mbr
> >> +bootloader --location=mbr --append="console=ttyS0,115200"
> >>     
> >
> > I see no reason to completely silence the regular console.  We can let
> > some output go to the regular console as well as the serial one by using
> >
> > --append="console=tty0 console=ttyS0"
> >
> >   
> Yes, we should always keep the tty0. Thanks for pointing that.

It seems like we are opening possibilities on windows land - it seems
that there is an administration console running on windows' serial ports
by default:

http://technet.microsoft.com/en-us/library/cc785873%28WS.10%29.aspx

Need to check that out.

Excellent. Jason, the patchset looks mostly good, please implement the
suggestions pointed out on the comments and re-send it.

      reply	other threads:[~2010-05-06 15:43 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-26 10:03 [PATCH 0/9] Make use of the redirection of guest serial Jason Wang
2010-04-26 10:03 ` [PATCH 1/9] KVM test: Introduce the prompt assist Jason Wang
2010-04-28 11:28   ` Michael Goldish
2010-05-06  2:55     ` Jason Wang
2010-05-06 15:18       ` Lucas Meneghel Rodrigues
2010-04-26 10:03 ` [PATCH 2/9] KVM test: Add the ability to send the username in remote_login() Jason Wang
2010-04-28 11:32   ` Michael Goldish
2010-04-26 10:03 ` [PATCH 3/9] KVM test: Make the login re suitable for serial console Jason Wang
2010-04-28 11:18   ` Michael Goldish
2010-05-06  2:57     ` Jason Wang
2010-04-26 10:03 ` [PATCH 4/9] KVM test: Redirect the serial to the unix domain socket Jason Wang
2010-04-26 10:04 ` [PATCH 5/9] KVM test: Log the content from guest serial console Jason Wang
2010-04-28 13:26   ` Michael Goldish
2010-05-06  3:03     ` Jason Wang
2010-04-26 10:04 ` [PATCH 6/9] KVM test: Raise error when met unknown type in kvm_vm.remote_login() Jason Wang
2010-05-06 15:15   ` [Autotest] " Lucas Meneghel Rodrigues
2010-04-26 10:04 ` [PATCH 7/9] KVM test: Introduce the local_login() Jason Wang
2010-04-28 12:01   ` Michael Goldish
2010-04-28 23:44     ` Amos Kong
     [not found]     ` <20100428234409.GA2738@akong@redhat.com>
2010-05-05  9:37       ` [Autotest] " Michael Goldish
2010-05-06  3:07     ` Jason Wang
2010-04-26 10:04 ` [PATCH 8/9] KVM test: Create the background threads before calling process() Jason Wang
2010-04-28 11:55   ` Michael Goldish
2010-05-06 15:35     ` [Autotest] " Lucas Meneghel Rodrigues
2010-04-26 10:04 ` [PATCH 9/9] KVM test: Redirect the console to serial for all linux guests Jason Wang
2010-04-28 12:24   ` Michael Goldish
2010-05-06  3:08     ` Jason Wang
2010-05-06 15:43       ` Lucas Meneghel Rodrigues [this message]

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=1273160630.15652.27.camel@freedom \
    --to=lmr@redhat.com \
    --cc=KVM@test.kernel.org \
    --cc=autotest@test.kernel.org \
    --cc=jasowang@redhat.com \
    --cc=kvm@vger.kernel.org \
    /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.