qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 8bafcb: memory: add early bail out from cpu_p


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 8bafcb: memory: add early bail out from cpu_physical_memor...
Date: Wed, 10 Feb 2016 06:00:07 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 8bafcb21643a39a5b29109f8bd5ee5a6f0f6850b
      
https://github.com/qemu/qemu/commit/8bafcb21643a39a5b29109f8bd5ee5a6f0f6850b
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M include/exec/ram_addr.h

  Log Message:
  -----------
  memory: add early bail out from cpu_physical_memory_set_dirty_range

This condition is true in the common case, so we can cut out the body of
the function.  In addition, this makes it easier for the compiler to do
at least partial inlining, even if it decides that fully inlining the
function is unreasonable.

Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 5b82b703b69acc67b78b98a5efc897a3912719eb
      
https://github.com/qemu/qemu/commit/5b82b703b69acc67b78b98a5efc897a3912719eb
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M exec.c
    M include/exec/ram_addr.h
    M migration/ram.c

  Log Message:
  -----------
  memory: RCU ram_list.dirty_memory[] for safe RAM hotplug

Although accesses to ram_list.dirty_memory[] use atomics so multiple
threads can safely dirty the bitmap, the data structure is not fully
thread-safe yet.

This patch handles the RAM hotplug case where ram_list.dirty_memory[] is
grown.  ram_list.dirty_memory[] is change from a regular bitmap to an
RCU array of pointers to fixed-size bitmap blocks.  Threads can continue
accessing bitmap blocks while the array is being extended.  See the
comments in the code for an in-depth explanation of struct
DirtyMemoryBlocks.

I have tested that live migration with virtio-blk dataplane works.

Signed-off-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 34689e206abddac87a5217d458534e24f2a05562
      
https://github.com/qemu/qemu/commit/34689e206abddac87a5217d458534e24f2a05562
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  qemu-char: Keep pty slave file descriptor open until the master is closed

If a process opens the slave pts device, writes data to it, then
immediately closes it, the data doesn't reliably get delivered to the
emulated serial port. This seems to be because a read of the master
pty device returns EIO on Linux if no process has the pts device open,
even when data is waiting "in the pipe".

A fix seems to be for QEMU to keep the pts file descriptor open until
the pty is closed, as per the below patch.

Signed-off-by: Ashley Jonathan <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Michael Tokarev <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 977a82ab56daac83623d730174f47d5a7edd73c9
      
https://github.com/qemu/qemu/commit/977a82ab56daac83623d730174f47d5a7edd73c9
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: sanity check the glib library that pkg-config finds

Developers on 64-bit machines will often try to perform a
32-bit build of QEMU by running

  ./configure --extra-cflags="-m32"

Unfortunately if PKG_CONFIG_LIBDIR is not set to point to
the location of the 32-bit pkg-config files, then configure
will silently pick up the 64-bit pkg-config files and still
succeed.

This causes a problem for glib because it means QEMU will
be pulling in /usr/lib64/glib-2.0/include/glibconfig.h
instead of /usr/lib/glib-2.0/include/glibconfig.h

This causes problems because the 'gsize' type (defined as
'unsigned long') will no longer be fully compatible with
the 'size_t' type (defined as 'unsigned int'). Although
both are the same size, the compiler refuses to allow
casts from 'unsigned long *' to 'unsigned int *' as they
are different pointer types. This results in non-obvious
compiler errors when building QEMU eg

qga/commands-posix.c: In function ‘qmp_guest_set_user_password’:
qga/commands-posix.c:1912:55: error: passing argument 2 of 
‘g_base64_decode’ from incompatible pointer type 
[-Werror=incompatible-pointer-types]
     rawpasswddata = (char *)g_base64_decode(password, &rawpasswdlen);
                                                      ^
In file included from /usr/include/glib-2.0/glib.h:35:0,
           from qga/commands-posix.c:14:
/usr/include/glib-2.0/glib/gbase64.h:52:9: note: expected ‘gsize * {aka long 
unsigned int *}’ but argument is of type ‘size_t * {aka unsigned int *}’
 guchar *g_base64_decode         (const gchar  *text,
   ^
cc1: all warnings being treated as errors

To detect this problem, add a check to configure that
verifies that GLIB_SIZEOF_SIZE_T matches sizeof(size_t).
If this fails print a warning suggesting that the dev
probably needs to set PKG_CONFIG_LIBDIR.

On Fedora x86_64 it passes with any of:

 # ./configure
 # PKG_CONFIG_LIBDIR=/usr/lib/pkgconfig ./configure --extra-cflags="-m32"
 # PKG_CONFIG_LIBDIR=/usr/lib64/pkgconfig ./configure --extra-cflags="-m64"

And fails with a mis-match

 # PKG_CONFIG_LIBDIR=/usr/lib64/pkgconfig ./configure --extra-cflags="-m32"
 # PKG_CONFIG_LIBDIR=/usr/lib/pkgconfig ./configure --extra-cflags="-m64"

ERROR: sizeof(size_t) doesn't match GLIB_SIZEOF_SIZE_T.
       You probably need to set PKG_CONFIG_LIBDIR
       to point to the right pkg-config files for your
       build target

Signed-off-by: Daniel P. Berrange <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: a0aa44b488b3601415d55041e4619aef5f3a4ba8
      
https://github.com/qemu/qemu/commit/a0aa44b488b3601415d55041e4619aef5f3a4ba8
  Author: Alex Bennée <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M include/qemu/atomic.h

  Log Message:
  -----------
  include/qemu/atomic.h: default to __atomic functions

The __atomic primitives have been available since GCC 4.7 and provide
a richer interface for describing memory ordering requirements. As a
bonus by using the primitives instead of hand-rolled functions we can
use tools such as the ThreadSanitizer which need the use of well
defined APIs for its analysis.

If we have __ATOMIC defines we exclusively use the __atomic primitives
for all our atomic access. Otherwise we fall back to the mixture of
__sync and hand-rolled barrier cases.

Signed-off-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>
[Use __ATOMIC_SEQ_CST for atomic_mb_read/atomic_mb_set on !POWER. - Paolo]
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 2ecab4084fa8b418f62d28bd310936d38ef6879b
      
https://github.com/qemu/qemu/commit/2ecab4084fa8b418f62d28bd310936d38ef6879b
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M hw/scsi/scsi-disk.c
    M include/hw/scsi/scsi.h

  Log Message:
  -----------
  scsi: push WWN fields up to SCSIDevice

SAS adapters need to access them in order to publish the SAS addresses
of the end devices connected to them.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 9fd7e85938e87ca52dcb1aeff34fa1675917831b
      
https://github.com/qemu/qemu/commit/9fd7e85938e87ca52dcb1aeff34fa1675917831b
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M hw/scsi/scsi-disk.c
    M hw/scsi/scsi-generic.c
    M include/hw/scsi/scsi.h

  Log Message:
  -----------
  scsi-generic: grab device and port SAS addresses from backend

This lets a SAS adapter expose them through its own configuration
mechanism.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e351b82611293683c4cabe4b69b7552bde5d4e2a
      
https://github.com/qemu/qemu/commit/e351b82611293683c4cabe4b69b7552bde5d4e2a
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M default-configs/pci.mak
    M hw/scsi/Makefile.objs
    A hw/scsi/mpi.h
    A hw/scsi/mptconfig.c
    A hw/scsi/mptendian.c
    A hw/scsi/mptsas.c
    A hw/scsi/mptsas.h
    M include/hw/pci/pci_ids.h
    M trace-events

  Log Message:
  -----------
  hw: Add support for LSI SAS1068 (mptsas) device

This adds the SAS1068 device, a SAS disk controller used in VMware that
is oldish but widely supported and has decent performance.  Unlike
megasas, it presents itself as a SAS controller and not as a RAID
controller.  The device corresponds to the mptsas kernel driver in
Linux.

A few small things in the device setup are based on Don Slutz's old
patch, but the device emulation was written from scratch based on Don's
SeaBIOS patch and on the FreeBSD and Linux drivers.  It is 2400 lines
shorter than Don's patch (and roughly the same size as MegaSAS---also
because it doesn't support the similar SPI controller), implements SCSI
task management functions (with asynchronous cancellation), supports
big-endian hosts, has complete support for migration and follows the
QEMU coding standards much more closely.

To write the driver, I first split Don's patch in two parts, with
the configuration bits in one file and the rest in a separate file.
I first left mptconfig.c in place and rewrote the rest, then deleted
mptconfig.c as well.  The configuration pages are still based mostly on
VirtualBox's, though not exactly the same.  However, the implementation
is completely different.  The contents of the pages themselves should
not be copyrightable.

Signed-off-by: Don Slutz <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: b9dbb61757eb3606fd1c256bf9b5a257a0475fa1
      
https://github.com/qemu/qemu/commit/b9dbb61757eb3606fd1c256bf9b5a257a0475fa1
  Author: Sitsofe Wheeler <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M qemu-nbd.texi

  Log Message:
  -----------
  qemu-nbd: Fix unintended texi verbatim formatting

Indented lines in the texi meant the perlpod produced interpreted the
paragraph as being verbatim (thus formatting codes were not
interpreted). Fix this by un-indenting problem lines.

Signed-off-by: Sitsofe Wheeler <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 7e8911bb40977aa831b04bbec23895b1f15db501
      
https://github.com/qemu/qemu/commit/7e8911bb40977aa831b04bbec23895b1f15db501
  Author: Sitsofe Wheeler <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M qemu-nbd.texi

  Log Message:
  -----------
  qemu-nbd: Minor texi updates

- Change some spacing.
- Add disconnect usage to synopsis.
- Highlight the command and its options in the synopsis.
- Fix up the grammar in the description.
- Move filename variable description out of the option table.
- Add a description of the dev variable.
- Remove duplicate entry for --format.
- Reword --discard documentation.
- Add --detect-zeroes documentation.
- Add reference to qemu man page to see also section.

Signed-off-by: Sitsofe Wheeler <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 50901218457aebf2eb1c1cac560eb80df3f23b77
      
https://github.com/qemu/qemu/commit/50901218457aebf2eb1c1cac560eb80df3f23b77
  Author: Sitsofe Wheeler <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M qemu-nbd.texi

  Log Message:
  -----------
  qemu-nbd: Fix texi sentence capitalisation

Capitalise the first letter of sentences (and reword for grammar) the
options section of qemu-nbd.texi.

Signed-off-by: Sitsofe Wheeler <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e3dd68df5287e544a41f77a162a02d08fbcb95ba
      
https://github.com/qemu/qemu/commit/e3dd68df5287e544a41f77a162a02d08fbcb95ba
  Author: Janosch Frank <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M scripts/kvm/kvm_stat

  Log Message:
  -----------
  scripts/kvm/kvm_stat: Fix tracefs access checking

On kernels build without CONFIG_TRACING kvm_stat will bail out even
when traces are not used. This is not very helpful, especially if the
user can't install a new kernel. Instead, we should warn the user and
fall back to debugfs statistics.

These changes check if trace statistics were selected without kernel
support, warn with a small timeout, set the debugfs statistics option
to True and the tracefs one to False.

Fixes: 7aa4ee5 ('scripts/kvm/kvm_stat: Improve debugfs access checking')
Signed-off-by: Janosch Frank <address@hidden>
Message-Id: <address@hidden>
[Exit if -t is passed explicitly. - Paolo]
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 667ad26ff8b0e4cabba67322b5e4c35c0b6c4eb8
      
https://github.com/qemu/qemu/commit/667ad26ff8b0e4cabba67322b5e4c35c0b6c4eb8
  Author: John Snow <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M nbd/server.c

  Log Message:
  -----------
  nbd: avoid unaligned uint64_t store

cpu_to_be64w can't be used to make unaligned stores, but stq_be_p can.
Also, the st?_be_p takes a void* so it is more clearly suited to the
case where you're writing into a byte buffer.

Use the st?_be_p family of functions everywhere in nbd/server.c.

Signed-off-by: John Snow <address@hidden>
[Changed to use st?_be_p everywhere. - Paolo]
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 844a3d34d684491b9a483a2d19f801f5b03f5569
      
https://github.com/qemu/qemu/commit/844a3d34d684491b9a483a2d19f801f5b03f5569
  Author: Andrew Jones <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M kvm-all.c

  Log Message:
  -----------
  kvm-all: trace: strerror fixup

Signed-off-by: Andrew Jones <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 1e94f23d82d78a26e7e7cd95502fe31dc5668762
      
https://github.com/qemu/qemu/commit/1e94f23d82d78a26e7e7cd95502fe31dc5668762
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  char: fix repeated registration of tcp chardev I/O handlers

In previous commit:

  commit f2001a7e0555b66d6db25a3ff1801540814045bb
  Author: Daniel P. Berrange <address@hidden>
  Date:   Tue Jan 19 11:14:30 2016 +0000

    char: don't assume telnet initialization will not block

The code which writes the telnet initialization sequence moved
to an event loop callback. If the TCP chardev is opened as a
server in blocking mode (ie -serial telnet:0.0.0.0:3000,server,wait)
this results in a state where the TCP chardev is connected, but not
yet ready to send/recv data when virtual hardware is created.

When the virtual hardware initialization registers its chardev
callbacks, it triggers tcp_chr_update_read_handler, which will
add I/O watches to the connection.

When the telnet initialization finally runs, it will then call
tcp_chr_connect to finish the connection setup. This will in
turn add I/O watches to the connection too.

There are now two sets of I/O watches registered on the same
connection. This ultimately causes data loss on the connection,
for example, when typing into the telnet console only every
second byte is echoed back to the client.

The same flaw can affect channels running with TLS encryption
too, since they also have delayed connection setup completion.

The fix is to update tcp_chr_update_read_handler so that it
avoids registering watches if the connection is not fully
setup yet.

Signed-off-by: Daniel P. Berrange <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: ca2f29f555805d07fb0b9ebfbbfc4e3656530977
      
https://github.com/qemu/qemu/commit/ca2f29f555805d07fb0b9ebfbbfc4e3656530977
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Create gen_lea_v_seg

Add forgotten zero-extension in the TARGET_X86_64, !CODE64, ss32 case;
use this new function to implement gen_string_movl_A0_EDI,
gen_string_movl_A0_ESI, gen_add_A0_ds_seg.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 64ae256c2450262e27f07657c5734d3197458d95
      
https://github.com/qemu/qemu/commit/64ae256c2450262e27f07657c5734d3197458d95
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Introduce mo_stacksize

Centralize computation of a MO_SIZE for the stack pointer.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: d6a2914984c89fa0a3125b9842e0cbf68de79a3d
      
https://github.com/qemu/qemu/commit/d6a2914984c89fa0a3125b9842e0cbf68de79a3d
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Use gen_lea_v_seg in gen_lea_modrm

Centralize handling of segment bases.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 77ebcad04f3659fa7eb799928fdd68280fac720d
      
https://github.com/qemu/qemu/commit/77ebcad04f3659fa7eb799928fdd68280fac720d
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Use gen_lea_v_seg in stack subroutines

I.e. gen_push_v, gen_pop_T0, gen_stack_A0.
More centralization of handling of segment bases.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 3558f8055f37a34762b7a2a0f02687e6eeab893d
      
https://github.com/qemu/qemu/commit/3558f8055f37a34762b7a2a0f02687e6eeab893d
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Access segs via TCG registers

Having segs[].base as a register significantly improves code
generation for real and protected modes, particularly for TBs
that have multiple memory references where the segment base
can be held in a hard register through the TB.

Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: d37ea0c04723f3e15fde55fe97cff6278159929b
      
https://github.com/qemu/qemu/commit/d37ea0c04723f3e15fde55fe97cff6278159929b
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Use gen_lea_v_seg in pusha/popa

More centralization of handling of segment bases.
Also fixes the note about 16-bit wrap around not fully handled.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 743e398e2fbf2f7183bf7a53c9d011fabcaa1770
      
https://github.com/qemu/qemu/commit/743e398e2fbf2f7183bf7a53c9d011fabcaa1770
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/helper.h
    M target-i386/seg_helper.c
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Rewrite gen_enter inline

Use gen_lea_v_seg for centralized segment base knowledge.  Unify
code across 32- and 64-bit.  Fix note about "must save state"
before using the out-of-line helpers.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 2045f04c3ae030bda650f84035f114bbd84909a9
      
https://github.com/qemu/qemu/commit/2045f04c3ae030bda650f84035f114bbd84909a9
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Rewrite leave

Unify the code across stack pointer widths.  Fix the note about
not updating ESP before the potential exception.

Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 4e85057b92d214decf10045d3d4faa2faf33d100
      
https://github.com/qemu/qemu/commit/4e85057b92d214decf10045d3d4faa2faf33d100
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Tidy gen_add_A0_im

Merge gen_op_addl_A0_im and gen_op_addq_A0_im into gen_add_A0_im
and clean up the ifdef.

Replace the one remaining user of gen_op_addl_A0_im with gen_add_A0_im.

Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 1d1cc4d0f481b2939c7e9f6606e571b2fc81971a
      
https://github.com/qemu/qemu/commit/1d1cc4d0f481b2939c7e9f6606e571b2fc81971a
  Author: Richard Henderson <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/translate.c

  Log Message:
  -----------
  target-i386: Deconstruct the cpu_T array

All references to cpu_T are done with a constant index.  It aids
readability to decompose the array into two scalar variables.

Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570
      
https://github.com/qemu/qemu/commit/ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M hw/ipmi/ipmi.c

  Log Message:
  -----------
  ipmi: do not take/drop iothread lock

This is not necessary and actually causes a hang; it was probably copied
and pasted from KVM code, that is one of the very few places that run
outside iothread lock.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 93a5364620dbfcf3cc13866d0e218fc3624c1edf
      
https://github.com/qemu/qemu/commit/93a5364620dbfcf3cc13866d0e218fc3624c1edf
  Author: Corey Minyard <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi_bmc_sim: Fix off by one in check.

Found by Paolo.

Cc: Paolo Bonzini <address@hidden>
Cc: Michael S. Tsirkin <address@hidden>
Cc: Peter Maydell <address@hidden>
Cc: Shannon Zhao <address@hidden>
Cc: Xiao Guangrong <address@hidden>
Cc: Stefan Hajnoczi <address@hidden>
Signed-off-by: Corey Minyard <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 37eebb8693368d890b700cca6e39ec31c7e980e5
      
https://github.com/qemu/qemu/commit/37eebb8693368d890b700cca6e39ec31c7e980e5
  Author: Corey Minyard <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi_bmc_sim: Add break to correct watchdog NMI check

It was falling through when it should have been a break.  Found by
Coverity.  The logic could be simplified a bit with a fallthrough,
probably the original thought, but that would be less clear, I think.

Cc: Paolo Bonzini <address@hidden>
Cc: Michael S. Tsirkin <address@hidden>
Cc: Peter Maydell <address@hidden>
Cc: Shannon Zhao <address@hidden>
Cc: Xiao Guangrong <address@hidden>
Cc: Stefan Hajnoczi <address@hidden>
Signed-off-by: Corey Minyard <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 5056c0c3de73c4d804a62d473039bc439718777d
      
https://github.com/qemu/qemu/commit/5056c0c3de73c4d804a62d473039bc439718777d
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M docs/memory.txt

  Log Message:
  -----------
  docs/memory.txt: Improve list of different memory regions

Improve the part of the memory region documentation which describes
the various different kinds of memory region:
 * add the missing types ROM, IOMMU and reservation
 * mention the functions used to initialize each type, as a hint
   for finding the API docs and examples of use

Signed-off-by: Peter Maydell <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 388ee48a88e684e719660a2cae9c21897b94fa37
      
https://github.com/qemu/qemu/commit/388ee48a88e684e719660a2cae9c21897b94fa37
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M target-i386/helper.c

  Log Message:
  -----------
  target-i386: fix PSE36 mode

(pde & 0x1fe000) is a 32-bit integer; when shifting it
into bits 39-32 the result is zero.  Fix it by making the
mask (and thus the result of the AND) a 64-bit integer.

Reported by Coverity.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 4db84796e75680aaae737fae01d9e8fe100ba028
      
https://github.com/qemu/qemu/commit/4db84796e75680aaae737fae01d9e8fe100ba028
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M scripts/get_maintainer.pl

  Log Message:
  -----------
  get_maintainer.pl: fall back to git if only lists are found

It's not 100% obvious to project newcomers that all patches should be sent
there; checkpatch doesn't say so, and since it mentions other lists to CC,
the wording "the list" from the SubmitAPatch wiki page can be taken
to mean only those lists, not the main list too.  We would like therefore
to add a catch-all entry for address@hidden

On its own, this would break fallback to git, because now every file
has a maintainer of sorts.  Modify get_maintainer.pl so that mailing
lists (L: lines) no longer prevent the fallback, only humans (M:
entries).

Several pre-existing entries have a list but no human.  These now
fall back to git.  That's a feature.

Cc: Paolo Bonzini <address@hidden>
Cc: Markus Armbruster <address@hidden>
Cc: John Snow <address@hidden>
Signed-off-by: Stephen Warren <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: c9a19d5b954fecc4deeb4b83c501ec2faffa88c4
      
https://github.com/qemu/qemu/commit/c9a19d5b954fecc4deeb4b83c501ec2faffa88c4
  Author: Stephen Warren <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: add all-match entry for qemu-devel@

Add an entry to MAINTAINERS that matches every patch, and requests the
user send patches to address@hidden

It's not 100% obvious to project newcomers that all patches should be sent
there; checkpatch doesn't say so, and since it mentions other lists to CC,
the wording "the list" from the SubmitAPatch wiki page can be taken
to mean only those lists, not the main list too.

The F: entries were taken from a similar entry in the Linux kernel.

Cc: Paolo Bonzini <address@hidden>
Cc: Markus Armbruster <address@hidden>
Cc: John Snow <address@hidden>
Signed-off-by: Stephen Warren <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 150dcd1aed6f9ebcf370dbb9b666e7d7c6d908e2
      
https://github.com/qemu/qemu/commit/150dcd1aed6f9ebcf370dbb9b666e7d7c6d908e2
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M io/channel-socket.c
    M qemu-char.c

  Log Message:
  -----------
  qemu-char, io: fix ordering of arguments for UDP socket creation

Two wrongs make a right, but they should be fixed anyway.

Cc: Daniel P. Berrange <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
Message-Id: <address@hidden>


  Commit: c9f19dff101e2c2cf3fa3967eceec2833e845e40
      
https://github.com/qemu/qemu/commit/c9f19dff101e2c2cf3fa3967eceec2833e845e40
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-09 (Tue, 09 Feb 2016)

  Changed paths:
    M MAINTAINERS
    M configure
    M default-configs/pci.mak
    M docs/memory.txt
    M exec.c
    M hw/ipmi/ipmi.c
    M hw/ipmi/ipmi_bmc_sim.c
    M hw/scsi/Makefile.objs
    A hw/scsi/mpi.h
    A hw/scsi/mptconfig.c
    A hw/scsi/mptendian.c
    A hw/scsi/mptsas.c
    A hw/scsi/mptsas.h
    M hw/scsi/scsi-disk.c
    M hw/scsi/scsi-generic.c
    M include/exec/ram_addr.h
    M include/hw/pci/pci_ids.h
    M include/hw/scsi/scsi.h
    M include/qemu/atomic.h
    M io/channel-socket.c
    M kvm-all.c
    M migration/ram.c
    M nbd/server.c
    M qemu-char.c
    M qemu-nbd.texi
    M scripts/get_maintainer.pl
    M scripts/kvm/kvm_stat
    M target-i386/helper.c
    M target-i386/helper.h
    M target-i386/seg_helper.c
    M target-i386/translate.c
    M trace-events

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging

* switch to C11 atomics (Alex)
* Coverity fixes for IPMI (Corey), i386 (Paolo), qemu-char (Paolo)
* at long last, fail on wrong .pc files if -m32 is in use (Daniel)
* qemu-char regression fix (Daniel)
* SAS1068 device (Paolo)
* memory region docs improvements (Peter)
* target-i386 cleanups (Richard)
* qemu-nbd docs improvements (Sitsofe)
* thread-safe memory hotplug (Stefan)

# gpg: Signature made Tue 09 Feb 2016 16:09:30 GMT using RSA key ID 78C7AE83
# gpg: Good signature from "Paolo Bonzini <address@hidden>"
# gpg:                 aka "Paolo Bonzini <address@hidden>"

* remotes/bonzini/tags/for-upstream: (33 commits)
  qemu-char, io: fix ordering of arguments for UDP socket creation
  MAINTAINERS: add all-match entry for qemu-devel@
  get_maintainer.pl: fall back to git if only lists are found
  target-i386: fix PSE36 mode
  docs/memory.txt: Improve list of different memory regions
  ipmi_bmc_sim: Add break to correct watchdog NMI check
  ipmi_bmc_sim: Fix off by one in check.
  ipmi: do not take/drop iothread lock
  target-i386: Deconstruct the cpu_T array
  target-i386: Tidy gen_add_A0_im
  target-i386: Rewrite leave
  target-i386: Rewrite gen_enter inline
  target-i386: Use gen_lea_v_seg in pusha/popa
  target-i386: Access segs via TCG registers
  target-i386: Use gen_lea_v_seg in stack subroutines
  target-i386: Use gen_lea_v_seg in gen_lea_modrm
  target-i386: Introduce mo_stacksize
  target-i386: Create gen_lea_v_seg
  char: fix repeated registration of tcp chardev I/O handlers
  kvm-all: trace: strerror fixup
  ...

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/f075c89f0a9c...c9f19dff101e

reply via email to

[Prev in Thread] Current Thread [Next in Thread]