From mboxrd@z Thu Jan 1 00:00:00 1970 From: Feng Wu Subject: [v3 09/15] vt-d: Extend struct iremap_entry to support VT-d Posted-Interrupts Date: Wed, 24 Jun 2015 13:18:23 +0800 Message-ID: <1435123109-10481-10-git-send-email-feng.wu@intel.com> References: <1435123109-10481-1-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-1-git-send-email-feng.wu@intel.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: xen-devel@lists.xen.org Cc: kevin.tian@intel.com, keir@xen.org, george.dunlap@eu.citrix.com, andrew.cooper3@citrix.com, jbeulich@suse.com, yang.z.zhang@intel.com, feng.wu@intel.com List-Id: xen-devel@lists.xenproject.org Extend struct iremap_entry according to VT-d Posted-Interrupts Spec. Signed-off-by: Feng Wu --- v3: - Use u32 instead of u64 to define the bitfields in 'struct iremap_entry' - Limit using bitfield if possible xen/drivers/passthrough/vtd/intremap.c | 92 +++++++++++++++++----------------- xen/drivers/passthrough/vtd/iommu.h | 42 ++++++++++------ xen/drivers/passthrough/vtd/utils.c | 10 ++-- 3 files changed, 80 insertions(+), 64 deletions(-) diff --git a/xen/drivers/passthrough/vtd/intremap.c b/xen/drivers/passthrough/vtd/intremap.c index 0333686..b7a42f6 100644 --- a/xen/drivers/passthrough/vtd/intremap.c +++ b/xen/drivers/passthrough/vtd/intremap.c @@ -123,9 +123,9 @@ static u16 hpetid_to_bdf(unsigned int hpet_id) static void set_ire_sid(struct iremap_entry *ire, unsigned int svt, unsigned int sq, unsigned int sid) { - ire->hi.svt = svt; - ire->hi.sq = sq; - ire->hi.sid = sid; + ire->remap.svt = svt; + ire->remap.sq = sq; + ire->remap.sid = sid; } static void set_ioapic_source_id(int apic_id, struct iremap_entry *ire) @@ -220,7 +220,7 @@ static unsigned int alloc_remap_entry(struct iommu *iommu, unsigned int nr) else p = &iremap_entries[i % (1 << IREMAP_ENTRY_ORDER)]; - if ( p->lo_val || p->hi_val ) /* not a free entry */ + if ( p->lo || p->hi ) /* not a free entry */ found = 0; else if ( ++found == nr ) break; @@ -254,7 +254,7 @@ static int remap_entry_to_ioapic_rte( GET_IREMAP_ENTRY(ir_ctrl->iremap_maddr, index, iremap_entries, iremap_entry); - if ( iremap_entry->hi_val == 0 && iremap_entry->lo_val == 0 ) + if ( iremap_entry->hi == 0 && iremap_entry->lo == 0 ) { dprintk(XENLOG_ERR VTDPREFIX, "%s: index (%d) get an empty entry!\n", @@ -264,13 +264,13 @@ static int remap_entry_to_ioapic_rte( return -EFAULT; } - old_rte->vector = iremap_entry->lo.vector; - old_rte->delivery_mode = iremap_entry->lo.dlm; - old_rte->dest_mode = iremap_entry->lo.dm; - old_rte->trigger = iremap_entry->lo.tm; + old_rte->vector = iremap_entry->remap.vector; + old_rte->delivery_mode = iremap_entry->remap.dlm; + old_rte->dest_mode = iremap_entry->remap.dm; + old_rte->trigger = iremap_entry->remap.tm; old_rte->__reserved_2 = 0; old_rte->dest.logical.__reserved_1 = 0; - old_rte->dest.logical.logical_dest = iremap_entry->lo.dst >> 8; + old_rte->dest.logical.logical_dest = iremap_entry->remap.dst >> 8; unmap_vtd_domain_page(iremap_entries); spin_unlock_irqrestore(&ir_ctrl->iremap_lock, flags); @@ -318,27 +318,28 @@ static int ioapic_rte_to_remap_entry(struct iommu *iommu, if ( rte_upper ) { if ( x2apic_enabled ) - new_ire.lo.dst = value; + new_ire.remap.dst = value; else - new_ire.lo.dst = (value >> 24) << 8; + new_ire.remap.dst = (value >> 24) << 8; } else { *(((u32 *)&new_rte) + 0) = value; - new_ire.lo.fpd = 0; - new_ire.lo.dm = new_rte.dest_mode; - new_ire.lo.tm = new_rte.trigger; - new_ire.lo.dlm = new_rte.delivery_mode; + new_ire.remap.fpd = 0; + new_ire.remap.dm = new_rte.dest_mode; + new_ire.remap.tm = new_rte.trigger; + new_ire.remap.dlm = new_rte.delivery_mode; /* Hardware require RH = 1 for LPR delivery mode */ - new_ire.lo.rh = (new_ire.lo.dlm == dest_LowestPrio); - new_ire.lo.avail = 0; - new_ire.lo.res_1 = 0; - new_ire.lo.vector = new_rte.vector; - new_ire.lo.res_2 = 0; + new_ire.remap.rh = (new_ire.remap.dlm == dest_LowestPrio); + new_ire.remap.avail = 0; + new_ire.remap.res_1 = 0; + new_ire.remap.vector = new_rte.vector; + new_ire.remap.res_2 = 0; set_ioapic_source_id(IO_APIC_ID(apic), &new_ire); - new_ire.hi.res_1 = 0; - new_ire.lo.p = 1; /* finally, set present bit */ + new_ire.remap.res_3 = 0; + new_ire.remap.res_4 = 0; + new_ire.remap.p = 1; /* finally, set present bit */ /* now construct new ioapic rte entry */ remap_rte->vector = new_rte.vector; @@ -511,7 +512,7 @@ static int remap_entry_to_msi_msg( GET_IREMAP_ENTRY(ir_ctrl->iremap_maddr, index, iremap_entries, iremap_entry); - if ( iremap_entry->hi_val == 0 && iremap_entry->lo_val == 0 ) + if ( iremap_entry->hi == 0 && iremap_entry->lo == 0 ) { dprintk(XENLOG_ERR VTDPREFIX, "%s: index (%d) get an empty entry!\n", @@ -524,25 +525,25 @@ static int remap_entry_to_msi_msg( msg->address_hi = MSI_ADDR_BASE_HI; msg->address_lo = MSI_ADDR_BASE_LO | - ((iremap_entry->lo.dm == 0) ? + ((iremap_entry->remap.dm == 0) ? MSI_ADDR_DESTMODE_PHYS: MSI_ADDR_DESTMODE_LOGIC) | - ((iremap_entry->lo.dlm != dest_LowestPrio) ? + ((iremap_entry->remap.dlm != dest_LowestPrio) ? MSI_ADDR_REDIRECTION_CPU: MSI_ADDR_REDIRECTION_LOWPRI); if ( x2apic_enabled ) - msg->dest32 = iremap_entry->lo.dst; + msg->dest32 = iremap_entry->remap.dst; else - msg->dest32 = (iremap_entry->lo.dst >> 8) & 0xff; + msg->dest32 = (iremap_entry->remap.dst >> 8) & 0xff; msg->address_lo |= MSI_ADDR_DEST_ID(msg->dest32); msg->data = MSI_DATA_TRIGGER_EDGE | MSI_DATA_LEVEL_ASSERT | - ((iremap_entry->lo.dlm != dest_LowestPrio) ? + ((iremap_entry->remap.dlm != dest_LowestPrio) ? MSI_DATA_DELIVERY_FIXED: MSI_DATA_DELIVERY_LOWPRI) | - iremap_entry->lo.vector; + iremap_entry->remap.vector; unmap_vtd_domain_page(iremap_entries); spin_unlock_irqrestore(&ir_ctrl->iremap_lock, flags); @@ -601,29 +602,30 @@ static int msi_msg_to_remap_entry( memcpy(&new_ire, iremap_entry, sizeof(struct iremap_entry)); /* Set interrupt remapping table entry */ - new_ire.lo.fpd = 0; - new_ire.lo.dm = (msg->address_lo >> MSI_ADDR_DESTMODE_SHIFT) & 0x1; - new_ire.lo.tm = (msg->data >> MSI_DATA_TRIGGER_SHIFT) & 0x1; - new_ire.lo.dlm = (msg->data >> MSI_DATA_DELIVERY_MODE_SHIFT) & 0x1; + new_ire.remap.fpd = 0; + new_ire.remap.dm = (msg->address_lo >> MSI_ADDR_DESTMODE_SHIFT) & 0x1; + new_ire.remap.tm = (msg->data >> MSI_DATA_TRIGGER_SHIFT) & 0x1; + new_ire.remap.dlm = (msg->data >> MSI_DATA_DELIVERY_MODE_SHIFT) & 0x1; /* Hardware require RH = 1 for LPR delivery mode */ - new_ire.lo.rh = (new_ire.lo.dlm == dest_LowestPrio); - new_ire.lo.avail = 0; - new_ire.lo.res_1 = 0; - new_ire.lo.vector = (msg->data >> MSI_DATA_VECTOR_SHIFT) & - MSI_DATA_VECTOR_MASK; - new_ire.lo.res_2 = 0; + new_ire.remap.rh = (new_ire.remap.dlm == dest_LowestPrio); + new_ire.remap.avail = 0; + new_ire.remap.res_1 = 0; + new_ire.remap.vector = (msg->data >> MSI_DATA_VECTOR_SHIFT) & + MSI_DATA_VECTOR_MASK; + new_ire.remap.res_2 = 0; if ( x2apic_enabled ) - new_ire.lo.dst = msg->dest32; + new_ire.remap.dst = msg->dest32; else - new_ire.lo.dst = ((msg->address_lo >> MSI_ADDR_DEST_ID_SHIFT) - & 0xff) << 8; + new_ire.remap.dst = ((msg->address_lo >> MSI_ADDR_DEST_ID_SHIFT) + & 0xff) << 8; if ( pdev ) set_msi_source_id(pdev, &new_ire); else set_hpet_source_id(msi_desc->hpet_id, &new_ire); - new_ire.hi.res_1 = 0; - new_ire.lo.p = 1; /* finally, set present bit */ + new_ire.remap.res_3 = 0; + new_ire.remap.res_4 = 0; + new_ire.remap.p = 1; /* finally, set present bit */ /* now construct new MSI/MSI-X rte entry */ remap_rte = (struct msi_msg_remap_entry *)msg; diff --git a/xen/drivers/passthrough/vtd/iommu.h b/xen/drivers/passthrough/vtd/iommu.h index e807253..49daa70 100644 --- a/xen/drivers/passthrough/vtd/iommu.h +++ b/xen/drivers/passthrough/vtd/iommu.h @@ -289,29 +289,43 @@ struct dma_pte { /* interrupt remap entry */ struct iremap_entry { union { - u64 lo_val; + struct { u64 lo, hi; }; struct { - u64 p : 1, + u16 p : 1, fpd : 1, dm : 1, rh : 1, tm : 1, dlm : 3, avail : 4, - res_1 : 4, - vector : 8, - res_2 : 8, - dst : 32; - }lo; - }; - union { - u64 hi_val; + res_1 : 4; + u8 vector; + u8 res_2; + u32 dst; + u16 sid; + u16 sq : 2, + svt : 2, + res_3 : 12; + u32 res_4 : 32; + } remap; struct { - u64 sid : 16, - sq : 2, + u16 p : 1, + fpd : 1, + res_1 : 6, + avail : 4, + res_2 : 2, + urg : 1, + im : 1; + u8 vector; + u8 res_3; + u32 res_4 : 6, + pda_l : 26; + u16 sid; + u16 sq : 2, svt : 2, - res_1 : 44; - }hi; + res_5 : 12; + u32 pda_h; + } post; }; }; diff --git a/xen/drivers/passthrough/vtd/utils.c b/xen/drivers/passthrough/vtd/utils.c index bd14c02..a5fe237 100644 --- a/xen/drivers/passthrough/vtd/utils.c +++ b/xen/drivers/passthrough/vtd/utils.c @@ -238,14 +238,14 @@ static void dump_iommu_info(unsigned char key) else p = &iremap_entries[i % (1 << IREMAP_ENTRY_ORDER)]; - if ( !p->lo.p ) + if ( !p->remap.p ) continue; printk(" %04x: %x %x %04x %08x %02x %x %x %x %x %x" " %x %x\n", i, - (u32)p->hi.svt, (u32)p->hi.sq, (u32)p->hi.sid, - (u32)p->lo.dst, (u32)p->lo.vector, (u32)p->lo.avail, - (u32)p->lo.dlm, (u32)p->lo.tm, (u32)p->lo.rh, - (u32)p->lo.dm, (u32)p->lo.fpd, (u32)p->lo.p); + (u32)p->remap.svt, (u32)p->remap.sq, (u32)p->remap.sid, + (u32)p->remap.dst, (u32)p->remap.vector, (u32)p->remap.avail, + (u32)p->remap.dlm, (u32)p->remap.tm, (u32)p->remap.rh, + (u32)p->remap.dm, (u32)p->remap.fpd, (u32)p->remap.p); print_cnt++; } if ( iremap_entries ) -- 2.1.0