From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Chen, Tiejun" <tiejun.chen@intel.com>
Subject: Re: [PATCH] xen/domctl: lower loglevel of
 XEN_DOMCTL_memory_mapping
Date: Thu, 10 Sep 2015 16:55:24 +0800
Message-ID: <55F1457C.5000708@intel.com>
References: <1441781425-11553-1-git-send-email-tiejun.chen@intel.com>
	<20150909142018.GA28134@l.oracle.com>
	<55F05F7002000078000A15B9@prv-mh.provo.novell.com>
	<20150909145013.GH28134@l.oracle.com> <55F04E1A.6070202@citrix.com>
	<55F0700B02000078000A1658@prv-mh.provo.novell.com>
	<55F11500.10909@intel.com>
	<55F157D602000078000A18C4@prv-mh.provo.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <55F157D602000078000A18C4@prv-mh.provo.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>, Ian Campbell <ian.campbell@citrix.com>, Tim Deegan <tim@xen.org>, IanJackson <ian.jackson@eu.citrix.com>, xen-devel@lists.xen.org, Malcolm Crossley <malcolm.crossley@citrix.com>
List-Id: xen-devel@lists.xenproject.org

> Sort of (the patch has the intended effect, but for its size very
> many rough edges).
>

I guess we need to amend the original parameter, once_mapping_mfns, like 
this,

/* xen_once_mapping_mfns: memory mapping mfn bumbers once. */
unsigned int xen_once_mapping_mfns;
size_param("once_mapping_mfns", xen_once_mapping_mfns);

static void xen_once_mapping_mfns_setup(void)
{
     if ( once_mapping_mfns < 64 )
         xen_once_mapping_mfns = 64;
     else if ( once_mapping_mfns > 1024 )
         xen_once_mapping_mfns = 1024;
     else
         xen_once_mapping_mfns = once_mapping_mfns;
}

Or what is your expected rule?

Thanks
Tiejun