diff mbox

mcf_fec: fix error in qemu_send_packet argument

Message ID 1474554510-11973-1-git-send-email-pbonzini@redhat.com
State New
Headers show

Commit Message

Paolo Bonzini Sept. 22, 2016, 2:28 p.m. UTC
This uses the wrong frame size for packets composed of multiple
descriptors.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 hw/net/mcf_fec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jason Wang Sept. 23, 2016, 5:30 a.m. UTC | #1
On 2016年09月22日 22:28, Paolo Bonzini wrote:
> This uses the wrong frame size for packets composed of multiple
> descriptors.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   hw/net/mcf_fec.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/net/mcf_fec.c b/hw/net/mcf_fec.c
> index 0ee8ad9..5a5fc69 100644
> --- a/hw/net/mcf_fec.c
> +++ b/hw/net/mcf_fec.c
> @@ -176,7 +176,7 @@ static void mcf_fec_do_tx(mcf_fec_state *s)
>           if (bd.flags & FEC_BD_L) {
>               /* Last buffer in frame.  */
>               DPRINTF("Sending packet\n");
> -            qemu_send_packet(qemu_get_queue(s->nic), frame, len);
> +            qemu_send_packet(qemu_get_queue(s->nic), frame, frame_size);
>               ptr = frame;
>               frame_size = 0;
>               s->eir |= FEC_INT_TXF;

Applied, thanks.
diff mbox

Patch

diff --git a/hw/net/mcf_fec.c b/hw/net/mcf_fec.c
index 0ee8ad9..5a5fc69 100644
--- a/hw/net/mcf_fec.c
+++ b/hw/net/mcf_fec.c
@@ -176,7 +176,7 @@  static void mcf_fec_do_tx(mcf_fec_state *s)
         if (bd.flags & FEC_BD_L) {
             /* Last buffer in frame.  */
             DPRINTF("Sending packet\n");
-            qemu_send_packet(qemu_get_queue(s->nic), frame, len);
+            qemu_send_packet(qemu_get_queue(s->nic), frame, frame_size);
             ptr = frame;
             frame_size = 0;
             s->eir |= FEC_INT_TXF;