[sheepdog] [PATCH] sheep: check EAGAIN for all the sd_lock helpers
Liu Yuan
namei.unix at gmail.com
Tue Dec 17 07:10:38 CET 2013
On Tue, Dec 17, 2013 at 02:35:27PM +0900, Hitoshi Mitake wrote:
> At Tue, 17 Dec 2013 13:08:24 +0800,
> Liu Yuan wrote:
> >
> > Some users report that:
> >
> > [root at pangkvm01 sheepdog]# cat sheep.log.bak |grep "Dec 16 23"
> > Dec 16 23:34:22 ERROR [main] modify_event(156) event info for fd 106 not
> > found
> > Dec 16 23:40:24 EMERG [io 5574] crash_handler(250) sheep exits unexpectedly
> > (Segmentation fault).
> > Dec 16 23:40:24 EMERG [io 7281] sd_write_lock(307) PANIC: failed to lock
> > for writing, Resource temporarily unavailable
> > Dec 16 23:40:24 EMERG [io 7281] crash_handler(250) sheep exits unexpectedly
> > (Aborted).
> >
> > It means some posix lock funcitions can return what is not documented in manual.
> > So it's better for us to defensively check it rather than panic out.
> >
> > Signed-off-by: Liu Yuan <namei.unix at gmail.com>
> > ---
> > include/util.h | 22 +++++++++++++++++++---
> > 1 file changed, 19 insertions(+), 3 deletions(-)
>
> Looks good to me.
> Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
>
Applied
Yuan
More information about the sheepdog
mailing list