[Qemu-commits] [qemu/qemu] 9a92aa: tests: Remove (mostly) useless architecture checks

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 9a92aa3b176407ccc7e433ecf2a653433cde1851
      
https://github.com/qemu/qemu/commit/9a92aa3b176407ccc7e433ecf2a653433cde1851
  Author: Thomas Huth <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M tests/fdc-test.c
    M tests/ide-test.c
    M tests/ipmi-bt-test.c
    M tests/ipmi-kcs-test.c

  Log Message:
  -----------
  tests: Remove (mostly) useless architecture checks

These checks at the beginning of some of the tests are mostly useless:
We only run the tests on x86 anyway, and g_test_message() does not
print anything unless you call g_test_init() first.

Signed-off-by: Thomas Huth <address@hidden>
Acked-by: John Snow <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 49e5ef07d71efeeaab3be65e034d34a2189d36ff
      
https://github.com/qemu/qemu/commit/49e5ef07d71efeeaab3be65e034d34a2189d36ff
  Author: Wei Yang <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M hw/i386/pc.c
    M include/hw/i386/pc.h

  Log Message:
  -----------
  hw/i386/pc.c: remove unused function pc_acpi_init()

Function pc_acpi_init() is not used anymore.

Remove the definition and declaration.

Signed-off-by: Wei Yang <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 120fc4eb94dcacf8f451c09fc280b18c8c192f1a
      
https://github.com/qemu/qemu/commit/120fc4eb94dcacf8f451c09fc280b18c8c192f1a
  Author: Wei Yang <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M hw/acpi/core.c
    M include/hw/acpi/acpi.h

  Log Message:
  -----------
  hw/acpi: remove unused function acpi_table_add_builtin()

Function acpi_table_add_builtin() is not used anymore.

Remove the definition and declaration.

Signed-off-by: Wei Yang <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 7d9b68acb37059bee347ce7e4c783ca5b7cf7ae0
      
https://github.com/qemu/qemu/commit/7d9b68acb37059bee347ce7e4c783ca5b7cf7ae0
  Author: Wei Yang <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M hw/acpi/core.c

  Log Message:
  -----------
  hw/acpi: remove unnecessary variable acpi_table_builtin

acpi_table_builtin is now always false, it is not necessary to check it
again.

This patch just removes it.

Signed-off-by: Wei Yang <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Tested-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: d80cf1eb2e87df3a9bfb226bcc7fb3a1aa858817
      
https://github.com/qemu/qemu/commit/d80cf1eb2e87df3a9bfb226bcc7fb3a1aa858817
  Author: Stephen Checkoway <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M hw/block/pflash_cfi02.c

  Log Message:
  -----------
  block/pflash_cfi02: Fix memory leak and potential use-after-free

Don't dynamically allocate the pflash's timer. But do use timer_del in
an unrealize function to make sure that the timer can't fire after the
pflash_t has been freed.

Signed-off-by: Stephen Checkoway <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 806be3734c3c55357f15ab6548df86433f81859f
      
https://github.com/qemu/qemu/commit/806be3734c3c55357f15ab6548df86433f81859f
  Author: Like Xu <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M docs/COLO-FT.txt
    M docs/amd-memory-encryption.txt
    M docs/can.txt
    M docs/colo-proxy.txt
    M docs/cpu-hotplug.rst
    M docs/qcow2-cache.txt
    M docs/qemu-block-drivers.texi
    M docs/qemu-cpu-models.texi
    M docs/rdma.txt
    M docs/replay.txt
    M docs/vfio-ap.txt

  Log Message:
  -----------
  doc: fix typos for documents in tree

Signed-off-by: Like Xu <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: acfea13780ab3e98fc9fa5f7773da6ee7472c42b
      
https://github.com/qemu/qemu/commit/acfea13780ab3e98fc9fa5f7773da6ee7472c42b
  Author: Greg Kurz <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M include/qemu/bswap.h

  Log Message:
  -----------
  bswap: Fix accessors syntax in comment

All accessors that have an endian infix DO have an underscore between
{size} and {endian}.

Signed-off-by: Greg Kurz <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 9ab71e5ee5167ca6e5c42a386b82060a7ca55b13
      
https://github.com/qemu/qemu/commit/9ab71e5ee5167ca6e5c42a386b82060a7ca55b13
  Author: Markus Armbruster <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M rules.mak

  Log Message:
  -----------
  build: Correct explanation of unnest-vars example

Cc: Fam Zheng <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 5c7ba877ef54e6a90982db729b4fda8c4790ef67
      
https://github.com/qemu/qemu/commit/5c7ba877ef54e6a90982db729b4fda8c4790ef67
  Author: Igor Mammedov <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M backends/hostmem-file.c

  Log Message:
  -----------
  hostmem-file: simplify ifdef-s in file_backend_memory_alloc()

cleanup file_backend_memory_alloc() by using one CONFIG_POSIX ifdef
instead of several ones within the function to make it simpler to follow.

Signed-off-by: Igor Mammedov <address@hidden>
Suggested-by: Wei Yang <address@hidden>
Reviewed-by: Wei Yang <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
[lv: s/hostmem/hostmem-file/]
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 0e081fde8a3d80383adf2e802fc0c03af44c5436
      
https://github.com/qemu/qemu/commit/0e081fde8a3d80383adf2e802fc0c03af44c5436
  Author: Aarushi Mehta <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M thunk.c

  Log Message:
  -----------
  thunk: fix of malloc to g_new

Note that since thunking occurs throughout the lifetime of the QEMU
instance, there is no matching 'free' to correct.

Signed-off-by: Aarushi Mehta <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Laurent Vivier <address@hidden>


  Commit: 9b748c5e061b1202fba59afd857e16a693743d90
      
https://github.com/qemu/qemu/commit/9b748c5e061b1202fba59afd857e16a693743d90
  Author: Peter Maydell <address@hidden>
  Date:   2019-03-06 (Wed, 06 Mar 2019)

  Changed paths:
    M backends/hostmem-file.c
    M docs/COLO-FT.txt
    M docs/amd-memory-encryption.txt
    M docs/can.txt
    M docs/colo-proxy.txt
    M docs/cpu-hotplug.rst
    M docs/qcow2-cache.txt
    M docs/qemu-block-drivers.texi
    M docs/qemu-cpu-models.texi
    M docs/rdma.txt
    M docs/replay.txt
    M docs/vfio-ap.txt
    M hw/acpi/core.c
    M hw/block/pflash_cfi02.c
    M hw/i386/pc.c
    M include/hw/acpi/acpi.h
    M include/hw/i386/pc.h
    M include/qemu/bswap.h
    M rules.mak
    M tests/fdc-test.c
    M tests/ide-test.c
    M tests/ipmi-bt-test.c
    M tests/ipmi-kcs-test.c
    M thunk.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/vivier2/tags/trivial-branch-pull-request' into staging

trivial patches pull request (20190206)

- acpi: remove unused functions/variables
- tests: remove useless architecture checks
- some typo fixes and documentation update
- flash_cfi02: fix memory leak

# gpg: Signature made Wed 06 Mar 2019 11:05:12 GMT
# gpg:                using RSA key F30C38BD3F2FBE3C
# gpg: Good signature from "Laurent Vivier <address@hidden>" [full]
# gpg:                 aka "Laurent Vivier <address@hidden>" [full]
# gpg:                 aka "Laurent Vivier (Red Hat) <address@hidden>" [full]
# Primary key fingerprint: CD2F 75DD C8E3 A4DC 2E4F  5173 F30C 38BD 3F2F BE3C

* remotes/vivier2/tags/trivial-branch-pull-request:
  thunk: fix of malloc to g_new
  hostmem-file: simplify ifdef-s in file_backend_memory_alloc()
  build: Correct explanation of unnest-vars example
  bswap: Fix accessors syntax in comment
  doc: fix typos for documents in tree
  block/pflash_cfi02: Fix memory leak and potential use-after-free
  hw/acpi: remove unnecessary variable acpi_table_builtin
  hw/acpi: remove unused function acpi_table_add_builtin()
  hw/i386/pc.c: remove unused function pc_acpi_init()
  tests: Remove (mostly) useless architecture checks

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


Compare: https://github.com/qemu/qemu/compare/b5b6b2b912bb...9b748c5e061b