xen-netback: fix guest-receive-side array sizes
authorPaul Durrant <Paul.Durrant@citrix.com>
Mon, 23 Dec 2013 09:27:17 +0000 (09:27 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 30 Dec 2013 03:31:30 +0000 (22:31 -0500)
commitac3d5ac277352fe6e27809286768e9f1f8aa388d
tree1291e41ffe15d90e5e7f6b6d019b62b9d1bbdaa6
parent7a399e3a2e05bc580a78ea72371b3896827f72e1
xen-netback: fix guest-receive-side array sizes

The sizes chosen for the metadata and grant_copy_op arrays on the guest
receive size are wrong;

- The meta array is needlessly twice the ring size, when we only ever
  consume a single array element per RX ring slot
- The grant_copy_op array is way too small. It's sized based on a bogus
  assumption: that at most two copy ops will be used per ring slot. This
  may have been true at some point in the past but it's clear from looking
  at start_new_rx_buffer() that a new ring slot is only consumed if a frag
  would overflow the current slot (plus some other conditions) so the actual
  limit is MAX_SKB_FRAGS grant_copy_ops per ring slot.

This patch fixes those two sizing issues and, because grant_copy_ops grows
so much, it pulls it out into a separate chunk of vmalloc()ed memory.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: David Vrabel <david.vrabel@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/xen-netback/common.h
drivers/net/xen-netback/interface.c
drivers/net/xen-netback/netback.c