[Buildroot] [PATCH/next] qemu: add an option to enable user mode networking (SLIRP)

Thomas Petazzoni thomas.petazzoni at bootlin.com
Wed Aug 29 20:04:38 UTC 2018


Hello,

On Mon, 27 Aug 2018 17:15:01 -0300, Carlos Santos wrote:
> User mode networking is a useful feature that does not depend on other
> packages. Add an option to enable it but keep it disabled by default,
> for backwards compatibility.
> 
> Signed-off-by: Carlos Santos <casantos at datacom.com.br>

I've applied to next, with some minor tweaks, see below.


> +config BR2_PACKAGE_QEMU_SLIRP
> +	bool "Keep user mode networking (SLIRP)"

I understand you want to use "Keep" because SLIRP is normally enabled
by default in Qemu, and enabling this option allows to "keep" it.

However, I don't think such details really need to surface up to the
Config.in option prompts, so I've changed "Keep" to "Enable".

Indeed, if the option is selected, SLIRP support is enabled, if the
option is not selected SLIRP support is disabled.

> +	help
> +	  Keep user mode network stack, which is the default networking

I've changed Keep to Enable here as well.


> +# There is no "--enable-slirp"
> +ifneq ($(BR2_PACKAGE_QEMU_SLIRP),y)

and used positive logic here:

ifeq ($(BR2_PACKAGE_QEMU_SLIRP),)

Applied to next with those changes. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list