[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 07 Jul 2011 11:32:25 -0700
From: Shirley Ma <mashirle@...ibm.com>
To: David Miller <davem@...emloft.net>
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
On Thu, 2011-07-07 at 04:01 -0700, David Miller wrote:
> 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;
> }
> --
You are right, will fix it.
Thanks
Shirley
--
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