Thu, Mar 12, 2020 at 11:54:17PM CET, pavel.contrib(a)gmail.com wrote:
It is possible to read 0 bytes from timerfd descriptor
despite the fact that descriptor notified poll() that
it has data.
It is possible to see such behaviour on some hardware
platforms.
Solve this by treating such situation as normal.
Signed-off-by: Pavel Shirshov <pavel.contrib(a)gmail.com>
In the other email you have different address in the Signed-off-by line.
Make up your mind please.
Otherwise, the patch looks fine to me.
>---
> teamd/teamd.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/teamd/teamd.c b/teamd/teamd.c
>index e035ac5..8cdc16d 100644
>--- a/teamd/teamd.c
>+++ b/teamd/teamd.c
>@@ -265,6 +265,10 @@ static int handle_period_fd(int fd)
> teamd_log_err("read() failed.");
> return -errno;
> }
>+ if (ret == 0) {
>+ teamd_log_warn("read() for timer_fd returned 0.");
>+ return 0;
>+ }
> if (ret != sizeof(uint64_t)) {
> teamd_log_err("read() returned unexpected number of bytes.");
> return -EINVAL;
>--
>2.7.4
>