[Top][All Lists]

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

Re: [Qemu-block] [Qemu-devel] [PATCH] aio-posix: fill error message that

From: Cao jin
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH] aio-posix: fill error message that is missed
Date: Wed, 13 Jul 2016 09:42:31 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0

On 07/12/2016 11:41 PM, Eric Blake wrote:
On 07/12/2016 05:34 AM, Cao jin wrote:
The current judegement of caller is meaningless, make it useful.

Is this something you can trigger? If so, what command line?  If not,
how did you find it?

No, I didn't find it by "trigger" it. I just a very carefully code reader when I learn it:)

Spelled 'judgment', not 'judegement'; but rather than just fixing the
typo, it might read better as:

The lone caller of aio_context_setup() only checks whether an error was
set, but aio_context_setup() is not setting an error.


Signed-off-by: Cao jin <address@hidden>
  aio-posix.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/aio-posix.c b/aio-posix.c
index 6006122..8b0deb7 100644
--- a/aio-posix.c
+++ b/aio-posix.c
@@ -18,6 +18,7 @@
  #include "block/block.h"
  #include "qemu/queue.h"
  #include "qemu/sockets.h"
+#include "qapi/error.h"
  #include <sys/epoll.h>
@@ -491,6 +492,7 @@ void aio_context_setup(AioContext *ctx, Error **errp)
      ctx->epollfd = epoll_create1(EPOLL_CLOEXEC);
      if (ctx->epollfd == -1) {
+        error_setg_errno(errp, errno, "Failed to create epoll instance");
          ctx->epoll_available = false;
      } else {
          ctx->epoll_available = true;

However, I'm not even sure your patch is right.  The mere fact that we
have ctx->epoll_available makes it sound like we WANT to be able to
gracefully fall back when epoll is not available, rather than raising an
error message.

Yours Sincerely,

Cao jin

reply via email to

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