qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] 00a252: MAINTAINERS: add missing support stat


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 00a252: MAINTAINERS: add missing support status fields
Date: Sat, 09 Mar 2019 12:54:13 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 00a25291e6fbde6a174af07bc2cf2e9f0771c170
      
https://github.com/qemu/qemu/commit/00a25291e6fbde6a174af07bc2cf2e9f0771c170
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2019-03-07 (Thu, 07 Mar 2019)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: add missing support status fields

This patch adds the "S:" line for areas of the codebase that currently
lack a support status field.

Note that there are a few more areas that are more abstract and do not
correspond to a specific set of files.  They have not been modified.

Cc: Alex Bennée <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: bf4069fbd72ae11f6af82f9150d3396c8a95bb18
      
https://github.com/qemu/qemu/commit/bf4069fbd72ae11f6af82f9150d3396c8a95bb18
  Author: Anastasiia Rusakova <address@hidden>
  Date:   2019-03-07 (Thu, 07 Mar 2019)

  Changed paths:
    M hw/block/virtio-blk.c

  Log Message:
  -----------
  hw/block/virtio-blk: Clean req->dev repetitions

Some functions sometimes uses req->dev even though a local variable
VirtIOBlock* s = req->dev has already been defined.
Updated places to use s everywhere in the file.

Signed-off-by: Anastasiia Rusakova <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 21c4d15b4708b7d30c450041a560df670f36cac8
      
https://github.com/qemu/qemu/commit/21c4d15b4708b7d30c450041a560df670f36cac8
  Author: Peter Xu <address@hidden>
  Date:   2019-03-08 (Fri, 08 Mar 2019)

  Changed paths:
    M include/sysemu/iothread.h
    M iothread.c

  Log Message:
  -----------
  iothread: replace init_done_cond with a semaphore

Only sending an init-done message using lock+cond seems an overkill to
me.  Replacing it with a simpler semaphore.

Meanwhile, init the semaphore unconditionally, then we can destroy it
unconditionally too in finalize which seems cleaner.

Signed-off-by: Peter Xu <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: b506e0f16c05b68b776752909e4451b54d461fd1
      
https://github.com/qemu/qemu/commit/b506e0f16c05b68b776752909e4451b54d461fd1
  Author: Peter Xu <address@hidden>
  Date:   2019-03-08 (Fri, 08 Mar 2019)

  Changed paths:
    M include/sysemu/iothread.h
    M iothread.c

  Log Message:
  -----------
  iothread: create the gcontext unconditionally

In existing code we create the gcontext dynamically at the first
access of the gcontext from caller.  That can bring some complexity
and potential races during using iothread.  Since the context itself
is not that big a resource, and we won't have millions of iothread,
let's simply create the gcontext unconditionally.

This will also be a preparation work further to move the thread
context push operation earlier than before (now it's only pushed right
before we want to start running the gmainloop).

Removing the g_once since it's not necessary, while introducing a new
run_gcontext boolean to show whether we want to run the gcontext.

Reviewed-by: Marc-André Lureau <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 0bd2d233c628dcfcd2b24b7b7f59e8fcd2dcbae3
      
https://github.com/qemu/qemu/commit/0bd2d233c628dcfcd2b24b7b7f59e8fcd2dcbae3
  Author: Peter Xu <address@hidden>
  Date:   2019-03-08 (Fri, 08 Mar 2019)

  Changed paths:
    M iothread.c

  Log Message:
  -----------
  iothread: create main loop unconditionally

Since we've have the gcontext always there, create the main loop
altogether.  The iothread_run() is even cleaner.

Reviewed-by: Marc-André Lureau <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: b60ec76a13b0d2cd06b4a681f025c180f4dd79a1
      
https://github.com/qemu/qemu/commit/b60ec76a13b0d2cd06b4a681f025c180f4dd79a1
  Author: Peter Xu <address@hidden>
  Date:   2019-03-08 (Fri, 08 Mar 2019)

  Changed paths:
    M iothread.c

  Log Message:
  -----------
  iothread: push gcontext earlier in the thread_fn

We were pushing the context until right before running the gmainloop.
Now since we have everything unconditionally, we can move this
earlier.

One benefit is that now it's done even before init_done_sem, so as
long as the iothread user calls iothread_create() and completes, we
know that the thread stack is ready.

Signed-off-by: Peter Xu <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>

[Tweaked comment wording as discussed with Peter Xu.
--Stefan]

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


  Commit: 6ca206204fa773c8626d59caf2a5676d6cc35f52
      
https://github.com/qemu/qemu/commit/6ca206204fa773c8626d59caf2a5676d6cc35f52
  Author: Peter Xu <address@hidden>
  Date:   2019-03-08 (Fri, 08 Mar 2019)

  Changed paths:
    M iothread.c

  Log Message:
  -----------
  iothread: document about why we need explicit aio_poll()

After consulting Paolo I know why we'd better keep the explicit
aio_poll() in iothread_run().  Document it directly into the code so
that future readers will know the answer from day one.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Message-id: address@hidden
Message-Id: <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: f5b4c31030f45293bb4517445722768434829d91
      
https://github.com/qemu/qemu/commit/f5b4c31030f45293bb4517445722768434829d91
  Author: Peter Maydell <address@hidden>
  Date:   2019-03-09 (Sat, 09 Mar 2019)

  Changed paths:
    M MAINTAINERS
    M hw/block/virtio-blk.c
    M include/sysemu/iothread.h
    M iothread.c

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

Pull request

# gpg: Signature made Fri 08 Mar 2019 16:53:34 GMT
# gpg:                using RSA key 9CA4ABB381AB73C8
# gpg: Good signature from "Stefan Hajnoczi <address@hidden>" [full]
# gpg:                 aka "Stefan Hajnoczi <address@hidden>" [full]
# Primary key fingerprint: 8695 A8BF D3F9 7CDA AC35  775A 9CA4 ABB3 81AB 73C8

* remotes/stefanha/tags/block-pull-request:
  iothread: document about why we need explicit aio_poll()
  iothread: push gcontext earlier in the thread_fn
  iothread: create main loop unconditionally
  iothread: create the gcontext unconditionally
  iothread: replace init_done_cond with a semaphore
  hw/block/virtio-blk: Clean req->dev repetitions
  MAINTAINERS: add missing support status fields

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


Compare: https://github.com/qemu/qemu/compare/4c7613748487...f5b4c31030f4



reply via email to

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