From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 1/6] xen-gntdev: Change page limit to be global instead of per-open Date: Fri, 14 Jan 2011 17:19:21 -0500 Message-ID: <20110114221921.GA9523@dumpdata.com> References: <1295019976-605-1-git-send-email-dgdegra@tycho.nsa.gov> <1295019976-605-2-git-send-email-dgdegra@tycho.nsa.gov> <20110114162000.GA12102@dumpdata.com> <4D30BBA2.4050703@tycho.nsa.gov> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <4D30BBA2.4050703@tycho.nsa.gov> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Daniel De Graaf Cc: jeremy@goop.org, xen-devel@lists.xensource.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org On Fri, Jan 14, 2011 at 04:09:54PM -0500, Daniel De Graaf wrote: > On 01/14/2011 11:20 AM, Konrad Rzeszutek Wilk wrote: > > On Fri, Jan 14, 2011 at 10:46:11AM -0500, Daniel De Graaf wrote: > >> Because there is no limitation on how many times a user can open a > >> given device file, an per-file-description limit on the number of > >> pages granted offers little to no benefit. Change to a global limit > >> and remove the ioctl() as the parameter can now be changed via sysfs. > > > > Can you add here the comment which c/s in the Xen hypervisor > > changed the tools so this won't cause them to fail. > > > > Just respond to this e-mail with the improved patch. > > > Latest xen-unstable.hg doesn't have the change; I recall Ian said > he would commit it sometime after -rc1. I'll post a modified patch > as soon as I see the changeset. Sweet. Thanks!