diff --git a/hypervisor/arch/x86/vtd.c b/hypervisor/arch/x86/vtd.c index 6d4642c12..4e072bfe8 100644 --- a/hypervisor/arch/x86/vtd.c +++ b/hypervisor/arch/x86/vtd.c @@ -128,6 +128,7 @@ struct dmar_drhd_rt { uint64_t root_table_addr; uint64_t ir_table_addr; + /* MAX_IR_ENTRIES is roundup (to power of 2) of CONFIG_MAX_PT_IRQ_ENTRIES. */ uint64_t irte_alloc_bitmap[MAX_IR_ENTRIES / 64U]; uint64_t irte_reserved_bitmap[MAX_IR_ENTRIES / 64U]; uint64_t qi_queue; diff --git a/hypervisor/include/arch/x86/asm/vtd.h b/hypervisor/include/arch/x86/asm/vtd.h index 4d3518286..284fc62db 100644 --- a/hypervisor/include/arch/x86/asm/vtd.h +++ b/hypervisor/include/arch/x86/asm/vtd.h @@ -42,13 +42,6 @@ #define DMAR_ICS_REG 0x9cU /* Invalidation complete status register */ #define DMAR_IRTA_REG 0xb8U /* Interrupt remapping table addr register */ -/* Make sure all PT IRQs work w/ interrupt remapping or post interrupt */ -#if (CONFIG_MAX_PT_IRQ_ENTRIES <= 256) -#define MAX_IR_ENTRIES 256 -#else -#define MAX_IR_ENTRIES powerof2_roundup(CONFIG_MAX_PT_IRQ_ENTRIES) -#endif - /* Values for entry_type in ACPI_DMAR_DEVICE_SCOPE - device types */ enum acpi_dmar_scope_type { ACPI_DMAR_SCOPE_TYPE_NOT_USED = 0, diff --git a/hypervisor/include/lib/util.h b/hypervisor/include/lib/util.h index ca3296556..d702ae253 100644 --- a/hypervisor/include/lib/util.h +++ b/hypervisor/include/lib/util.h @@ -25,27 +25,6 @@ /** Replaces 'x' by the string "x". */ #define STRINGIFY(x) #x -/* - * This algorithm get the round up 2^n - * - * n = input - 1; 0x1002 ----> 0x1001 - * n |= n >> 1; 0x1001 | 0x800 - * n |= n >> 2; 0x1801 | 0x600 - * n |= n >> 4; 0x1e01 | 0x1e0 - * n |= n >> 8; 0x1fe1 | 0x1f - * n |= n >> 16; 0x1fff - * n |= n >> 32; 0x1fff - * n += 1; 0x2000 - */ -#define ROUND0(x) ((x)-1) -#define ROUND1(x) (ROUND0(x) |(ROUND0(x)>>1)) -#define ROUND2(x) (ROUND1(x) |(ROUND1(x)>>2)) -#define ROUND4(x) (ROUND2(x) |(ROUND2(x)>>4)) -#define ROUND8(x) (ROUND4(x) |(ROUND4(x)>>8)) -#define ROUND16(x) (ROUND8(x) |(ROUND8(x)>>16)) -#define ROUND32(x) (ROUND16(x) |(ROUND16(x)>>32)) -#define powerof2_roundup(x) ((ROUND32(x) == (~0UL)) ? ~0UL : (ROUND32(x) +1)) - /* Macro used to check if a value is aligned to the required boundary. * Returns TRUE if aligned; FALSE if not aligned * NOTE: The required alignment must be a power of 2 (2, 4, 8, 16, 32, etc)