From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel De Graaf Subject: Re: [PATCH 0/3] XSM PIRQ and rcu_lock_target cleanup Date: Wed, 23 Jan 2013 09:37:54 -0500 Message-ID: <50FFF5C2.103@tycho.nsa.gov> References: <1358448953-2892-1-git-send-email-dgdegra@tycho.nsa.gov> <1358501009.13856.110.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1358501009.13856.110.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org On 01/18/2013 04:23 AM, Ian Campbell wrote: > On Thu, 2013-01-17 at 18:55 +0000, Daniel De Graaf wrote: >> These three patches finish up the XSM IS_PRIV reworking, in addition to >> the patches posted by Ian and Lars fixing things for ARM. I did not >> include a patch removing the rcu_lock_target_domain_by_id function >> because it's still referenced in ARM at the moment, but it looks like >> those locations are being changed (or will be changed in the future). > > Am I right that the appropriate transformation is: > rc = rcu_lock_target_domain_by_id(domid, &d); > becomes: > d = rcu_lock_domain_by_any_id(a.domid); > if ( d == NULL ) > return -ESRCH; > > rc = xsm_SOMETHING(XSM_TARGET, d, PARAMS); > if ( rc ) > ... unlock .. return or goto err > > Ian. > Yes, this is correct. Looking at today's xen-unstable, only the XENMAPSPACE_gmfn_foreign mapping needs a new ARM-only XSM hook. -- Daniel De Graaf National Security Agency