qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] a2da73: hw/nvme: skip queue processing if not


From: Richard Henderson
Subject: [Qemu-commits] [qemu/qemu] a2da73: hw/nvme: skip queue processing if notifier is cleared
Date: Mon, 01 Aug 2022 08:49:51 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: a2da737729efbd9e35eded30a5b8966423bde62d
      
https://github.com/qemu/qemu/commit/a2da737729efbd9e35eded30a5b8966423bde62d
  Author: Klaus Jensen <k.jensen@samsung.com>
  Date:   2022-08-01 (Mon, 01 Aug 2022)

  Changed paths:
    M hw/nvme/ctrl.c

  Log Message:
  -----------
  hw/nvme: skip queue processing if notifier is cleared

While it is safe to process the queues when they are empty, skip it if
the event notifier callback was invoked spuriously.

Reviewed-by: Keith Busch <kbusch@kernel.org>
Reviewed-by: Jinhao Fan <fanjinhao21s@ict.ac.cn>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>


  Commit: 04e8da889016e77a3b9fea7e11569f61aadafbcb
      
https://github.com/qemu/qemu/commit/04e8da889016e77a3b9fea7e11569f61aadafbcb
  Author: Klaus Jensen <k.jensen@samsung.com>
  Date:   2022-08-01 (Mon, 01 Aug 2022)

  Changed paths:
    M hw/nvme/ctrl.c

  Log Message:
  -----------
  hw/nvme: unregister the event notifier handler on the main loop

Make sure the notifier handler is unregistered in the main loop prior to
cleaning it up.

Fixes: 2e53b0b45024 ("hw/nvme: Use ioeventfd to handle doorbell updates")
Reviewed-by: Keith Busch <kbusch@kernel.org>
Reviewed-by: Jinhao Fan <fanjinhao21s@ict.ac.cn>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>


  Commit: e2e137f64282a2ee2f359b6df4cd93c83a308e7b
      
https://github.com/qemu/qemu/commit/e2e137f64282a2ee2f359b6df4cd93c83a308e7b
  Author: Klaus Jensen <k.jensen@samsung.com>
  Date:   2022-08-01 (Mon, 01 Aug 2022)

  Changed paths:
    M hw/nvme/ctrl.c

  Log Message:
  -----------
  hw/nvme: do not enable ioeventfd by default

Do not enable ioeventfd by default. Let the feature mature a bit before
we consider enabling it by default.

Fixes: 2e53b0b45024 ("hw/nvme: Use ioeventfd to handle doorbell updates")
Reviewed-by: Keith Busch <kbusch@kernel.org>
Reviewed-by: Jinhao Fan <fanjinhao21s@ict.ac.cn>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>


  Commit: e34de2d13a1f83e952372a3f406b849b79c16584
      
https://github.com/qemu/qemu/commit/e34de2d13a1f83e952372a3f406b849b79c16584
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2022-08-01 (Mon, 01 Aug 2022)

  Changed paths:
    M hw/nvme/ctrl.c

  Log Message:
  -----------
  Merge tag 'nvme-next-pull-request' of git://git.infradead.org/qemu-nvme into 
staging

hw/nvme fixes

Some fixes for hw/nvme ioeventfd support.

# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCAAdFiEEUigzqnXi3OaiR2bATeGvMW1PDekFAmLnpQ4ACgkQTeGvMW1P
# Delnywf/fM7Qp5WMR0ExHqVuKfpN1EBK8C6wa3xp7EI0pCDP50ZwJfcMmeZOhDAe
# 3KXdNTXV43cxygkdzG/uxh7j26Zq8zGzcf1RVLAIUqzL9wTkoFcJ4+aIuzlV4wjU
# oS1jK+pItJ9/Kb9SeavVGOzTzyBzs2G9odRvjyjGrwrEfpEBEtncwfXAK7mPqt1V
# TNDR0M0qJ9Bheaoqs7miqa4zOevtKKp3pPeG0n2BOhWromGx6MbeHwTuNg96zR0+
# VXuEmUyj8r6ezKmQdgnJPoqFCwEJI0iBGPSKhpIp3EGrkAww2IuBAN8C4bo03Bzq
# Fj3pne5Sr9iBV/nhz9vN267STQzdsw==
# =AL7y
# -----END PGP SIGNATURE-----
# gpg: Signature made Mon 01 Aug 2022 03:03:58 AM PDT
# gpg:                using RSA key 522833AA75E2DCE6A24766C04DE1AF316D4F0DE9
# gpg: Good signature from "Klaus Jensen <its@irrelevant.dk>" [unknown]
# gpg:                 aka "Klaus Jensen <k.jensen@samsung.com>" [unknown]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: DDCA 4D9C 9EF9 31CC 3468  4272 63D5 6FC5 E55D A838
#      Subkey fingerprint: 5228 33AA 75E2 DCE6 A247  66C0 4DE1 AF31 6D4F 0DE9

* tag 'nvme-next-pull-request' of git://git.infradead.org/qemu-nvme:
  hw/nvme: do not enable ioeventfd by default
  hw/nvme: unregister the event notifier handler on the main loop
  hw/nvme: skip queue processing if notifier is cleared

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/4e06b3fc1b5e...e34de2d13a1f



reply via email to

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