On 29-11-23, 13:42, Cornelia Huck wrote:
> On Wed, Nov 29 2023, Viresh Kumar <viresh.ku...@linaro.org> wrote:
>
> > On 29-11-23, 18:31, Haixu Cui wrote:
> >> Hi Viresh,
> >>
> >> Thank you for your helpful comments. In next patch, I will clearly
> >> point
> >> this out:
> >
> > Great, finally we are on the same page. Thanks Haixu.
> >
> >> "For full-duplex read and write transfer, both \field{tx_buf} and
> >> \field{rx_buf} are used and the buffer size of \field{tx_buf} must be same
> >> as \field{rx_buf}."
> >
> > Suggest rewriting as:
> >
> > In full-duplex transfer mode, both \field{tx_buf} and \field{rx_buf} are
> > sent by
> > the driver, \field{tx_buf} followed by \field{rx_buf}. The length of both
> > the
> > buffers MUST be same.
>
> Is that in a non-normative section? (Sorry, I've lost track here...)
I think yes, though I am not sure where Haixu will put it eventually :)
> If
> so, I would say:
>
> "The length of both buffers has to be the same."
That's better. Thanks.
> > device MUST fail the transfer and notify the driver.
>
> "notify the driver" == "set an appropriate error value"?
>
> Can this be covered by one of the existing error values?
I think yes, this can be handled by the Param error.
--
viresh
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org
