On Mon, Jun 3, 2024 at 7:48 AM Fea.Wang <fea.w...@sifive.com> wrote:
> Fix the transmission return size because not all bytes could be > transmitted successfully. So, return a successful length instead of a > constant value. > > How did you test this patch, on Linux or something else? I have some memory that we had some trouble with similar patches before. Anyway, the change looks good to me: Reviewed-by: Edgar E. Iglesias <edgar.igles...@amd.com> > Signed-off-by: Fea.Wang <fea.w...@sifive.com> > --- > hw/net/xilinx_axienet.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/net/xilinx_axienet.c b/hw/net/xilinx_axienet.c > index 7d1fd37b4a..05d41bd548 100644 > --- a/hw/net/xilinx_axienet.c > +++ b/hw/net/xilinx_axienet.c > @@ -847,7 +847,7 @@ static ssize_t eth_rx(NetClientState *nc, const > uint8_t *buf, size_t size) > axienet_eth_rx_notify(s); > > enet_update_irq(s); > - return size; > + return s->rxpos; > } > > static size_t > -- > 2.34.1 > >
