Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752291AbbHTL1P (ORCPT ); Thu, 20 Aug 2015 07:27:15 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:22302 "EHLO SMTP02.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751427AbbHTL1O (ORCPT ); Thu, 20 Aug 2015 07:27:14 -0400 X-IronPort-AV: E=Sophos;i="5.15,714,1432598400"; d="scan'208";a="296268723" Message-ID: <55D5B98F.4060607@citrix.com> Date: Thu, 20 Aug 2015 12:27:11 +0100 From: David Vrabel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.8.0 MIME-Version: 1.0 To: Murilo Opsfelder Araujo , , CC: Konrad Rzeszutek Wilk , Boris Ostrovsky , Julien Grall , "Stefano Stabellini" Subject: Re: [PATCH] xen/tmem: Pass page instead of pfn to xen_tmem_get_page() References: <1439990719-19517-1-git-send-email-mopsfelder@gmail.com> In-Reply-To: <1439990719-19517-1-git-send-email-mopsfelder@gmail.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-DLP: MIA2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1054 Lines: 22 On 19/08/15 14:25, Murilo Opsfelder Araujo wrote: > The commit 091208a676dfdabb2b8fe86ee155c6fc80081b69 "xen/tmem: Use > xen_page_to_gfn rather than pfn_to_gfn" left behind a call to > xen_tmem_get_page() receiving pfn instead of page. > > This change also fixes the following build warning: > > drivers/xen/tmem.c: In function ‘tmem_cleancache_get_page’: > drivers/xen/tmem.c:194:47: warning: passing argument 4 of ‘xen_tmem_get_page’ makes pointer from integer without a cast > ret = xen_tmem_get_page((u32)pool, oid, ind, pfn); > ^ > drivers/xen/tmem.c:138:12: note: expected ‘struct page *’ but argument is of type ‘long unsigned int’ > static int xen_tmem_get_page(u32 pool_id, struct tmem_oid oid, I've folded this in, thanks. David -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/