All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick Ohly <patrick.ohly@intel.com>
To: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] runqemu: avoid overridden user input for bootparams
Date: Tue, 28 Feb 2017 11:52:02 +0100	[thread overview]
Message-ID: <1488279122.7785.34.camel@intel.com> (raw)
In-Reply-To: <1487690288-18275-1-git-send-email-dmitry.rozhkov@linux.intel.com>

On Tue, 2017-02-21 at 17:18 +0200, Dmitry Rozhkov wrote:
> Currently runqemu hardcodes the "ip=" kernel boot parameter
> when configuring QEMU to use tap or slirp networking. This makes
> the guest system to have a network interface pre-configured
> by kernel and causes systemd to fail renaming the interface
> to whatever pleases it:
> 
>   Feb 21 10:10:20 intel-corei7-64 systemd-udevd[201]: Error changing
>       net interface name 'eth0' to 'enp0s3': Device or resource busy,
> 
> Always append user input for kernel boot params after the ones
> added by the script. This way user input has priority over runqemu's
> default params.

Presumably the user's bootparams then would include "ip="?

Either way, the change looks good to me. I did something similar for the
qemu parameters in "runqemu: let command line parameters override
defaults".

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.





  reply	other threads:[~2017-02-28 10:52 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-21 15:18 [PATCH] runqemu: avoid overridden user input for bootparams Dmitry Rozhkov
2017-02-28 10:52 ` Patrick Ohly [this message]
2017-02-28 11:44   ` Dmitry Rozhkov

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=1488279122.7785.34.camel@intel.com \
    --to=patrick.ohly@intel.com \
    --cc=dmitry.rozhkov@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.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.