[sheepdog] [PATCH 1/4] sheep: priority to process EPOLLERR/EPOLLHUP event

Liu Yuan namei.unix at gmail.com
Thu Jun 28 06:24:08 CEST 2012


On 06/25/2012 03:24 PM, Liu Yuan wrote:
> On 06/23/2012 09:40 PM, Yunkai Zhang wrote:
>> When EPOLLERR/EPOLLHUP events occur,  other events(eg: EPOLLIN) may be
>> inclued at the same time, but we should priority to process them as the
>> connection was destroyed.
> 
> Applied this series.
> 

This whole series need revisiting:

1) cause performance degradation, because it forces concurrent requests
from QEMU to be serialized
2) Deadlock explained in patch 'sheep: take truns to monitor EPOLLIN and
EPOLLOUT events.' is questionable. I don't see a deadlock by code review.

Thanks
Yuan



More information about the sheepdog mailing list