[Qemu-commits] [qemu/qemu] 192392: 9p: use g_new(T, n) instead of g_malloc(sizeof(T) ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 1923923bfac0df7bd527af59c4e413dbb0f59626
      
https://github.com/qemu/qemu/commit/1923923bfac0df7bd527af59c4e413dbb0f59626
  Author: Greg Kurz <address@hidden>
  Date:   2018-12-12 (Wed, 12 Dec 2018)

  Changed paths:
    M hw/9pfs/9p.c

  Log Message:
  -----------
  9p: use g_new(T, n) instead of g_malloc(sizeof(T) * n)

Because it is a recommended coding practice (see HACKING).

Signed-off-by: Greg Kurz <address@hidden>
Reviewed-by: Anthony PERARD <address@hidden>


  Commit: 75607e0dcc89af8c3a1ee025bdaf5e91891e1ccd
      
https://github.com/qemu/qemu/commit/75607e0dcc89af8c3a1ee025bdaf5e91891e1ccd
  Author: Greg Kurz <address@hidden>
  Date:   2018-12-12 (Wed, 12 Dec 2018)

  Changed paths:
    M hw/9pfs/xen-9p-backend.c

  Log Message:
  -----------
  xen/9pfs: use g_new(T, n) instead of g_malloc(sizeof(T) * n)

Because it is a recommended coding practice (see HACKING).

Signed-off-by: Greg Kurz <address@hidden>
Acked-by: Anthony PERARD <address@hidden>


  Commit: 93aee84f575d46699f49af3c96194012527e0b22
      
https://github.com/qemu/qemu/commit/93aee84f575d46699f49af3c96194012527e0b22
  Author: Greg Kurz <address@hidden>
  Date:   2018-12-12 (Wed, 12 Dec 2018)

  Changed paths:
    M fsdev/qemu-fsdev.c
    R hw/9pfs/9p-handle.c
    M hw/9pfs/Makefile.objs
    M qemu-deprecated.texi
    M qemu-options.hx

  Log Message:
  -----------
  9p: remove support for the "handle" backend

The "handle" fsdev backend was deprecated in QEMU 2.12.0 with:

commit db3b3c7281ca82e2647e072a1f97db111313dd73
Author: Greg Kurz <address@hidden>
Date:   Mon Jan 8 11:18:23 2018 +0100

    9pfs: deprecate handle backend

    This backend raise some concerns:

    - doesn't support symlinks
    - fails +100 tests in the PJD POSIX file system test suite [1]
    - requires the QEMU process to run with the CAP_DAC_READ_SEARCH
      capability, which isn't recommended for security reasons

    This backend should not be used and wil be removed. The 'local'
    backend is the recommended alternative.

    [1] https://www.tuxera.com/community/posix-test-suite/

    Signed-off-by: Greg Kurz <address@hidden>
    Reviewed-by: Daniel P. Berrange <address@hidden>
    Reviewed-by: Aneesh Kumar K.V <address@hidden>

It has passed the two release cooling period without any complaint.

Remove it now.

Signed-off-by: Greg Kurz <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>


  Commit: d058a37a6e8daa8d71a6f2b613eb415b69363755
      
https://github.com/qemu/qemu/commit/d058a37a6e8daa8d71a6f2b613eb415b69363755
  Author: Peter Maydell <address@hidden>
  Date:   2018-12-14 (Fri, 14 Dec 2018)

  Changed paths:
    M fsdev/qemu-fsdev.c
    R hw/9pfs/9p-handle.c
    M hw/9pfs/9p.c
    M hw/9pfs/Makefile.objs
    M hw/9pfs/xen-9p-backend.c
    M qemu-deprecated.texi
    M qemu-options.hx

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

Most notable change in this PR is the full removal of the "handle" fsdev
backend.

# gpg: Signature made Wed 12 Dec 2018 13:20:42 GMT
# gpg:                using RSA key 71D4D5E5822F73D6
# gpg: Good signature from "Greg Kurz <address@hidden>"
# gpg:                 aka "Gregory Kurz <address@hidden>"
# gpg:                 aka "[jpeg image of size 3330]"
# Primary key fingerprint: B482 8BAF 9431 40CE F2A3  4910 71D4 D5E5 822F 73D6

* remotes/gkurz/tags/for-upstream:
  9p: remove support for the "handle" backend
  xen/9pfs: use g_new(T, n) instead of g_malloc(sizeof(T) * n)
  9p: use g_new(T, n) instead of g_malloc(sizeof(T) * n)

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


Compare: https://github.com/qemu/qemu/compare/110b1a8c7c2b...d058a37a6e8d
      **NOTE:** This service has been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.