• Using the new host_net_redir command you can easily create redirections
    on the fly while your VM is running.
    
    While that's great, it's missing the removal of redirections, in case you
    want to have a port closed again at a later point in time.
    
    This patch adds support for removal of redirections.
    
    Signed-off-by: Alexander Graf <agraf@suse.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Alexander Graf authored
     
    Browse Code »
  • All,
         I've recently been playing around with migration via exec.  Unfortunately,
    when starting the incoming qemu process with "-incoming exec:cmd", it suffers
    the same problem that -incoming tcp used to suffer; namely, that you can't
    interact with the monitor until after the migration has happened.  This causes
    problems for libvirt usage of -incoming exec, since libvirt expects to be able
    to access the monitor ahead of time.  This fairly simple patch allows you to
    access the monitor both before and after the migration has completed using exec.
    
    (note: developed/tested with qemu-kvm, but applies perfectly fine to qemu)
    
    Signed-off-by: Chris Lalancette <clalance@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Chris Lalancette authored
     
    Browse Code »
  • Now that we have a separate aio pool structure we can remove those
    aio pool details from BlockDriver.
    
    Every driver supporting AIO now needs to declare a static AIOPool
    with the aiocb size and the cancellation method.  This cleans up the
    current code considerably and will make it cleaner and more obvious
    to support two different aio implementations behind a single
    BlockDriver.
    
    Signed-off-by: Christoph Hellwig <hch@lst.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Christoph Hellwig authored
     
    Browse Code »
  • [this one is required for [PATCH] fully split aio_pool from BlockDriver,
     sorry for not sending it out earlier]
    
    Add a qcow_aio_setup helper to qcow to shared common code between
    the aio_readv and aio_writev methods.  Based on the function with
    the same name in qcow2.
    
    Signed-off-by: Christoph Hellwig <hch@lst.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Christoph Hellwig authored
     
    Browse Code »
  • We do need hdev_create unconditionally on all platforms so that qemu-img
    create support for host device works on all platforms.
    
    Also relax the check to allow character devices in addition to block
    devices.  On many Unix platforms block devices have buffered block
    nodes and unbuffered character device nodes, and on FreeBSD the block
    nodes don't even exist anymore.  Also on Linux we do support the
    /dev/sgN scsi passthrough devices through the host device driver,
    and probably the old-style /dev/raw/rawN raw devices although I haven't
    tested that.
    
    Signed-off-by: Christoph Hellwig <hch@lst.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Christoph Hellwig authored
     
    Browse Code »
  • raw_pread_aligned currently returns the raw return value from
    lseek/read, which is always -1 in case of an error.  But the
    callers higher up the stack expect it to return the negated
    errno just like raw_pwrite_aligned.
    
    Signed-off-by: Christoph Hellwig <hch@lst.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Christoph Hellwig authored
     
    Browse Code »
  • Pointer vs addresses a VncDisplay structure,
    so it is sufficient to allocate sizeof(VncDisplay)
    or sizeof(*vs) bytes instead of the much larger
    sizeof(VncState).
    
    Maybe the misleading name should be fixed, too:
    the code contains many places where vs is used,
    sometimes it is a VncState *, sometimes it is a
    VncDisplay *. vd would be a better name.
    
    Signed-off-by: Stefan Weil <weil@mail.berlios.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Stefan Weil authored
     
    Browse Code »
  • This patch converts the remaining users of bdrv_create2 to bdrv_create and
    removes the now unused function.
    
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • Don't write each single changed refcount block entry to the disk after it is
    written, but update all entries of the block and write all of them at once.
    
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • This is a preparation patch with no functional changes. It moves the allocation
    of new refcounts block to a new function and makes update_cluster_refcount (for
    one cluster) call update_refcount (for multiple clusters) instead the other way
    round.
    
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • There is only one (internal) user left and it can be switched to the normal
    emulation provided in block.c
    
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • When a reset is requested, the current e1000 emulation never clears the
    reset bit which may cause a driver to hang. This patch masks the reset
    bit out when setting the control registert, so the reset is immediately
    completed.
    
    Signed-off-by: Kevin Wolf <mail@kevin-wolf.de>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • Commit ffad4116 removed the "scratch buffer"
    from check_params, but didn't care for the error messages which actually
    included this string to tell the user which option was wrong. Now this string
    is uninitialized, so this patch removes it from the message.
    
    This means that the user is only told the whole parameter string and has to
    pick the wrong option by himself as the callers of check_params can't know this
    value any more. An alternative approach would be to revert that commit and do
    whatever is needed to fix the original problem without changing check_params.
    
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
    Kevin Wolf authored
     
    Browse Code »
  • If libfdt is not available, disable the fdt manipulation features.
    
    Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
    Edgar E. Iglesias authored
     
    Browse Code »
  • Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
    Edgar E. Iglesias authored
     
    Browse Code »
  • Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
    Edgar E. Iglesias authored
     
    Browse Code »
  • Signed-off-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>
    Edgar E. Iglesias authored
     
    Browse Code »