Re: [PATCH] mini-os: xenbus: support large messages

2021-09-15 Thread Samuel Thibault
Juergen Gross, le mer. 15 sept. 2021 14:03:35 +0200, a ecrit: > On 15.09.21 13:20, Samuel Thibault wrote: > > So we really need to make the test after the rsp_cons update, like Linux > > does: > > > > > > Linux' code uses (intf->rsp_prod - cons >= XENSTORE_RING_SIZE), *after* > > > > the rsp_cons

Re: [PATCH] mini-os: xenbus: support large messages

2021-09-15 Thread Juergen Gross
On 15.09.21 13:20, Samuel Thibault wrote: Juergen Gross, le mer. 15 sept. 2021 12:48:44 +0200, a ecrit: On 15.09.21 01:17, Samuel Thibault wrote: +prod = xenstore_buf->rsp_prod; +DEBUG("Rsp_cons %d, rsp_prod %d.\n", xenstore_buf->rsp_cons, prod); +size = min(len - off, p

Re: [PATCH] mini-os: xenbus: support large messages

2021-09-15 Thread Samuel Thibault
Juergen Gross, le mer. 15 sept. 2021 12:48:44 +0200, a ecrit: > On 15.09.21 01:17, Samuel Thibault wrote: > > > +prod = xenstore_buf->rsp_prod; > > > +DEBUG("Rsp_cons %d, rsp_prod %d.\n", xenstore_buf->rsp_cons, > > > prod); > > > +size = min(len - off, prod - xenstore_buf-

Re: [PATCH] mini-os: xenbus: support large messages

2021-09-15 Thread Juergen Gross
On 15.09.21 01:17, Samuel Thibault wrote: Hello, Thanks for having worked on this! Juergen Gross, le mer. 18 août 2021 17:26:10 +0200, a ecrit: +static void xenbus_read_data(char *buf, unsigned int len) +{ +unsigned int off = 0; +unsigned int prod; +unsigned int size; +int noti

Re: [PATCH] mini-os: xenbus: support large messages

2021-09-14 Thread Samuel Thibault
Hello, Thanks for having worked on this! Juergen Gross, le mer. 18 août 2021 17:26:10 +0200, a ecrit: > +static void xenbus_read_data(char *buf, unsigned int len) > +{ > +unsigned int off = 0; > +unsigned int prod; > +unsigned int size; > +int notify; > + > +while (off != len)

Re: [PATCH] mini-os: xenbus: support large messages

2021-09-09 Thread Juergen Gross
On 18.08.21 17:26, Juergen Gross wrote: Today the implementation of the xenbus protocol in Mini-OS will only allow to transfer the complete message to or from the ring page buffer. This is limiting the maximum message size to lower values as the xenbus protocol normally would allow. Change that

[PATCH] mini-os: xenbus: support large messages

2021-08-18 Thread Juergen Gross
Today the implementation of the xenbus protocol in Mini-OS will only allow to transfer the complete message to or from the ring page buffer. This is limiting the maximum message size to lower values as the xenbus protocol normally would allow. Change that by allowing to transfer the xenbus message