From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: [PATCH 68/75] hv: netvsc: convert to SKB paged frag API. Date: Fri, 19 Aug 2011 16:37:39 +0300 Message-ID: <20110819133739.GA3715@shale.localdomain> References: <1313760393.5010.356.camel@zakaz.uk.xensource.com> <1313760467-8598-68-git-send-email-ian.campbell@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: devel@driverdev.osuosl.org, Abhishek Kane , netdev@vger.kernel.org, Haiyang Zhang , Greg Kroah-Hartman , linux-kernel@vger.kernel.org To: Ian Campbell Return-path: Content-Disposition: inline In-Reply-To: <1313760467-8598-68-git-send-email-ian.campbell@citrix.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devel-bounces@linuxdriverproject.org Sender: devel-bounces@linuxdriverproject.org List-Id: netdev.vger.kernel.org On Fri, Aug 19, 2011 at 02:27:40PM +0100, Ian Campbell wrote: > /* usually there's just one frag */ > - skb_frag_set_page(skb, 0, gl->frags[0].page); > + skb_frag_set_page(skb, 0, gl->frags[0].page.p); /* XXX */ There are bunch of new /* XXX */ comments. What are they for? regards, dan carpenter