From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel De Graaf Subject: Re: [PATCH 1/6] xen-gntdev: Change page limit to be global instead of per-open Date: Fri, 14 Jan 2011 16:09:54 -0500 Message-ID: <4D30BBA2.4050703@tycho.nsa.gov> 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> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20110114162000.GA12102@dumpdata.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Konrad Rzeszutek Wilk Cc: jeremy@goop.org, xen-devel@lists.xensource.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org 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.