All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Hannes Reinecke <hare@suse.de>
Cc: stefanha@gmail.com, qemu-devel@nongnu.org, nab@linux-iscsi.org,
	kraxel@redhat.com
Subject: [Qemu-devel] Re: [PATCH] Make SCSI HBA configurable
Date: Mon, 22 Nov 2010 12:04:18 +0100	[thread overview]
Message-ID: <4CEA4E32.1070007@redhat.com> (raw)
In-Reply-To: <20101122101536.41288F90AF@ochil.suse.de>

On 11/22/2010 11:15 AM, Hannes Reinecke wrote:
>
> This patch introduces configuration variables
> CONFIG_SCSI_LSI
> CONFIG_SCSI_MEGASAS
> and renames the existing CONFIG_ESP to CONFIG_SCSI_ESP.
> With this the available SCSI HBAs can be configured for each
> target configuration instead of compiling it in for everyone.
>
> Signed-off-by: Hannes Reinecke<hare@suse.de>
> ---
>   Makefile.objs                        |    5 +++--
>   default-configs/i386-softmmu.mak     |    2 ++
>   default-configs/mips-softmmu.mak     |    2 +-
>   default-configs/mips64-softmmu.mak   |    2 +-
>   default-configs/mips64el-softmmu.mak |    2 +-
>   default-configs/mipsel-softmmu.mak   |    2 +-
>   default-configs/ppc-softmmu.mak      |    2 ++
>   default-configs/ppc64-softmmu.mak    |    2 ++
>   default-configs/ppcemb-softmmu.mak   |    2 ++
>   default-configs/sparc-softmmu.mak    |    2 +-
>   default-configs/sparc64-softmmu.mak  |    2 ++
>   default-configs/x86_64-softmmu.mak   |    2 ++
>   12 files changed, 20 insertions(+), 7 deletions(-)
>
> diff --git a/Makefile.objs b/Makefile.objs
> index 89c8661..3b040cd 100644
> --- a/Makefile.objs
> +++ b/Makefile.objs
> @@ -231,8 +231,9 @@ hw-obj-$(CONFIG_IDE_MACIO) += ide/macio.o
>   hw-obj-$(CONFIG_IDE_VIA) += ide/via.o
>
>   # SCSI layer
> -hw-obj-y += lsi53c895a.o megasas.o
> -hw-obj-$(CONFIG_ESP) += esp.o
> +hw-obj-$(CONFIG_SCSI_LSI) += lsi53c895a.o
> +hw-obj-$(CONFIG_SCSI_MEGASAS) += megasas.o
> +hw-obj-$(CONFIG_SCSI_ESP) += esp.o
>
>   hw-obj-y += dma-helpers.o sysbus.o isa-bus.o
>   hw-obj-y += qdev-addr.o
> diff --git a/default-configs/i386-softmmu.mak b/default-configs/i386-softmmu.mak
> index ed00471..130c157 100644
> --- a/default-configs/i386-softmmu.mak
> +++ b/default-configs/i386-softmmu.mak
> @@ -19,6 +19,8 @@ CONFIG_IDE_QDEV=y
>   CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_PIIX=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_NE2000_ISA=y
>   CONFIG_PIIX_PCI=y
>   CONFIG_SOUND=y
> diff --git a/default-configs/mips-softmmu.mak b/default-configs/mips-softmmu.mak
> index 3d0af83..7e840a4 100644
> --- a/default-configs/mips-softmmu.mak
> +++ b/default-configs/mips-softmmu.mak
> @@ -1,7 +1,7 @@
>   # Default configuration for mips-softmmu
>
>   CONFIG_ISA_MMIO=y
> -CONFIG_ESP=y
> +CONFIG_SCSI_ESP=y
>   CONFIG_VGA_PCI=y
>   CONFIG_VGA_ISA=y
>   CONFIG_VGA_ISA_MM=y
> diff --git a/default-configs/mips64-softmmu.mak b/default-configs/mips64-softmmu.mak
> index 0030de4..3816be3 100644
> --- a/default-configs/mips64-softmmu.mak
> +++ b/default-configs/mips64-softmmu.mak
> @@ -1,7 +1,7 @@
>   # Default configuration for mips64-softmmu
>
>   CONFIG_ISA_MMIO=y
> -CONFIG_ESP=y
> +CONFIG_SCSI_ESP=y
>   CONFIG_VGA_PCI=y
>   CONFIG_VGA_ISA=y
>   CONFIG_VGA_ISA_MM=y
> diff --git a/default-configs/mips64el-softmmu.mak b/default-configs/mips64el-softmmu.mak
> index fa2a3ff..21c59e0 100644
> --- a/default-configs/mips64el-softmmu.mak
> +++ b/default-configs/mips64el-softmmu.mak
> @@ -1,7 +1,7 @@
>   # Default configuration for mips64el-softmmu
>
>   CONFIG_ISA_MMIO=y
> -CONFIG_ESP=y
> +CONFIG_SCSI_ESP=y
>   CONFIG_VGA_PCI=y
>   CONFIG_VGA_ISA=y
>   CONFIG_VGA_ISA_MM=y
> diff --git a/default-configs/mipsel-softmmu.mak b/default-configs/mipsel-softmmu.mak
> index 238b73a..3cf853a 100644
> --- a/default-configs/mipsel-softmmu.mak
> +++ b/default-configs/mipsel-softmmu.mak
> @@ -1,7 +1,7 @@
>   # Default configuration for mipsel-softmmu
>
>   CONFIG_ISA_MMIO=y
> -CONFIG_ESP=y
> +CONFIG_SCSI_ESP=y
>   CONFIG_VGA_PCI=y
>   CONFIG_VGA_ISA=y
>   CONFIG_VGA_ISA_MM=y
> diff --git a/default-configs/ppc-softmmu.mak b/default-configs/ppc-softmmu.mak
> index 940f4bf..0cc5914 100644
> --- a/default-configs/ppc-softmmu.mak
> +++ b/default-configs/ppc-softmmu.mak
> @@ -29,6 +29,8 @@ CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_CMD646=y
>   CONFIG_IDE_MACIO=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_NE2000_ISA=y
>   CONFIG_SOUND=y
>   CONFIG_VIRTIO_PCI=y
> diff --git a/default-configs/ppc64-softmmu.mak b/default-configs/ppc64-softmmu.mak
> index e1bc6b8..ce83254 100644
> --- a/default-configs/ppc64-softmmu.mak
> +++ b/default-configs/ppc64-softmmu.mak
> @@ -29,6 +29,8 @@ CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_CMD646=y
>   CONFIG_IDE_MACIO=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_NE2000_ISA=y
>   CONFIG_SOUND=y
>   CONFIG_VIRTIO_PCI=y
> diff --git a/default-configs/ppcemb-softmmu.mak b/default-configs/ppcemb-softmmu.mak
> index 8f1cc09..44111c3 100644
> --- a/default-configs/ppcemb-softmmu.mak
> +++ b/default-configs/ppcemb-softmmu.mak
> @@ -29,6 +29,8 @@ CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_CMD646=y
>   CONFIG_IDE_MACIO=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_NE2000_ISA=y
>   CONFIG_SOUND=y
>   CONFIG_VIRTIO_PCI=y
> diff --git a/default-configs/sparc-softmmu.mak b/default-configs/sparc-softmmu.mak
> index becf880..872929d 100644
> --- a/default-configs/sparc-softmmu.mak
> +++ b/default-configs/sparc-softmmu.mak
> @@ -1,7 +1,7 @@
>   # Default configuration for sparc-softmmu
>
>   CONFIG_ECC=y
> -CONFIG_ESP=y
> +CONFIG_SCSI_ESP=y
>   CONFIG_ESCC=y
>   CONFIG_M48T59=y
>   CONFIG_PTIMER=y
> diff --git a/default-configs/sparc64-softmmu.mak b/default-configs/sparc64-softmmu.mak
> index 1cc3f13..b4a3c27 100644
> --- a/default-configs/sparc64-softmmu.mak
> +++ b/default-configs/sparc64-softmmu.mak
> @@ -13,4 +13,6 @@ CONFIG_IDE_QDEV=y
>   CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_CMD646=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_VIRTIO_PCI=y
> diff --git a/default-configs/x86_64-softmmu.mak b/default-configs/x86_64-softmmu.mak
> index 5183203..f28b66d 100644
> --- a/default-configs/x86_64-softmmu.mak
> +++ b/default-configs/x86_64-softmmu.mak
> @@ -19,6 +19,8 @@ CONFIG_IDE_QDEV=y
>   CONFIG_IDE_PCI=y
>   CONFIG_IDE_ISA=y
>   CONFIG_IDE_PIIX=y
> +CONFIG_SCSI_LSI=y
> +CONFIG_SCSI_MEGASAS=y
>   CONFIG_NE2000_ISA=y
>   CONFIG_PIIX_PCI=y
>   CONFIG_SOUND=y

Acked-By: Paolo Bonzini <pbonzini@redhat.com>

Paolo

  reply	other threads:[~2010-11-22 11:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-22 10:15 [Qemu-devel] [PATCH] Make SCSI HBA configurable Hannes Reinecke
2010-11-22 11:04 ` Paolo Bonzini [this message]
2010-11-22 15:05 ` Jes Sorensen
2010-11-22 22:01   ` Alexander Graf
2010-11-25 10:59 ` Paul Brook
2010-11-25 12:54   ` Alexander Graf
2010-11-25 13:22     ` Paul Brook
2010-11-25 13:24       ` Alexander Graf
2010-11-26  8:01       ` Jes Sorensen

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=4CEA4E32.1070007@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=hare@suse.de \
    --cc=kraxel@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.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.