On Wed, 2012-08-01 at 09:15 +0200, Jan Kiszka wrote: > On 2012-08-01 07:18, Alex Williamson wrote: > > Signed-off-by: Alex Williamson <alex.williamson@xxxxxxxxxx> > > --- > > > > MAINTAINERS | 5 +++++ > > configure | 12 ++++++++++++ > > hw/i386/Makefile.objs | 1 + > > 3 files changed, 18 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 2d219d2..9680d69 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -460,6 +460,11 @@ M: Gerd Hoffmann <kraxel@xxxxxxxxxx> > > S: Maintained > > F: hw/usb* > > > > +VFIO > > +M: Alex Williamson <alex.williamson@xxxxxxxxxx> > > +S: Supported > > +F: hw/vfio* > > + > > vhost > > M: Michael S. Tsirkin <mst@xxxxxxxxxx> > > S: Supported > > diff --git a/configure b/configure > > index c65b5f6..81108dc 100755 > > --- a/configure > > +++ b/configure > > @@ -143,6 +143,7 @@ attr="" > > libattr="" > > xfs="" > > > > +vfio_pci="no" > > vhost_net="no" > > kvm="no" > > gprof="no" > > @@ -489,6 +490,7 @@ Haiku) > > usb="linux" > > kvm="yes" > > vhost_net="yes" > > + vfio_pci="yes" > > if [ "$cpu" = "i386" -o "$cpu" = "x86_64" ] ; then > > audio_possible_drivers="$audio_possible_drivers fmod" > > fi > > @@ -824,6 +826,10 @@ for opt do > > ;; > > --disable-guest-agent) guest_agent="no" > > ;; > > + --disable-vfio-pci) vfio_pci="no" > > + ;; > > + --enable-vfio-pci) vfio_pci="yes" > > + ;; > > Do we need this level of control? Open question I'm just wondering every > time a new feature gets added together with --disable/--enable switches. Well, I could certainly understand if some downstream wanted to ship a qemu that didn't enable device assignment. I'm sure they'd rather have a config option to do that instead of needing to modify code. I generally find --enable useful to force an error and tell me what I'm missing when I specifically want a feature rather than having it silently disabled. > > *) echo "ERROR: unknown option $opt"; show_help="yes" > > ;; > > esac > > @@ -1112,6 +1118,8 @@ echo " --disable-guest-agent disable building of the QEMU Guest Agent" > > echo " --enable-guest-agent enable building of the QEMU Guest Agent" > > echo " --with-coroutine=BACKEND coroutine backend. Supported options:" > > echo " gthread, ucontext, sigaltstack, windows" > > +echo " --disable-vfio-pci disable vfio pci device assignement support" > > +echo " --enable-vfio-pci enable vfio pci device assignment support" > > echo "" > > echo "NOTE: The object files are built at the place where configure is launched" > > exit 1 > > @@ -3072,6 +3080,7 @@ echo "OpenGL support $opengl" > > echo "libiscsi support $libiscsi" > > echo "build guest agent $guest_agent" > > echo "coroutine backend $coroutine_backend" > > +echo "VFIO PCI support $vfio_pci" > > > > if test "$sdl_too_old" = "yes"; then > > echo "-> Your SDL version is too old - please upgrade to have SDL support" > > @@ -3754,6 +3763,9 @@ case "$target_arch2" in > > *) > > echo "CONFIG_NO_XEN=y" >> $config_target_mak > > esac > > +if test "$vfio_pci" = "yes" -a "$target_softmmu" = "yes" ; then > > + echo "CONFIG_VFIO_PCI=y" >> $config_target_mak > > +fi > > Does this already somehow depend on host == Linux? If not, you may break > the others. Hmm, probably missing that, I'll look where to add it. Thanks, Alex -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html