From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Tian, Kevin" Subject: Re: [v3 15/15] Add a command line parameter for VT-d posted-interrupts Date: Wed, 8 Jul 2015 11:25:46 +0000 Message-ID: References: <1435123109-10481-1-git-send-email-feng.wu@intel.com> <1435123109-10481-16-git-send-email-feng.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1435123109-10481-16-git-send-email-feng.wu@intel.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Wu, Feng" , "xen-devel@lists.xen.org" Cc: "Zhang, Yang Z" , "andrew.cooper3@citrix.com" , "keir@xen.org" , "george.dunlap@eu.citrix.com" , "jbeulich@suse.com" List-Id: xen-devel@lists.xenproject.org > From: Wu, Feng > Sent: Wednesday, June 24, 2015 1:18 PM > > Enable VT-d Posted-Interrupts and add a command line > parameter for it. > > Signed-off-by: Feng Wu > --- > v3: > Remove the redundant "no intremp then no intpost" logic > > docs/misc/xen-command-line.markdown | 9 ++++++++- > xen/drivers/passthrough/iommu.c | 4 +++- > 2 files changed, 11 insertions(+), 2 deletions(-) > > diff --git a/docs/misc/xen-command-line.markdown > b/docs/misc/xen-command-line.markdown > index aa684c0..f8ec15f 100644 > --- a/docs/misc/xen-command-line.markdown > +++ b/docs/misc/xen-command-line.markdown > @@ -875,6 +875,13 @@ debug hypervisor only). > >> Control the use of interrupt remapping (DMA remapping will always be enabled > >> if IOMMU functionality is enabled). > > +> `intpost` > + > +> Default: `true` > + > +>> Control the use of interrupt posting, interrupt posting is dependant on > +>> interrupt remapping. "Control the use of interrupt posting, which depends on the availability of interrupt remapping." > + > > `qinval` (VT-d) > > > Default: `true` > diff --git a/xen/drivers/passthrough/iommu.c b/xen/drivers/passthrough/iommu.c > index 597f676..e13251c 100644 > --- a/xen/drivers/passthrough/iommu.c > +++ b/xen/drivers/passthrough/iommu.c > @@ -52,7 +52,7 @@ bool_t __read_mostly iommu_passthrough; > bool_t __read_mostly iommu_snoop = 1; > bool_t __read_mostly iommu_qinval = 1; > bool_t __read_mostly iommu_intremap = 1; > -bool_t __read_mostly iommu_intpost; > +bool_t __read_mostly iommu_intpost = 1; > bool_t __read_mostly iommu_hap_pt_share = 1; > bool_t __read_mostly iommu_debug; > bool_t __read_mostly amd_iommu_perdev_intremap = 1; > @@ -97,6 +97,8 @@ static void __init parse_iommu_param(char *s) > iommu_qinval = val; > else if ( !strcmp(s, "intremap") ) > iommu_intremap = val; > + else if ( !strcmp(s, "intpost") ) > + iommu_intpost = val; > else if ( !strcmp(s, "debug") ) > { > iommu_debug = val; > -- > 2.1.0 Reviewed-by: Kevin Tian