[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 07 Jul 2011 04:01:49 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: mashirle@...ibm.com
Cc: mst@...hat.com, netdev@...r.kernel.org, kvm@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH V8 2/4 net-next] skbuff: skb supports zero-copy buffers
From: Shirley Ma <mashirle@...ibm.com>
Date: Wed, 06 Jul 2011 15:22:12 -0700
> + while (head) {
> + put_page(head);
> + head = (struct page *)head->private;
> + }
Looks like you might be referencing the page after it's
release here. I think you need something like:
while (head) {
struct page *next = (struct page *)head->private;
put_page(head);
head = next;
}
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists