This patch removes some unused arrays from the netfront
private data structure.
Also removes an unused address variable.
Signed-off-by: Vincenzo Maffione <[email protected]>
---
Hello,
I don't see the point in having those unused arrays in the private
structure.
Sorry if I'm missing something.
Cheers,
Vincenzo
drivers/net/xen-netfront.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index 158b5e6..1403ac7 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -137,10 +137,6 @@ struct netfront_info {
grant_ref_t gref_rx_head;
grant_ref_t grant_rx_ref[NET_RX_RING_SIZE];
- unsigned long rx_pfn_array[NET_RX_RING_SIZE];
- struct multicall_entry rx_mcl[NET_RX_RING_SIZE+1];
- struct mmu_update rx_mmu[NET_RX_RING_SIZE];
-
/* Statistics */
struct netfront_stats __percpu *stats;
@@ -252,7 +248,6 @@ static void xennet_alloc_rx_buffers(struct net_device *dev)
RING_IDX req_prod = np->rx.req_prod_pvt;
grant_ref_t ref;
unsigned long pfn;
- void *vaddr;
struct xen_netif_rx_request *req;
if (unlikely(!netif_carrier_ok(dev)))
@@ -322,7 +317,6 @@ no_skb:
np->grant_rx_ref[id] = ref;
pfn = page_to_pfn(skb_frag_page(&skb_shinfo(skb)->frags[0]));
- vaddr = page_address(skb_frag_page(&skb_shinfo(skb)->frags[0]));
req = RING_GET_REQUEST(&np->rx, req_prod + i);
gnttab_grant_foreign_access_ref(ref,
--
1.9.2
On 15/04/14 18:39, Vincenzo Maffione wrote:
> This patch removes some unused arrays from the netfront
> private data structure.
> Also removes an unused address variable.
>
> Signed-off-by: Vincenzo Maffione <[email protected]>
Huh. I wonder what they were for.
Acked-by: David Vrabel <[email protected]>
For net related patches please say whether they're for the net (current
RCs) or net-next branch. e.g.,
[PATCH net-next] xen-netfront: removed unused arrays.
Thanks.
David
2014-04-15 19:54 GMT+02:00 David Vrabel <[email protected]>:
> On 15/04/14 18:39, Vincenzo Maffione wrote:
>> This patch removes some unused arrays from the netfront
>> private data structure.
>> Also removes an unused address variable.
>>
>> Signed-off-by: Vincenzo Maffione <[email protected]>
>
> Huh. I wonder what they were for.
>
> Acked-by: David Vrabel <[email protected]>
>
> For net related patches please say whether they're for the net (current
> RCs) or net-next branch. e.g.,
>
> [PATCH net-next] xen-netfront: removed unused arrays.
>
Ok, thanks.
I used linux-next
git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
but I don't really know what is the most appropriate.
(I guess git://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git is "net"?)
Vincenzo
> Thanks.
>
> David
--
Vincenzo Maffione
On Tue, 2014-04-15 at 18:54 +0100, David Vrabel wrote:
> On 15/04/14 18:39, Vincenzo Maffione wrote:
> > This patch removes some unused arrays from the netfront
> > private data structure.
> > Also removes an unused address variable.
> >
> > Signed-off-by: Vincenzo Maffione <[email protected]>
>
> Huh. I wonder what they were for.
Looks like they were leftovers from the old flipping mode of operation.
Which was removed with:
commit 5dcddfae63bd26da0f11a8e40f0ae555b5ac624e
Author: Jeremy Fitzhardinge <[email protected]>
Date: Tue Aug 7 14:56:42 2007 -0700
xen-netfront: remove dead code
Which missed these bits.
Ian.
> Acked-by: David Vrabel <[email protected]>
>
> For net related patches please say whether they're for the net (current
> RCs) or net-next branch. e.g.,
>
> [PATCH net-next] xen-netfront: removed unused arrays.
>
> Thanks.
>
> David
>
> _______________________________________________
> Xen-devel mailing list
> [email protected]
> http://lists.xen.org/xen-devel