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

Re: [XTF] xenbus: Don't wait if the response ring is full


  • To: "Wieczorkiewicz, Pawel" <wipawel@xxxxxxxxx>, Julien Grall <julien@xxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Fri, 10 Jul 2020 12:01:16 +0100
  • Authentication-results: esa5.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Grall, Julien" <jgrall@xxxxxxxxxxxx>
  • Delivery-date: Fri, 10 Jul 2020 11:01:29 +0000
  • Ironport-sdr: X5MtrZIjo/eiVV36SsriNXzfVv2nd5uvgr5eWfCiv93k/TR2/wGgs4btYZcRJLFVV2QXVmNgcR Fpgw88ROkfNvjRnh0xVLFfr+VtM2I0CjU/E9nkXAu0TYWhCth9EuV98Zu0MbhxlobixSBqy/aF E83VOhguMkd2FLdfEHcFd+2MHBk9A1fxOqEs1/K9FiHOfFtEOdHeBW0/kFZgESEj7NCwlfJEjv Cv6GbmKxfS3hsJqvyCQNhOkU71Rq0c3beyUQJbkGKRYjAzDgyYSwOyxmS7igmHF/KKrPuyov42 /tU=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 10/07/2020 08:53, Wieczorkiewicz, Pawel wrote:
>> On 9. Jul 2020, at 20:46, Julien Grall <julien@xxxxxxx> wrote:
>>
>> XenStore response can be bigger than the response ring. In this case,
>> it is possible to have the ring full (e.g cons = 19 and prod = 1043).
>>
>> However, XTF will consider that there is no data and therefore wait for
>> more input. This will result to block indefinitely as the ring is full.
>>
>> This can be solved by avoiding to mask the difference between prod and
>> cons.
>>
>> Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Pawel Wieczorkiewicz <wipawel@xxxxxxxxx>

Applied, thanks.

~Andrew



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.