[Qemu-commits] [qemu/qemu] 55ad78: use g_path_get_dirname instead of dirname

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 55ad781ca7fcaed19e04554b558727206d638a99
      
https://github.com/qemu/qemu/commit/55ad781ca7fcaed19e04554b558727206d638a99
  Author: Wei Jiangang <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M os-posix.c
    M util/oslib-posix.c

  Log Message:
  -----------
  use g_path_get_dirname instead of dirname

Use g_path_get_basename to get the directory components of
a file name, and free its return when no longer needed.

Signed-off-by: Wei Jiangang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: fd9a30483091eb2e297401dd1d860045188d779a
      
https://github.com/qemu/qemu/commit/fd9a30483091eb2e297401dd1d860045188d779a
  Author: Md Haris Iqbal <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M bsd-user/qemu.h

  Log Message:
  -----------
  Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h

Signed-off-by: Md Haris Iqbal <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Sean Bruno <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 3cbeb52467a8b8f3a836c7783a7ebc6450c5ef30
      
https://github.com/qemu/qemu/commit/3cbeb52467a8b8f3a836c7783a7ebc6450c5ef30
  Author: Antonio Borneo <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  hw/i386: add device tree support

With "-dtb" on command-line:
- append the device tree blob to the kernel image;
- pass the blob's pointer to the kernel through setup_data, as
  requested by upstream kernel commit da6b737b9ab7 ("x86: Add
  device tree support").

The device tree blob is passed as-is to the guest; none of its
fields is modified nor updated. This is not an issue; the kernel
commit above uses the device tree only as an extension to the
traditional kernel configuration.

To: "Michael S. Tsirkin" <address@hidden>
To: Paolo Bonzini <address@hidden>
To: Richard Henderson <address@hidden>
To: Eduardo Habkost <address@hidden>
Cc: address@hidden
Cc: Sebastian Andrzej Siewior <address@hidden>
Signed-off-by: Antonio Borneo <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 8bff06a0bbf257a2083223534c1607bf87d913e6
      
https://github.com/qemu/qemu/commit/8bff06a0bbf257a2083223534c1607bf87d913e6
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M include/qemu/compiler.h
    M tcg/tcg.h

  Log Message:
  -----------
  compiler: never omit assertions if using a static analysis tool

Assertions help both Coverity and the clang static analyzer avoid
false positives, but on the other hand both are confused when
the condition is compiled as (void)(x != FOO).  Always expand
assertion macros when using Coverity or clang, through a new
QEMU_STATIC_ANALYSIS preprocessor symbol.

This fixes a couple false positives in TCG.

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


  Commit: 8cc46787b5b58f01a11c919c7ff939ed009e27fc
      
https://github.com/qemu/qemu/commit/8cc46787b5b58f01a11c919c7ff939ed009e27fc
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M hw/scsi/megasas.c

  Log Message:
  -----------
  megasas: remove useless check for cmd->frame

megasas_enqueue_frame always returns with non-NULL cmd->frame.
Remove the "else" part as it is dead code.

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


  Commit: d211bd6016a5d2d59911c6d3d343f114e9853366
      
https://github.com/qemu/qemu/commit/d211bd6016a5d2d59911c6d3d343f114e9853366
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M scripts/checkpatch.pl

  Log Message:
  -----------
  checkpatch: consider git extended headers valid patches

Renames look like this with git-diff(1) when diff.renames = true is set:

  diff --git a/a b/b
  similarity index 100%
  rename from a
  rename to b

This raises the "Does not appear to be a unified-diff format patch"
error because checkpatch.pl only considers a diff valid if it contains
at least one "@@" hunk.

This patch accepts renames and copies too so that checkpatch.pl exits
successfully when a diff only renames/copies files.  The git diff
extended header format is described on the git-diff(1) man page.

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


  Commit: 101420b886eec36990419bc9ed5b503622af8a0d
      
https://github.com/qemu/qemu/commit/101420b886eec36990419bc9ed5b503622af8a0d
  Author: Peter Lieven <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec: avoid realloc in phys_map_node_reserve

this is the first step in reducing the brk heap fragmentation
created by the map->nodes memory allocation. Since the introduction
of RCU the freeing of the PhysPageMaps is delayed so that sometimes
several hundred are allocated at the same time.

Even worse the memory for map->nodes is allocated and shortly
afterwards reallocated. Since the number of nodes it grows
to in the end is the same for all PhysPageMaps remember this value
and at least avoid the reallocation.

The large number of simultaneous allocations (about 450 x 70kB in
my configuration) has to be addressed later.

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


  Commit: ca7d8e1c9ccb95257a9c9288617b270727ec3e79
      
https://github.com/qemu/qemu/commit/ca7d8e1c9ccb95257a9c9288617b270727ec3e79
  Author: Sergey Fedorov <address@hidden>
  Date:   2016-07-17 (Sun, 17 Jul 2016)

  Changed paths:
    M cpu-exec.c

  Log Message:
  -----------
  cpu-exec: Move down some declarations in cpu_exec()

This will fix a compiler warning with -Wclobbered:

http://lists.nongnu.org/archive/html/qemu-devel/2016-07/msg03347.html

Reported-by: Stefan Weil <address@hidden>
Signed-off-by: Sergey Fedorov <address@hidden>
Signed-off-by: Sergey Fedorov <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e5dfc5e8e715c572aea44ac4d96c43941d4741c7
      
https://github.com/qemu/qemu/commit/e5dfc5e8e715c572aea44ac4d96c43941d4741c7
  Author: Pranith Kumar <address@hidden>
  Date:   2016-07-19 (Tue, 19 Jul 2016)

  Changed paths:
    R README
    A README.md

  Log Message:
  -----------
  Move README to markdown

Move the README file to markdown so that it makes the github page look
prettier. I know that github repo is a mirror and not the official
repo, but I think it doesn't hurt to have it in markdown format.

Signed-off-by: Pranith Kumar <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: eb36b953e0ebf4129b188a241fbc367062ac2e06
      
https://github.com/qemu/qemu/commit/eb36b953e0ebf4129b188a241fbc367062ac2e06
  Author: Peter Lieven <address@hidden>
  Date:   2016-07-19 (Tue, 19 Jul 2016)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  block/iscsi: fix rounding in iscsi_allocationmap_set

when setting clusters as alloacted the boundaries have
to be expanded. As Paolo pointed out the calculation of
the number of clusters is wrong:

Suppose cluster_sectors is 2, sector_num = 1, nb_sectors = 6:

In the "mark allocated" case, you want to set 0..8, i.e.
cluster_num=0, nb_clusters=4.

   0--.--2--.--4--.--6--.--8
   <--|_________________|-->  (<--> = expanded)

Instead you are setting nb_clusters=3, so that 6..8 is not marked.

   0--.--2--.--4--.--6--.--8
   <--|______________|!!!     (! = wrong)

Cc: address@hidden
Reported-by: Paolo Bonzini <address@hidden>
Signed-off-by: Peter Lieven <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e1123a3b40a1a9a625a29c8ed4debb7e206ea690
      
https://github.com/qemu/qemu/commit/e1123a3b40a1a9a625a29c8ed4debb7e206ea690
  Author: Peter Lieven <address@hidden>
  Date:   2016-07-19 (Tue, 19 Jul 2016)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  block/iscsi: allow caching of the allocation map

until now the allocation map was used only as a hint if a cluster
is allocated or not. If a block was not allocated (or Qemu had
no info about the allocation status) a get_block_status call was
issued to check the allocation status and possibly avoid
a subsequent read of unallocated sectors. If a block known to be
allocated the get_block_status call was omitted. In the other case
a get_block_status call was issued before every read to avoid
the necessity for a consistent allocation map. To avoid the
potential overhead of calling get_block_status for each and
every read request this took only place for the bigger requests.

This patch enhances this mechanism to cache the allocation
status and avoid calling get_block_status for blocks where
the allocation status has been queried before. This allows
for bypassing the read request even for smaller requests and
additionally omits calling get_block_status for known to be
unallocated blocks.

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


  Commit: da6d48e3348bbc266896cf8adf0c33f1eaf5b31f
      
https://github.com/qemu/qemu/commit/da6d48e3348bbc266896cf8adf0c33f1eaf5b31f
  Author: Sergey Fedorov <address@hidden>
  Date:   2016-07-19 (Tue, 19 Jul 2016)

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

  Log Message:
  -----------
  target-i386: Remove redundant HF_SOFTMMU_MASK

'HF_SOFTMMU_MASK' is only set when 'CONFIG_SOFTMMU' is defined. So
there's no need in this flag: test 'CONFIG_SOFTMMU' instead.

Suggested-by: Paolo Bonzini <address@hidden>
Signed-off-by: Sergey Fedorov <address@hidden>
Signed-off-by: Sergey Fedorov <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: a3b343772114c8c98986508f3352a631164f913c
      
https://github.com/qemu/qemu/commit/a3b343772114c8c98986508f3352a631164f913c
  Author: Peter Maydell <address@hidden>
  Date:   2016-07-19 (Tue, 19 Jul 2016)

  Changed paths:
    R README
    A README.md
    M block/iscsi.c
    M bsd-user/qemu.h
    M cpu-exec.c
    M exec.c
    M hw/i386/pc.c
    M hw/scsi/megasas.c
    M include/qemu/compiler.h
    M os-posix.c
    M target-i386/cpu.c
    M target-i386/cpu.h
    M target-i386/translate.c
    M tcg/tcg.h
    M util/oslib-posix.c

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

* two old patches from prospective GSoC students
* i386 -kernel device tree support
* Coverity fix
* memory usage improvement from Peter
* checkpatch fix
* g_path_get_dirname cleanup
* caching of block status for iSCSI

# gpg: Signature made Tue 19 Jul 2016 07:43:41 BST
# gpg:                using RSA key 0xBFFBD25F78C7AE83
# gpg: Good signature from "Paolo Bonzini <address@hidden>"
# gpg:                 aka "Paolo Bonzini <address@hidden>"
# Primary key fingerprint: 46F5 9FBD 57D6 12E7 BFD4  E2F7 7E15 100C CD36 69B1
#      Subkey fingerprint: F133 3857 4B66 2389 866C  7682 BFFB D25F 78C7 AE83

* remotes/bonzini/tags/for-upstream:
  target-i386: Remove redundant HF_SOFTMMU_MASK
  block/iscsi: allow caching of the allocation map
  block/iscsi: fix rounding in iscsi_allocationmap_set
  Move README to markdown
  cpu-exec: Move down some declarations in cpu_exec()
  exec: avoid realloc in phys_map_node_reserve
  checkpatch: consider git extended headers valid patches
  megasas: remove useless check for cmd->frame
  compiler: never omit assertions if using a static analysis tool
  hw/i386: add device tree support
  Changed malloc to g_malloc, free to g_free in bsd-user/qemu.h
  use g_path_get_dirname instead of dirname

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


Compare: https://github.com/qemu/qemu/compare/db3e07dbdecd...a3b343772114