[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 2/4] hw/i386: Introduce AMD IOMMU
From: |
David kiarie |
Subject: |
Re: [Qemu-devel] [RFC 2/4] hw/i386: Introduce AMD IOMMU |
Date: |
Wed, 9 Sep 2015 11:31:16 +0300 |
On Wed, Sep 9, 2015 at 11:14 AM, Valentine Sinitsyn
<address@hidden> wrote:
> Oops, that was a long ago. Sorry & thanks.
No problem.
Will fix all other comments.
>
> Valentine
>
>
> On 09.09.2015 13:12, David kiarie wrote:
>>
>> I wrote something basic on the cover letter
>> http://lists.nongnu.org/archive/html/qemu-devel/2015-08/msg02759.html
>>
>> On Wed, Sep 9, 2015 at 11:07 AM, Valentine Sinitsyn
>> <address@hidden> wrote:
>>>
>>> On 09.09.2015 12:59, David kiarie wrote:
>>>>
>>>>
>>>> On Wed, Sep 9, 2015 at 10:45 AM, Valentine Sinitsyn
>>>> <address@hidden> wrote:
>>>>>
>>>>>
>>>>> On 09.09.2015 12:30, David kiarie wrote:
>>>>> ...snip...
>>>>>
>>>>>
>>>>>>>> +static void amd_iommu_cmdbuf_exec(AMDIOMMUState *s)
>>>>>>>> +{
>>>>>>>> + unsigned type;
>>>>>>>> + uint8_t cmd[IOMMU_COMMAND_SIZE];
>>>>>>>> +
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "");
>>>>>>>> + memset(cmd, 0, IOMMU_COMMAND_SIZE);
>>>>>>>> +
>>>>>>>> + if(dma_memory_read(&address_space_memory, s->cmdbuf +
>>>>>>>> s->cmdbuf_head,
>>>>>>>> cmd, IOMMU_COMMAND_SIZE)){
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "error: fail to access memory at
>>>>>>>> 0x%"PRIx64
>>>>>>>> + " + %"PRIu8, s->cmdbuf, s->cmdbuf_head);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + type = cmd[CMDBUF_ID_BYTE] >> CMDBUF_ID_RSHIFT;
>>>>>>>> +
>>>>>>>> + switch(type){
>>>>>>>> + case CMD_COMPLETION_WAIT:
>>>>>>>> + /* pretend to wait for command execution to complete */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "completion wait requested");
>>>>>>>> + amd_iommu_completion_wait(s, cmd);
>>>>>>>> + break;
>>>>>>>> + case CMD_INVAL_DEVTAB_ENTRY:
>>>>>>>> + /* Not implemented yet - should just clear device table
>>>>>>>> contents */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "device table entries
>>>>>>>> invalidated");
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Better make it clear that these commands are not implemented yet. An
>>>>>>> IOMMU
>>>>>>> is hardly usable without invalidators.
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> I think the "debug prinft" here is misleading. AFAIK, the commands
>>>>>> listed here(there are others which I haven't listed) relate to 'cache'
>>>>>> which means this commands do nothing in the absence of cache.
>>>>>>
>>>>>> The reason why I advertise 'cache' is because the linux driver checks
>>>>>> for cache and other optional features to determine the IOMMU version
>>>>>> whereby versions 2 of IOMMU are assummed to support this features.
>>>>>
>>>>>
>>>>>
>>>>> Did you check how address translation is emulated in QEMU. I didn't :)
>>>>> But
>>>>> you return IOMMTLBEntry from your translation routine, and I'd rather
>>>>> ensure
>>>>> it is not cached in some other layer.
>>>>
>>>>
>>>>
>>>> Am expected to cache the IOMMUTLBEntries myself, probably through a
>>>> hashtable. Also, that's what emulated VT-d in Qemu does.
>>>>
>>>>>
>>>>> Besides, omitting invalidation commands will be a problem on real
>>>>> hardware.
>>>>> As IOMMU emulation is mainly useful for debugging, I'd reproduce this
>>>>> behaviour in the emulator. I mean, if I forget to do
>>>>> CMD_INVAL_DEVTAB_ENTRY
>>>>> in my guest code, things shouldn't work properly.
>>>>
>>>>
>>>>
>>>> Yes, 'cache' is important for debugging. I or someone else could work
>>>> on cache, now or later.
>>>>
>>>>>
>>>>> You can just copy all data structures to AMDIOMMUState and use this
>>>>> "cahced"
>>>>> copies, updated when you run invalidation commands. But this isn't a #1
>>>>> priority, you are right.
>>>>>
>>>>> Can you share some tests results for this code?
>>>>
>>>>
>>>>
>>>> Do you mean the whole IOMMU?It works, as far as am concerned. I got
>>>> was able to 'passthrough' a sound device but that's the only device I
>>>> tested.
>>>
>>>
>>> That's great. Could you describe your tests setup, please? Maybe in
>>> README
>>> or in the cover letter, so anyone interested could reproduce it just
>>> following your steps.
>>>
>>> Valentine
>>>
>>>
>>>>
>>>>>
>>>>>>
>>>>>>>
>>>>>>>> + break;
>>>>>>>> + case CMD_INVAL_IOMMU_PAGES:
>>>>>>>> + /* invalidate IOMMU cache for a particular domain */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "IOMMU pages invalidated");
>>>>>>>> + break;
>>>>>>>> + case CMD_INVAL_IOTLB_PAGES:
>>>>>>>> + /* Only present if remote IOTLB present */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "IOTLB pages invalidated");
>>>>>>>> + break;
>>>>>>>> + case CMD_INVAL_INTR_TABLE:
>>>>>>>> + /* no support for interrupt remapping yet */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "interrupt table invalidated");
>>>>>>>> + break;
>>>>>>>> + default:
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "unhandled command %d", type);
>>>>>>>> + break;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_cmdbuf_run(AMDIOMMUState *s)
>>>>>>>> +{
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "");
>>>>>>>> +
>>>>>>>> + uint64_t *mmio_cmdbuf_head = (uint64_t*)s->mmior +
>>>>>>>> MMIO_COMMAND_HEAD;
>>>>>>>> +
>>>>>>>> + if(!s->cmdbuf_enabled){
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "Command buffer not enabled");
>>>>>>>> + return;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + while(s->cmdbuf_head != s->cmdbuf_tail) {
>>>>>>>> + /* check if there is work to do. */
>>>>>>>> + IOMMU_DPRINTF(COMMAND, "COMMAND BUFFER head at %x COMMAND
>>>>>>>> BUFFER
>>>>>>>> tail at %x",
>>>>>>>> + s->cmdbuf_head, s->cmdbuf_tail);
>>>>>>>> + amd_iommu_cmdbuf_exec(s);
>>>>>>>> + s->cmdbuf_head += CMDBUF_ENTRY_SIZE;
>>>>>>>> +
>>>>>>>> + /* wrap head pointer */
>>>>>>>> + if (s->cmdbuf_head >= s->cmdbuf_len * CMDBUF_ENTRY_SIZE) {
>>>>>>>> + s->cmdbuf_head = 0;
>>>>>>>> + }
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + *mmio_cmdbuf_head = cpu_to_le64(s->cmdbuf_head);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* System Software might never read from some of this fields but
>>>>>>>> anyways
>>>>>>>> */
>>>>>>>> +static uint64_t amd_iommu_mmio_read(void *opaque, hwaddr addr,
>>>>>>>> unsigned
>>>>>>>> size)
>>>>>>>> +{
>>>>>>>> + AMDIOMMUState *s = opaque;
>>>>>>>> +
>>>>>>>> + uint64_t val = -1;
>>>>>>>> + if(addr + size > MMIO_SIZE) {
>>>>>>>> + IOMMU_DPRINTF(MMIO, "error: addr outside region: max
>>>>>>>> 0x%"PRIX64
>>>>>>>> + ", got 0x%"PRIx64 " %d",
>>>>>>>> + (uint64_t)MMIO_SIZE, addr, size);
>>>>>>>> + return (uint64_t) - 1;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + switch(addr & ~0x07){
>>>>>>>> + case MMIO_DEVICE_TABLE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_DEVICE_TABLE read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> What makes these cases different, except debug printf()s? Maybe leave
>>>>>>> printf()s inside the switch, and factor out the rest. This applies to
>>>>>>> other
>>>>>>> similar cases as well.
>>>>>>>
>>>>>>>
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_BASE read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_BASE read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_CONTROL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_CONTROL read addr 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EXCL_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EXCL_BASE read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EXCL_LIMIT:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EXCL_LIMIT read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_HEAD:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_HEAD read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_TAIL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_TAIL read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_HEAD:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_HEAD read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_TAIL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_TAIL read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_STATUS:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_STATUS read addr 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EXT_FEATURES:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EXT_FEATURES read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + val = amd_iommu_readw(s, addr);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + val = amd_iommu_readl(s, addr);
>>>>>>>> + }
>>>>>>>> + else if(size == 8){
>>>>>>>> + val = amd_iommu_readq(s, addr);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + default:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "UNHANDLED MMIO read addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d offset 0x%"PRIx64, addr, size,
>>>>>>>> offset);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + return val;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void iommu_handle_control_write(AMDIOMMUState *s)
>>>>>>>> +{
>>>>>>>> + /* read whatever is already written incase software is writing
>>>>>>>> in
>>>>>>>> chucks less than 8 bytes */
>>>>>>>> + unsigned long control = amd_iommu_readq(s, MMIO_CONTROL);
>>>>>>>> + s->enabled = !!(control & MMIO_CONTROL_IOMMUEN);
>>>>>>>> +
>>>>>>>> + s->ats_enabled = !!(control & MMIO_CONTROL_HTTUNEN);
>>>>>>>> + s->evtlog_enabled = s->enabled && !!(control &
>>>>>>>> MMIO_CONTROL_EVENTLOGEN);
>>>>>>>> +
>>>>>>>> + s->evtlog_intr = !!(control & MMIO_CONTROL_EVENTINTEN);
>>>>>>>> + s->completion_wait_intr = !!(control &
>>>>>>>> MMIO_CONTROL_COMWAITINTEN);
>>>>>>>> + s->cmdbuf_enabled = s->enabled && !!(control &
>>>>>>>> MMIO_CONTROL_CMDBUFLEN);
>>>>>>>> +
>>>>>>>> + /* update the flags depending on the control register */
>>>>>>>> + if(s->cmdbuf_enabled) {
>>>>>>>> + (*(uint64_t*)&s->mmior[MMIO_STATUS]) |=
>>>>>>>> MMIO_STATUS_CMDBUF_RUN;
>>>>>>>> + } else {
>>>>>>>> + (*(uint64_t*)&s->mmior[MMIO_STATUS]) &=
>>>>>>>> ~MMIO_STATUS_CMDBUF_RUN;
>>>>>>>> + }
>>>>>>>> + if (s->evtlog_enabled) {
>>>>>>>> + (*(uint64_t*)&s->mmior[MMIO_STATUS]) |=
>>>>>>>> MMIO_STATUS_EVTLOG_RUN;
>>>>>>>> + } else {
>>>>>>>> + (*(uint64_t*)&s->mmior[MMIO_STATUS]) &=
>>>>>>>> ~MMIO_STATUS_EVTLOG_RUN;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_STATUS state 0x%"PRIx64, control);
>>>>>>>> +
>>>>>>>> + amd_iommu_cmdbuf_run(s);
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Does any write to control register result in draining command buffer?
>>>>>>> To
>>>>>>> me,
>>>>>>> it looks strange, and I'd expect IOMMU to execute commands only if I
>>>>>>> change
>>>>>>> tail pointer.
>>>>>>>
>>>>>>>
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* FIXME: something might go wrong if System Software writes in
>>>>>>>> chunks
>>>>>>>> + * of one byte but linux writes in chunks of 4 bytes so currently
>>>>>>>> it
>>>>>>>> + * works correctly with linux but will definitely be busted if
>>>>>>>> software
>>>>>>>> + * reads/writes 8 bytes
>>>>>>>> + */
>>>>>>>> +static void amd_iommu_mmio_write(void *opaque, hwaddr addr,
>>>>>>>> uint64_t
>>>>>>>> val,
>>>>>>>> unsigned size)
>>>>>>>> +{
>>>>>>>> +
>>>>>>>> + AMDIOMMUState *s = opaque;
>>>>>>>> + unsigned long offset = addr & 0x07;
>>>>>>>> +
>>>>>>>> + if(addr + size > MMIO_SIZE) {
>>>>>>>> + IOMMU_DPRINTF(MMIO, "error: addr outside region: max
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", got 0x%"PRIx64 " %d",
>>>>>>>> + (uint64_t)MMIO_SIZE, addr, size);
>>>>>>>> + return;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + switch(addr & ~0x07){
>>>>>>>> + case MMIO_CONTROL:
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_CONTROL write addr 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64
>>>>>>>> , addr, size, val, offset);
>>>>>>>> + iommu_handle_control_write(s);
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_DEVICE_TABLE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_DEVICE_TABLE write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* set device table address
>>>>>>>> + * This also suffers from inability to tell whether
>>>>>>>> software
>>>>>>>> is done writing
>>>>>>>> + */
>>>>>>>> + if(offset || (size == 8)){
>>>>>>>> + unsigned long device_base = amd_iommu_readq(s,
>>>>>>>> MMIO_DEVICE_TABLE);
>>>>>>>> + s->devtab = (dma_addr_t)(device_base &
>>>>>>>> MMIO_CMDBUF_BASE_MASK);
>>>>>>>> + /* set device table length */
>>>>>>>> + s->devtab_len = ((device_base &
>>>>>>>> MMIO_DEVTAB_SIZE_MASK)
>>>>>>>> +
>>>>>>>> 1) * (MMIO_DEVTAB_SIZE_UNIT /
>>>>>>>> + MMIO_DEVTAB_ENTRY_SIZE);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_HEAD:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_HEAD write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + s->cmdbuf_head = val & MMIO_CMDBUF_HEAD_MASK;
>>>>>>>> + amd_iommu_cmdbuf_run(s);
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_BASE write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* FIXME - make sure System Software has finished
>>>>>>>> writing
>>>>>>>> incase it writes
>>>>>>>> + * in chucks less than 8 bytes in a robust way.As for
>>>>>>>> now,
>>>>>>>> this hacks works
>>>>>>>> + * for the linux driver
>>>>>>>> + */
>>>>>>>> + if(offset || (size == 8)){
>>>>>>>> + unsigned long command_base = amd_iommu_readq(s,
>>>>>>>> MMIO_COMMAND_BASE);
>>>>>>>> + s->cmdbuf = (dma_addr_t)(command_base &
>>>>>>>> MMIO_CMDBUF_BASE_MASK);
>>>>>>>> + s->cmdbuf_len = 1UL <<
>>>>>>>> (s->mmior[MMIO_CMDBUF_SIZE_BYTE]
>>>>>>>> &
>>>>>>>> MMIO_CMDBUF_SIZE_MASK);
>>>>>>>> + s->cmdbuf_head = s->cmdbuf_tail = 0;
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_COMMAND_TAIL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_COMMAND_TAIL write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + s->cmdbuf_tail = val & MMIO_CMDBUF_TAIL_MASK;
>>>>>>>> + amd_iommu_cmdbuf_run(s);
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_BASE write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + s->evtlog = (val & MMIO_EVTLOG_BASE_MASK);
>>>>>>>> + s->evtlog_len = 1UL <<
>>>>>>>> (*(uint64_t*)&s->mmior[MMIO_EVTLOG_SIZE_BYTE] &
>>>>>>>> MMIO_EVTLOG_SIZE_MASK);
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EXCL_LIMIT:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EXCL_LIMIT write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + s->excl_limit = (val & MMIO_EXCL_LIMIT_MASK) |
>>>>>>>> MMIO_EXCL_LIMIT_LOW;
>>>>>>>> + break;
>>>>>>>> + //TODO : Event handling fixes
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_HEAD:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_HEAD write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + case MMIO_EVENT_TAIL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_EVENT_TAIL write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> + /* PPR log base - unused for now */
>>>>>>>> + case MMIO_PPR_BASE:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_PPR_BASE write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> + /* PPR log head - also unused for now */
>>>>>>>> + case MMIO_PPR_HEAD:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_PPR_HEAD write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> + /* PPR log tail - unused for now */
>>>>>>>> + case MMIO_PPR_TAIL:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "MMIO_PPR_TAIL write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + if(size == 2){
>>>>>>>> + amd_iommu_writew(s, addr, val);
>>>>>>>> + } else if(size == 4){
>>>>>>>> + amd_iommu_writel(s, addr, val);
>>>>>>>> + } else if(size == 8){
>>>>>>>> + amd_iommu_writeq(s, addr, val);
>>>>>>>> + }
>>>>>>>> + break;
>>>>>>>> +
>>>>>>>> + default:
>>>>>>>> + IOMMU_DPRINTF(MMIO, "UNHANDLED MMIO write addr
>>>>>>>> 0x%"PRIx64
>>>>>>>> + ", size %d, val 0x%"PRIx64 ", offset
>>>>>>>> 0x%"PRIx64, addr, size, val, offset);
>>>>>>>> + ;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_log_event(AMDIOMMUState *s, AMDIOMMUEvent
>>>>>>>> *evt)
>>>>>>>> +{
>>>>>>>> + /* event logging not enabled */
>>>>>>>> + if(!s->evtlog_enabled || *(uint64_t*)&s->mmior[MMIO_STATUS] |
>>>>>>>> MMIO_STATUS_EVTLOG_OF){
>>>>>>>> + return;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* event log buffer full */
>>>>>>>> + if(s->evtlog_tail >= s->evtlog_len) {
>>>>>>>> + *(uint64_t*)&s->mmior[MMIO_STATUS] |=
>>>>>>>> MMIO_STATUS_EVTLOG_OF;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + if(dma_memory_write(&address_space_memory, s->evtlog_len +
>>>>>>>> s->evtlog_tail, &evt, EVENT_LEN)){
>>>>>>>> + //IOMMU_DPRINTF(ELOG, "error: fail to write at address
>>>>>>>> 0%x"PRIu8,
>>>>>>>> s->evtlog + s->evtlog_tail);
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + s->evtlog_tail += EVENT_LEN;
>>>>>>>> + *(uint64_t*)&s->mmior[MMIO_STATUS] |= MMIO_STATUS_EVTLOG_INTR;
>>>>>>>> + /* Disable logging if Overflow ? */
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> No, the spec details event logging restart procedure. Make sure
>>>>>>> emulated
>>>>>>> IOMMU behaves as described there.
>>>>>>>
>>>>>>>
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_page_fault(AMDIOMMUState *s, int devfn,
>>>>>>>> unsigned
>>>>>>>> domid, dma_addr_t addr, int present,
>>>>>>>> + int is_write)
>>>>>>>> +{
>>>>>>>> + IOMMU_DPRINTF(ELOG, "");
>>>>>>>> +
>>>>>>>> + AMDIOMMUEvent evt;
>>>>>>>> + unsigned info;
>>>>>>>> + uint16_t status;
>>>>>>>> +
>>>>>>>> + evt.devfn = cpu_to_le16(devfn);
>>>>>>>> + evt.reserved = 0;
>>>>>>>> + evt.domid = cpu_to_le16(domid);
>>>>>>>> + evt.addr = cpu_to_le16(addr);
>>>>>>>> +
>>>>>>>> + info = EVENT_IOPF;
>>>>>>>> +
>>>>>>>> + if(present) {
>>>>>>>> + info |= EVENT_IOPF_PR;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + if (is_write) {
>>>>>>>> + info |= EVENT_IOPF_RW;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* log a page fault */
>>>>>>>> + amd_iommu_log_event(s, &evt);
>>>>>>>> +
>>>>>>>> + /* Abort the translation */
>>>>>>>> + status = pci_get_word(s->dev.config + PCI_STATUS);
>>>>>>>> + pci_set_word(s->dev.config + PCI_STATUS, status |
>>>>>>>> PCI_STATUS_SIG_TARGET_ABORT);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static inline uint64_t amd_iommu_get_perms(uint64_t entry)
>>>>>>>> +{
>>>>>>>> + return (entry &(DEV_PERM_READ | DEV_PERM_WRITE)) >>
>>>>>>>> DEV_PERM_SHIFT;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* FIXME deduplication */
>>>>>>>> +AddressSpace *bridge_host_amd_iommu(PCIBus *bus, void *opaque, int
>>>>>>>> devfn)
>>>>>>>> +{
>>>>>>>> + AMDIOMMUState *s = opaque;
>>>>>>>> + AMDIOMMUAddressSpace **iommu_as;
>>>>>>>> + int bus_num = pci_bus_num(bus);
>>>>>>>> +
>>>>>>>> + /* just in case */
>>>>>>>> + assert(0 <= bus_num && bus_num <= PCI_BUS_MAX);
>>>>>>>> + assert(0 <= devfn && devfn <= PCI_DEVFN_MAX);
>>>>>>>> +
>>>>>>>> + iommu_as = s->address_spaces[bus_num];
>>>>>>>> +
>>>>>>>> + /* allocate memory during the first run */
>>>>>>>> + if(!iommu_as) {
>>>>>>>> + iommu_as =
>>>>>>>> g_malloc0(sizeof(AMDIOMMUAddressSpace*)*PCI_DEVFN_MAX);
>>>>>>>> + s->address_spaces[bus_num] = iommu_as;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* set up IOMMU region */
>>>>>>>> + if(!iommu_as[devfn]){
>>>>>>>> + iommu_as[devfn] = g_malloc0(sizeof(AMDIOMMUAddressSpace));
>>>>>>>> + iommu_as[devfn]->bus_num = (uint8_t)bus_num;
>>>>>>>> + iommu_as[devfn]->devfn = (uint8_t)devfn;
>>>>>>>> + iommu_as[devfn]->iommu_state = s;
>>>>>>>> +
>>>>>>>> + memory_region_init_iommu(&iommu_as[devfn]->iommu, OBJECT(s),
>>>>>>>> &s->iommu_ops, "amd-iommu", UINT64_MAX);
>>>>>>>> + address_space_init(&iommu_as[devfn]->as,
>>>>>>>> &iommu_as[devfn]->iommu,
>>>>>>>> "amd-iommu");
>>>>>>>> + }
>>>>>>>> + return &iommu_as[devfn]->as;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* validate a page table entry */
>>>>>>>> +static bool amd_iommu_validate_pte(uint64_t pte)
>>>>>>>> +{
>>>>>>>> + return pte & DEV_VALID && pte & DEV_TRANSLATION_VALID;
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> I'd suggest parenthesis here.
>>>>>>>
>>>>>>>
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* get a device table entry given the device function */
>>>>>>>> +static bool amd_iommu_get_dte(AMDIOMMUState *s, int devfn, uint64_t
>>>>>>>> *entry)
>>>>>>>> +{
>>>>>>>> + /* Could lead to out of range accesses */
>>>>>>>> + assert(0 < devfn && devfn < PCI_DEVFN_MAX);
>>>>>>>> +
>>>>>>>> + uint32_t offset = devfn * DEVTAB_ENTRY_SIZE;
>>>>>>>> + if(dma_memory_read(&address_space_memory, s->devtab + offset,
>>>>>>>> entry,
>>>>>>>> 32)){
>>>>>>>> + IOMMU_DPRINTF(MMU, "error: fail to access Device Entry
>>>>>>>> devtab
>>>>>>>> 0x%"PRIx64
>>>>>>>> + "offset 0x%"PRIx32, s->devtab, offset);
>>>>>>>> + return false;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + if(!amd_iommu_validate_pte(entry[0])){
>>>>>>>> + IOMMU_DPRINTF(MMU,
>>>>>>>> + "Pte entry at 0x%"PRIx64" is invalid",
>>>>>>>> entry[0]);
>>>>>>>> + return false;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + return true;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static uint64_t get_pte_translation_mode(uint64_t pte)
>>>>>>>> +{
>>>>>>>> + uint64_t level = (pte >> DEV_MODE_RSHIFT) & DEV_MODE_MASK;
>>>>>>>> +
>>>>>>>> + if(level == 7){
>>>>>>>> + /* reserved level - abort ? */
>>>>>>>> + IOMMU_DPRINTF(MMU,
>>>>>>>> + "Reserved translation level (7) detected");
>>>>>>>> + return 0;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + return level;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* TODO : Mark addresses as Accessed and Dirty */
>>>>>>>> +static void amd_iommu_do_translate(AMDIOMMUAddressSpace *as, hwaddr
>>>>>>>> addr,
>>>>>>>> bool is_write, IOMMUTLBEntry *ret)
>>>>>>>> +{
>>>>>>>> + AMDIOMMUState *s = as->iommu_state;
>>>>>>>> +
>>>>>>>> + int present;
>>>>>>>> + dma_addr_t pte_addr;
>>>>>>>> + uint64_t entry[4], pte, pte_perms;
>>>>>>>> + unsigned level;
>>>>>>>> + unsigned perms;
>>>>>>>> +
>>>>>>>> + if(!amd_iommu_get_dte(s, as->devfn, entry)){
>>>>>>>> + goto no_translation;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + pte = entry[0];
>>>>>>>> +
>>>>>>>> + /*
>>>>>>>> + * It's okay to check for either read or write permissions
>>>>>>>> + * even for memory maps, since we don't support R/W maps.
>>>>>>>> + */
>>>>>>>> + perms = is_write ? IOMMU_PERM_WRITE : IOMMU_PERM_READ;
>>>>>>>> +
>>>>>>>> + if(!(level = get_pte_translation_mode(pte))){
>>>>>>>> + goto no_translation;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + while(level > 0){
>>>>>>>> + /*
>>>>>>>> + * check permissions: the bitwise
>>>>>>>> + * implication perms -> entry_perms must be true. Pages
>>>>>>>> must
>>>>>>>> be
>>>>>>>> present
>>>>>>>> + * and permissions on all levels must be similar
>>>>>>>> + */
>>>>>>>> + pte_perms = amd_iommu_get_perms(pte);
>>>>>>>> + present = pte & 1;
>>>>>>>> + if(!present || perms != (perms & pte_perms)){
>>>>>>>> + amd_iommu_page_fault(s, as->devfn, entry[1] &
>>>>>>>> DEV_DOMID_ID_MASK, addr, present,
>>>>>>>> + !!(perms & IOMMU_PERM_WRITE));
>>>>>>>> + return;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + /* go to the next lower level */
>>>>>>>> + pte_addr = pte & DEV_PT_ROOT_MASK;
>>>>>>>> + pte_addr += ((addr >> ( 9 * level)) & 0xff8);
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Does this work for six level page tables? The highest level has
>>>>>>> different
>>>>>>> bit size there IIRC.
>>>>>>>
>>>>>>>> + pte = ldq_phys(&address_space_memory, pte_addr);
>>>>>>>> + level = (pte >> DEV_MODE_RSHIFT) & DEV_MODE_MASK;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + ret->iova = addr & IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret->translated_addr = (pte & DEV_PT_ROOT_MASK) &
>>>>>>>> IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret->addr_mask = ~IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret->perm = IOMMU_RW;
>>>>>>>> + return;
>>>>>>>> +
>>>>>>>> +no_translation:
>>>>>>>> + ret->iova = addr;
>>>>>>>> + ret->translated_addr = addr & IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret->addr_mask = ~IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret->perm = IOMMU_RW;
>>>>>>>> + return;
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Are you sure these transactions needs to be passed through rather
>>>>>>> than
>>>>>>> target-aborted?
>>>>>>>
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static IOMMUTLBEntry amd_iommu_translate(MemoryRegion *iommu,
>>>>>>>> hwaddr
>>>>>>>> addr, bool is_write)
>>>>>>>> +{
>>>>>>>> +
>>>>>>>> + IOMMU_DPRINTF(GENERAL, "");
>>>>>>>> +
>>>>>>>> + AMDIOMMUAddressSpace *as = container_of(iommu,
>>>>>>>> AMDIOMMUAddressSpace,
>>>>>>>> iommu);
>>>>>>>> + AMDIOMMUState *s = as->iommu_state;
>>>>>>>> +
>>>>>>>> + IOMMUTLBEntry ret = {
>>>>>>>> + .target_as = &address_space_memory,
>>>>>>>> + .iova = addr,
>>>>>>>> + .translated_addr = 0,
>>>>>>>> + .addr_mask = ~(hwaddr)0,
>>>>>>>> + .perm = IOMMU_NONE,
>>>>>>>> + };
>>>>>>>> +
>>>>>>>> + if(!s->enabled){
>>>>>>>> + /* IOMMU disabled - corresponds to iommu=off not failure to
>>>>>>>> provide any parameter */
>>>>>>>> + ret.iova = addr & IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret.translated_addr = addr & IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret.addr_mask = ~IOMMU_PAGE_MASK_4K;
>>>>>>>> + ret.perm = IOMMU_RW;
>>>>>>>> + return ret;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> + amd_iommu_do_translate(as, addr, is_write, &ret);
>>>>>>>> +
>>>>>>>> + IOMMU_DPRINTF(MMU,
>>>>>>>> + "devid: %02x:%02x.%x gpa 0x%"PRIx64 " hpa
>>>>>>>> 0x%"PRIx64,
>>>>>>>> as->bus_num,
>>>>>>>> + PCI_SLOT(as->devfn), PCI_FUNC(as->devfn), addr,
>>>>>>>> ret.translated_addr);
>>>>>>>> +
>>>>>>>> + return ret;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static const MemoryRegionOps mmio_mem_ops = {
>>>>>>>> + .read = amd_iommu_mmio_read,
>>>>>>>> + .write = amd_iommu_mmio_write,
>>>>>>>> + .endianness = DEVICE_LITTLE_ENDIAN,
>>>>>>>> + .valid = {
>>>>>>>> + .min_access_size = 1,
>>>>>>>> + .max_access_size = 8,
>>>>>>>> + .unaligned = false,
>>>>>>>> + },
>>>>>>>> +
>>>>>>>> + .valid = {
>>>>>>>> + .min_access_size = 8,
>>>>>>>> + .max_access_size = 8,
>>>>>>>> + },
>>>>>>>> +};
>>>>>>>> +
>>>>>>>> +static void amd_iommu_set_misc_capab(uint32_t host_va, uint32_t
>>>>>>>> host_pa,
>>>>>>>> uint32_t guest_va, uint32_t *reg)
>>>>>>>> +{
>>>>>>>> + *reg = 0;
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Redundant line.
>>>>>>>
>>>>>>>
>>>>>>>> +
>>>>>>>> + *reg |= MAX_VA_ADDR | MAX_PH_ADDR | MAX_GVA_ADDR;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* FIXME: The similar code in reset and init could be combined into
>>>>>>>> a
>>>>>>>> function */
>>>>>>>> +static void amd_iommu_reset(DeviceState *dev)
>>>>>>>> +{
>>>>>>>> + AMDIOMMUState *s = AMD_IOMMU_DEVICE(dev);
>>>>>>>> +
>>>>>>>> + uint32_t capab_header = CAPAB_FEATURES;
>>>>>>>> + uint32_t capab_base_addr_low = s->mmio.addr & ~(0xffff0000);
>>>>>>>> + uint32_t capab_base_addr_high = (s->mmio.addr & ~(0xffff)) >>
>>>>>>>> 16;
>>>>>>>> + uint32_t capab_range = 0xff000000;
>>>>>>>> + uint32_t capab_misc = 0x0;
>>>>>>>> +
>>>>>>>> + /* reset MMIO */
>>>>>>>> + memset(s->mmior, 0, MMIO_SIZE);
>>>>>>>> + amd_iommu_writeq(s, MMIO_EXT_FEATURES, EXT_FEATURES);
>>>>>>>> + /* TODO :Disable event logging and reset all buffers */
>>>>>>>> + amd_iommu_set_misc_capab(MAX_PH_ADDR, MAX_VA_ADDR,
>>>>>>>> MAX_GVA_ADDR,
>>>>>>>> &capab_misc);
>>>>>>>> +
>>>>>>>> + /* reset IOMMU specific capabilities */
>>>>>>>> + pci_set_long(s->dev.config + s->capab_offset, capab_header);
>>>>>>>> + pci_set_long(s->dev.config + s->capab_offset + CAPAB_BAR_LOW,
>>>>>>>> capab_base_addr_low);
>>>>>>>> + pci_set_long(s->dev.config + s->capab_offset + CAPAB_BAR_HIGH,
>>>>>>>> capab_base_addr_high);
>>>>>>>> + pci_set_long(s->dev.config + s->capab_offset + CAPAB_RANGE,
>>>>>>>> capab_range);
>>>>>>>> + pci_set_long(s->dev.config + s->capab_offset + CAPAB_MISC,
>>>>>>>> capab_misc);
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> Do we also need to reset MSI capability block?
>>>>>>>
>>>>>>>
>>>>>>>> +
>>>>>>>> + /* reset device ident */
>>>>>>>> + pci_config_set_vendor_id(s->dev.config, PCI_VENDOR_ID_AMD);
>>>>>>>> + pci_config_set_device_id(s->dev.config,
>>>>>>>> PCI_DEVICE_ID_RD890_IOMMU);
>>>>>>>> + pci_config_set_class(s->dev.config, 0x0806);
>>>>>>>> + pci_config_set_prog_interface(s->dev.config, 00);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_write_capab(PCIDevice *dev, uint32_t addr,
>>>>>>>> uint32_t
>>>>>>>> val, int len)
>>>>>>>> +{
>>>>>>>> + pci_default_write_config(dev, addr, val, len);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +/* I honestly don't know how to reserve MMIO */
>>>>>>>> +static void amd_iommu_mmio_map(AMDIOMMUState *s, hwaddr addr)
>>>>>>>> +{
>>>>>>>> + if(s->mmio.addr == addr){
>>>>>>>> + return;
>>>>>>>> + }
>>>>>>>> +
>>>>>>>> +// if(s->mmio.addr != (hwaddr)-1) {
>>>>>>>> +// memory_region_del_subregion(get_system_memory(),
>>>>>>>> &s->mmio);
>>>>>>>> +// }
>>>>>>>> +
>>>>>>>> + s->mmio.addr = addr;
>>>>>>>> + memory_region_add_subregion(get_system_memory(), addr,
>>>>>>>> &s->mmio);
>>>>>>>> +
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static int amd_iommu_pci_initfn(PCIDevice *dev)
>>>>>>>> +{
>>>>>>>> + AMDIOMMUState *s = container_of(dev, AMDIOMMUState, dev);
>>>>>>>> + uint32_t capab_header = CAPAB_FEATURES;
>>>>>>>> + uint32_t capab_base_addr_low = (s->mmio.addr & ~(0xffff0000));
>>>>>>>> + uint32_t capab_base_addr_high = (s->mmio.addr & ~(0x0000ffff)
>>>>>>>> >>
>>>>>>>> 16);
>>>>>>>> + uint32_t capab_range = 0xff000000;
>>>>>>>> + uint32_t capab_misc = 0x0;
>>>>>>>> +
>>>>>>>> + memset(s->mmior, 0, MMIO_SIZE);
>>>>>>>> + amd_iommu_set_misc_capab(MAX_PH_ADDR, MAX_VA_ADDR,
>>>>>>>> MAX_GVA_ADDR,
>>>>>>>> &capab_misc);
>>>>>>>> +
>>>>>>>> + s->iommu_ops.translate = amd_iommu_translate;
>>>>>>>> + s->enabled = false;
>>>>>>>> + s->ats_enabled = false;
>>>>>>>> + s->cmdbuf_enabled = false;
>>>>>>>> + s->devtab_len = 0;
>>>>>>>> + s->cmdbuf_len = 0;
>>>>>>>> + s->cmdbuf_head = 0;
>>>>>>>> + s->cmdbuf_tail = 0;
>>>>>>>> + s->evtlog_head = 0;
>>>>>>>> + s->evtlog_tail = 0;
>>>>>>>> + s->excl_enabled = false;
>>>>>>>> + s->excl_allow = false;
>>>>>>>> + s->mmio_enabled = false;
>>>>>>>> + dev->config_write = amd_iommu_write_capab;
>>>>>>>> +
>>>>>>>> + size_t offset;
>>>>>>>> + /* Save offset for IVRS */
>>>>>>>> + s->capab_offset = offset = pci_add_capability(dev,
>>>>>>>> PCI_CAP_ID_SEC,
>>>>>>>> 0,
>>>>>>>> CAPAB_SIZE);
>>>>>>>> + pci_set_long(s->dev.config + offset, capab_header);
>>>>>>>> + pci_set_long(s->dev.config + offset + CAPAB_BAR_LOW,
>>>>>>>> capab_base_addr_low);
>>>>>>>> + pci_set_long(s->dev.config + offset + CAPAB_BAR_HIGH,
>>>>>>>> capab_base_addr_high);
>>>>>>>> + pci_set_long(s->dev.config + offset + CAPAB_RANGE,
>>>>>>>> capab_range);
>>>>>>>> + pci_set_long(s->dev.config + offset + CAPAB_MISC, capab_misc);
>>>>>>>> +
>>>>>>>> + /* add msi and hypertransport capabilities */
>>>>>>>> + pci_add_capability(dev, PCI_CAP_ID_MSI, 0, CAPAB_REG_SIZE);
>>>>>>>> + pci_add_capability(dev, PCI_CAP_ID_HT, 0, CAPAB_REG_SIZE);
>>>>>>>> +
>>>>>>>> + /* set up MMIO */
>>>>>>>> + memory_region_init_io(&s->mmio, OBJECT(s), &mmio_mem_ops, s,
>>>>>>>> "amd-iommu", MMIO_SIZE);
>>>>>>>> + amd_iommu_mmio_map(s, BUS_AMD_IOMMU_ADDR);
>>>>>>>> +
>>>>>>>> + /* reset device ident */
>>>>>>>> + pci_config_set_vendor_id(dev->config, PCI_VENDOR_ID_AMD);
>>>>>>>> + pci_config_set_device_id(dev->config,
>>>>>>>> PCI_DEVICE_ID_RD890_IOMMU);
>>>>>>>> +
>>>>>>>> + /* may not be necessary */
>>>>>>>> + pci_config_set_interrupt_pin(dev->config, 0x1a);
>>>>>>>> +
>>>>>>>> + return 0;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_realize(PCIDevice *dev, Error **error)
>>>>>>>> +{
>>>>>>>> + amd_iommu_pci_initfn(dev);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static const VMStateDescription vmstate_amd_iommu = {
>>>>>>>> + .name = "amd-iommu",
>>>>>>>> + .fields = (VMStateField[]) {
>>>>>>>> + VMSTATE_PCI_DEVICE(dev, AMDIOMMUState),
>>>>>>>> + VMSTATE_END_OF_LIST()
>>>>>>>> + }
>>>>>>>> +};
>>>>>>>> +
>>>>>>>> +static Property amd_iommu_properties[] = {
>>>>>>>> + DEFINE_PROP_UINT32("version", AMDIOMMUState, version, 2),
>>>>>>>> + DEFINE_PROP_END_OF_LIST(),
>>>>>>>> +};
>>>>>>>> +
>>>>>>>> +/* not used for now but will be when cache is added */
>>>>>>>> +static void amd_iommu_uninit(PCIDevice *dev)
>>>>>>>> +{
>>>>>>>> + return;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static void amd_iommu_class_init(ObjectClass *klass, void* data)
>>>>>>>> +{
>>>>>>>> + DeviceClass *dc = DEVICE_CLASS(klass);
>>>>>>>> + PCIDeviceClass *pci = PCI_DEVICE_CLASS(klass);
>>>>>>>> +
>>>>>>>> + pci->realize = amd_iommu_realize;
>>>>>>>> + pci->exit = amd_iommu_uninit;
>>>>>>>> +
>>>>>>>> + dc->reset = amd_iommu_reset;
>>>>>>>> + dc->vmsd = &vmstate_amd_iommu;
>>>>>>>> + dc->props = amd_iommu_properties;
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +static const TypeInfo amd_iommu = {
>>>>>>>> + .name = TYPE_AMD_IOMMU_DEVICE,
>>>>>>>> + .parent = TYPE_PCI_DEVICE,
>>>>>>>> + .instance_size = sizeof(AMDIOMMUState),
>>>>>>>> + .class_init = amd_iommu_class_init
>>>>>>>> +};
>>>>>>>> +
>>>>>>>> +static void amd_iommu_register_types(void)
>>>>>>>> +{
>>>>>>>> + type_register_static(&amd_iommu);
>>>>>>>> +}
>>>>>>>> +
>>>>>>>> +type_init(amd_iommu_register_types);
>>>>>>>> diff --git a/hw/i386/amd_iommu.h b/hw/i386/amd_iommu.h
>>>>>>>> new file mode 100644
>>>>>>>> index 0000000..90309eb
>>>>>>>> --- /dev/null
>>>>>>>> +++ b/hw/i386/amd_iommu.h
>>>>>>>> @@ -0,0 +1,298 @@
>>>>>>>> +/*
>>>>>>>> + * QEMU emulation of an AMD IOMMU (AMD-Vi)
>>>>>>>> + *
>>>>>>>> + * Copyright (C) 2011 Eduard-Gabriel Mateanu
>>>>>>>> + * Copyright (C) 2015 David Kiarie, <address@hidden>
>>>>>>>> + *
>>>>>>>> + * This program is free software; you can redistribute it and/or
>>>>>>>> modify
>>>>>>>> + * it under the terms of the GNU General Public License as
>>>>>>>> published
>>>>>>>> by
>>>>>>>> + * the Free Software Foundation; either version 2 of the License,
>>>>>>>> or
>>>>>>>> + * (at your option) any later version.
>>>>>>>> +
>>>>>>>> + * This program is distributed in the hope that it will be useful,
>>>>>>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>>>>>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>>>>>>> + * GNU General Public License for more details.
>>>>>>>> +
>>>>>>>> + * You should have received a copy of the GNU General Public
>>>>>>>> License
>>>>>>>> along
>>>>>>>> + * with this program; if not, see <http://www.gnu.org/licenses/>.
>>>>>>>> + */
>>>>>>>> +
>>>>>>>> +#ifndef AMD_IOMMU_H_
>>>>>>>> +#define AMD_IOMMU_H_
>>>>>>>> +
>>>>>>>> +#include "hw/hw.h"
>>>>>>>> +#include "hw/pci/pci.h"
>>>>>>>> +#include "hw/sysbus.h"
>>>>>>>> +#include "sysemu/dma.h"
>>>>>>>> +#include "qemu/osdep.h"
>>>>>>>> +#include "qemu/event_notifier.h"
>>>>>>>> +
>>>>>>>> +/* Capability registers */
>>>>>>>> +#define CAPAB_HEADER 0x00
>>>>>>>> +#define CAPAB_REV_TYPE 0x02
>>>>>>>> +#define CAPAB_FLAGS 0x03
>>>>>>>> +#define CAPAB_BAR_LOW 0x04
>>>>>>>> +#define CAPAB_BAR_HIGH 0x08
>>>>>>>> +#define CAPAB_RANGE 0x0C
>>>>>>>> +#define CAPAB_MISC 0x10
>>>>>>>> +#define CAPAB_MISC1 0x14
>>>>>>>> +
>>>>>>>> +#define CAPAB_SIZE 0x18
>>>>>>>> +#define CAPAB_REG_SIZE 0x04
>>>>>>>> +
>>>>>>>> +/* Capability header data */
>>>>>>>> +#define CAPAB_FLAT_EXT (1 << 28)
>>>>>>>> +#define CAPAB_EFR_SUP (1 << 27)
>>>>>>>> +#define CAPAB_FLAG_NPCACHE (1 << 26)
>>>>>>>> +#define CAPAB_FLAG_HTTUNNEL (1 << 25)
>>>>>>>> +#define CAPAB_FLAG_IOTLBSUP (1 << 24)
>>>>>>>> +#define CAPAB_INIT_REV (1 << 19)
>>>>>>>> +#define CAPAB_INIT_TYPE (3 << 16)
>>>>>>>> +#define CAPAB_INIT_REV_TYPE (CAPAB_REV | CAPAB_TYPE)
>>>>>>>> +#define CAPAB_INIT_FLAGS (CAPAB_FLAG_NPCACHE |
>>>>>>>> CAPAB_FLAG_HTTUNNEL)
>>>>>>>> +#define CAPAB_INIT_MISC ((64 << 15) | (48 << 8))
>>>>>>>> +#define CAPAB_BAR_MASK (~((1UL << 14) - 1))
>>>>>>>> +
>>>>>>>> +/* MMIO registers */
>>>>>>>> +#define MMIO_DEVICE_TABLE 0x0000
>>>>>>>> +#define MMIO_COMMAND_BASE 0x0008
>>>>>>>> +#define MMIO_EVENT_BASE 0x0010
>>>>>>>> +#define MMIO_CONTROL 0x0018
>>>>>>>> +#define MMIO_EXCL_BASE 0x0020
>>>>>>>> +#define MMIO_EXCL_LIMIT 0x0028
>>>>>>>> +#define MMIO_EXT_FEATURES 0x0030
>>>>>>>> +#define MMIO_COMMAND_HEAD 0x2000
>>>>>>>> +#define MMIO_COMMAND_TAIL 0x2008
>>>>>>>> +#define MMIO_EVENT_HEAD 0x2010
>>>>>>>> +#define MMIO_EVENT_TAIL 0x2018
>>>>>>>> +#define MMIO_STATUS 0x2020
>>>>>>>> +#define MMIO_PPR_BASE 0x0038
>>>>>>>> +#define MMIO_PPR_HEAD 0x2030
>>>>>>>> +#define MMIO_PPR_TAIL 0x2038
>>>>>>>> +
>>>>>>>> +#define MMIO_SIZE 0x4000
>>>>>>>> +
>>>>>>>> +#define MMIO_DEVTAB_SIZE_MASK ((1ULL << 12) - 1)
>>>>>>>> +#define MMIO_DEVTAB_BASE_MASK (((1ULL << 52) - 1) &
>>>>>>>> ~MMIO_DEVTAB_SIZE_MASK)
>>>>>>>> +#define MMIO_DEVTAB_ENTRY_SIZE 32
>>>>>>>> +#define MMIO_DEVTAB_SIZE_UNIT 4096
>>>>>>>> +
>>>>>>>> +#define MMIO_CMDBUF_SIZE_BYTE (MMIO_COMMAND_BASE + 7)
>>>>>>>> +#define MMIO_CMDBUF_SIZE_MASK 0x0F
>>>>>>>> +#define MMIO_CMDBUF_BASE_MASK MMIO_DEVTAB_BASE_MASK
>>>>>>>> +#define MMIO_CMDBUF_DEFAULT_SIZE 8
>>>>>>>> +#define MMIO_CMDBUF_HEAD_MASK (((1ULL << 19) - 1) & ~0x0F)
>>>>>>>> +#define MMIO_CMDBUF_TAIL_MASK MMIO_EVTLOG_HEAD_MASK
>>>>>>>> +
>>>>>>>> +#define MMIO_EVTLOG_SIZE_BYTE (MMIO_EVENT_BASE + 7)
>>>>>>>> +#define MMIO_EVTLOG_SIZE_MASK MMIO_CMDBUF_SIZE_MASK
>>>>>>>> +#define MMIO_EVTLOG_BASE_MASK MMIO_CMDBUF_BASE_MASK
>>>>>>>> +#define MMIO_EVTLOG_DEFAULT_SIZE MMIO_CMDBUF_DEFAULT_SIZE
>>>>>>>> +#define MMIO_EVTLOG_HEAD_MASK (((1ULL << 19) - 1) & ~0x0F)
>>>>>>>> +#define MMIO_EVTLOG_TAIL_MASK MMIO_EVTLOG_HEAD_MASK
>>>>>>>> +
>>>>>>>> +#define MMIO_EXCL_BASE_MASK MMIO_DEVTAB_BASE_MASK
>>>>>>>> +#define MMIO_EXCL_ENABLED_MASK (1ULL << 0)
>>>>>>>> +#define MMIO_EXCL_ALLOW_MASK (1ULL << 1)
>>>>>>>> +#define MMIO_EXCL_LIMIT_MASK MMIO_DEVTAB_BASE_MASK
>>>>>>>> +#define MMIO_EXCL_LIMIT_LOW 0xFFF
>>>>>>>> +
>>>>>>>> +#define MMIO_CONTROL_IOMMUEN (1ULL << 0)
>>>>>>>> +#define MMIO_CONTROL_HTTUNEN (1ULL << 1)
>>>>>>>> +#define MMIO_CONTROL_EVENTLOGEN (1ULL << 2)
>>>>>>>> +#define MMIO_CONTROL_EVENTINTEN (1ULL << 3)
>>>>>>>> +#define MMIO_CONTROL_COMWAITINTEN (1ULL << 4)
>>>>>>>> +#define MMIO_CONTROL_CMDBUFLEN (1ULL << 12)
>>>>>>>> +
>>>>>>>> +#define MMIO_STATUS_EVTLOG_OF (1ULL << 0)
>>>>>>>> +#define MMIO_STATUS_EVTLOG_INTR (1ULL << 1)
>>>>>>>> +#define MMIO_STATUS_COMWAIT_INTR (1ULL << 2)
>>>>>>>> +#define MMIO_STATUS_EVTLOG_RUN (1ULL << 3)
>>>>>>>> +#define MMIO_STATUS_CMDBUF_RUN (1ULL << 4)
>>>>>>>> +
>>>>>>>> +#define CMDBUF_ID_BYTE 0x07
>>>>>>>> +#define CMDBUF_ID_RSHIFT 4
>>>>>>>> +#define CMDBUF_ENTRY_SIZE 0x10
>>>>>>>> +
>>>>>>>> +#define CMD_COMPLETION_WAIT 0x01
>>>>>>>> +#define CMD_INVAL_DEVTAB_ENTRY 0x02
>>>>>>>> +#define CMD_INVAL_IOMMU_PAGES 0x03
>>>>>>>> +#define CMD_INVAL_IOTLB_PAGES 0x04
>>>>>>>> +#define CMD_INVAL_INTR_TABLE 0x05
>>>>>>>> +
>>>>>>>> +#define DEVTAB_ENTRY_SIZE 32
>>>>>>>> +
>>>>>>>> +/* Device table entry bits 0:63 */
>>>>>>>> +#define DEV_VALID (1ULL << 0)
>>>>>>>> +#define DEV_TRANSLATION_VALID (1ULL << 1)
>>>>>>>> +#define DEV_MODE_MASK 0x7
>>>>>>>> +#define DEV_MODE_RSHIFT 9
>>>>>>>> +#define DEV_PT_ROOT_MASK 0xFFFFFFFFFF000
>>>>>>>> +#define DEV_PT_ROOT_RSHIFT 12
>>>>>>>> +#define DEV_PERM_SHIFT 61
>>>>>>>> +#define DEV_PERM_READ (1ULL << 61)
>>>>>>>> +#define DEV_PERM_WRITE (1ULL << 62)
>>>>>>>> +
>>>>>>>> +/* Device table entry bits 64:127 */
>>>>>>>> +#define DEV_DOMID_ID_MASK ((1ULL << 16) - 1)
>>>>>>>> +#define DEV_IOTLB_SUPPORT (1ULL << 17)
>>>>>>>> +#define DEV_SUPPRESS_PF (1ULL << 18)
>>>>>>>> +#define DEV_SUPPRESS_ALL_PF (1ULL << 19)
>>>>>>>> +#define DEV_IOCTL_MASK (~3)
>>>>>>>> +#define DEV_IOCTL_RSHIFT 20
>>>>>>>> +#define DEV_IOCTL_DENY 0
>>>>>>>> +#define DEV_IOCTL_PASSTHROUGH 1
>>>>>>>> +#define DEV_IOCTL_TRANSLATE 2
>>>>>>>> +#define DEV_CACHE (1ULL << 37)
>>>>>>>> +#define DEV_SNOOP_DISABLE (1ULL << 38)
>>>>>>>> +#define DEV_EXCL (1ULL << 39)
>>>>>>>> +
>>>>>>>> +/* Event codes and flags, as stored in the info field */
>>>>>>>> +#define EVENT_ILLEGAL_DEVTAB_ENTRY (0x1U << 24)
>>>>>>>> +#define EVENT_IOPF (0x2U << 24)
>>>>>>>> +#define EVENT_IOPF_I (1U << 3)
>>>>>>>> +#define EVENT_IOPF_PR (1U << 4)
>>>>>>>> +#define EVENT_IOPF_RW (1U << 5)
>>>>>>>> +#define EVENT_IOPF_PE (1U << 6)
>>>>>>>> +#define EVENT_IOPF_RZ (1U << 7)
>>>>>>>> +#define EVENT_IOPF_TR (1U << 8)
>>>>>>>> +#define EVENT_DEV_TAB_HW_ERROR (0x3U << 24)
>>>>>>>> +#define EVENT_PAGE_TAB_HW_ERROR (0x4U << 24)
>>>>>>>> +#define EVENT_ILLEGAL_COMMAND_ERROR (0x5U << 24)
>>>>>>>> +#define EVENT_COMMAND_HW_ERROR (0x6U << 24)
>>>>>>>> +#define EVENT_IOTLB_INV_TIMEOUT (0x7U << 24)
>>>>>>>> +#define EVENT_INVALID_DEV_REQUEST (0x8U << 24)
>>>>>>>> +
>>>>>>>> +#define EVENT_LEN 16
>>>>>>>> +
>>>>>>>> +#define IOMMU_PERM_READ (1 << 0)
>>>>>>>> +#define IOMMU_PERM_WRITE (1 << 1)
>>>>>>>> +#define IOMMU_PERM_RW (IOMMU_PERM_READ |
>>>>>>>> IOMMU_PERM_WRITE)
>>>>>>>> +
>>>>>>>> +/* AMD RD890 Chipset */
>>>>>>>> +#define PCI_DEVICE_ID_RD890_IOMMU 0x20
>>>>>>>> +
>>>>>>>> +#define PCI_CAP_ID_SEC 0xf /* IOMMU capability
>>>>>>>> header
>>>>>>>> register */
>>>>>>>> +#define PCI_CAP_ID_MMIO_LOW 0x0 /* MMIO base address low
>>>>>>>> register */
>>>>>>>> +#define PCI_CAP_ID_MMIO_HIGH 0x0 /* MMIO base address high
>>>>>>>> register */
>>>>>>>> +#define PCI_CAP_ID_RANGE 0x0 /* Device range register
>>>>>>>> */
>>>>>>>> +#define PCI_CAP_ID_MISC 0x0 /* miscellaneous
>>>>>>>> Information
>>>>>>>> register 0 */
>>>>>>>> +#define PCI_CAP_ID_MISC1 0x0 /* miscellaneous
>>>>>>>> Information
>>>>>>>> register 1 */
>>>>>>>> +
>>>>>>>> +#define FEATURE_PREFETCH (1ULL<<0)
>>>>>>>> +#define FEATURE_PPR (1ULL<<1)
>>>>>>>> +#define FEATURE_NX (1ULL<<3)
>>>>>>>> +#define FEATURE_GT (1ULL<<4)
>>>>>>>> +#define FEATURE_IA (1ULL<<6)
>>>>>>>> +#define FEATURE_GA (1ULL<<7)
>>>>>>>> +#define FEATURE_HE (1ULL<<8)
>>>>>>>> +#define FEATURE_PC (1ULL<<9)
>>>>>>>> +
>>>>>>>> +/* IOMMU paging mode */
>>>>>>>> +#define GATS_MODE (6ULL << 12)
>>>>>>>> +#define HATS_MODE (6ULL << 10)
>>>>>>>> +/* PCI SIG constants */
>>>>>>>> +#define PCI_BUS_MAX 256
>>>>>>>> +#define PCI_SLOT_MAX 32
>>>>>>>> +#define PCI_FUNC_MAX 8
>>>>>>>> +#define PCI_DEVFN_MAX 256
>>>>>>>> +
>>>>>>>> +/* extended feature support */
>>>>>>>> +#define EXT_FEATURES (FEATURE_PREFETCH | FEATURE_PPR | FEATURE_NX |
>>>>>>>> FEATURE_GT | FEATURE_IA | FEATURE_GA | FEATURE_HE | GATS_MODE |
>>>>>>>> HATS_MODE )
>>>>>>>> +
>>>>>>>> +/* capabilities header */
>>>>>>>> +#define CAPAB_FEATURES (CAPAB_FLAT_EXT | CAPAB_FLAG_NPCACHE |
>>>>>>>> CAPAB_FLAG_IOTLBSUP | PCI_CAP_ID_SEC | CAPAB_INIT_TYPE |
>>>>>>>> CAPAB_FLAG_HTTUNNEL
>>>>>>>> | CAPAB_EFR_SUP)
>>>>>>>> +
>>>>>>>> +/* command constants */
>>>>>>>> +#define COM_STORE_ADDRESS_MASK 0xffffffffffff8
>>>>>>>> +#define COM_COMPLETION_STORE_MASK 0x1
>>>>>>>> +#define COM_COMPLETION_INTR 0x2
>>>>>>>> +#define COM_COMPLETION_DATA_OFF 0x8
>>>>>>>> +#define IOMMU_COMMAND_SIZE 0x10
>>>>>>>> +
>>>>>>>> +/* IOMMU default address */
>>>>>>>> +#define BUS_AMD_IOMMU_ADDR 0xfeb00000
>>>>>>>> +
>>>>>>>> +/* page management constants */
>>>>>>>> +#define IOMMU_PAGE_SHIFT 12
>>>>>>>> +#define IOMMU_PAGE_SIZE (1ULL << IOMMU_PAGE_SHIFT)
>>>>>>>> +
>>>>>>>> +#define IOMMU_PAGE_SHIFT_4K 12
>>>>>>>> +#define IOMMU_PAGE_MASK_4K (~((1ULL << IOMMU_PAGE_SHIFT_4K) - 1))
>>>>>>>> +#define IOMMU_PAGE_SHIFT_2M 21
>>>>>>>> +#define IOMMU_PAGE_MASK_2M (~((1ULL << IOMMU_PAGE_SHIFT_2M) -1))
>>>>>>>> +#define IOMMU_PAGE_SHIFT_1G 30
>>>>>>>> +#define IOMMU_PAGE_MASK_1G (~((1ULL << IOMMU_PAGE_SHIFT_1G) - 1))
>>>>>>>> +
>>>>>>>> +#define TYPE_AMD_IOMMU_DEVICE "amd-iommu"
>>>>>>>> +#define AMD_IOMMU_DEVICE(obj)\
>>>>>>>> + OBJECT_CHECK(AMDIOMMUState, (obj), TYPE_AMD_IOMMU_DEVICE)
>>>>>>>> +
>>>>>>>> +typedef struct AMDIOMMUState AMDIOMMUState;
>>>>>>>> +
>>>>>>>> +typedef struct AMDIOMMUAddressSpace{
>>>>>>>> + uint8_t bus_num; /* bus number
>>>>>>>> */
>>>>>>>> + uint8_t devfn; /* device function
>>>>>>>> */
>>>>>>>> + AMDIOMMUState *iommu_state; /* IOMMU - one per machine
>>>>>>>> */
>>>>>>>> + MemoryRegion iommu; /* Device's iommu region
>>>>>>>> */
>>>>>>>> + AddressSpace as; /* device's corresponding address
>>>>>>>> space
>>>>>>>> */
>>>>>>>> +}AMDIOMMUAddressSpace;
>>>>>>>> +
>>>>>>>> +struct AMDIOMMUState {
>>>>>>>> + PCIDevice dev; /* The PCI device itself */
>>>>>>>> +
>>>>>>>> + uint32_t version;
>>>>>>>> +
>>>>>>>> + int capab_offset; /* capability offset pointer */
>>>>>>>> + unsigned char *capab; /* capabilities registers */
>>>>>>>> +
>>>>>>>> + bool enabled; /* IOMMU enabled */
>>>>>>>> + bool ats_enabled; /* address translation enabled */
>>>>>>>> + bool cmdbuf_enabled; /* command buffer enabled */
>>>>>>>> + bool evtlog_enabled; /* event log enabled */
>>>>>>>> + bool excl_enabled;
>>>>>>>> +
>>>>>>>> + dma_addr_t devtab; /* base address device table */
>>>>>>>> + size_t devtab_len; /* device table length */
>>>>>>>> +
>>>>>>>> + dma_addr_t cmdbuf; /* command buffer base address */
>>>>>>>> + uint64_t cmdbuf_len; /* command buffer length */
>>>>>>>> + unsigned cmdbuf_head; /* current IOMMU read position */
>>>>>>>> + unsigned cmdbuf_tail; /* next Software write position */
>>>>>>>> + int completion_wait_intr;
>>>>>>>> +
>>>>>>>> + dma_addr_t evtlog; /* base address event log */
>>>>>>>> + size_t evtlog_intr;
>>>>>>>> + size_t evtlog_len; /* event log length */
>>>>>>>> + size_t evtlog_head; /* event log head */
>>>>>>>> + size_t evtlog_tail; /* event log tail */
>>>>>>>> +
>>>>>>>> + /* unused for now */
>>>>>>>> + dma_addr_t excl_base; /* base DVA - IOMMU exclusion
>>>>>>>> range
>>>>>>>> */
>>>>>>>> + dma_addr_t excl_limit; /* limit of IOMMU exclusion range
>>>>>>>> */
>>>>>>>> + bool excl_allow; /* translate accesses to the
>>>>>>>> exclusion
>>>>>>>> range */
>>>>>>>> + bool excl_enable; /* exclusion range enabled
>>>>>>>> */
>>>>>>>> +
>>>>>>>> + MemoryRegion mmio; /* MMIO region */
>>>>>>>> + unsigned char mmior[MMIO_SIZE];
>>>>>>>> + bool mmio_enabled;
>>>>>>>> +
>>>>>>>> + /* IOMMU function */
>>>>>>>> + MemoryRegionIOMMUOps iommu_ops;
>>>>>>>> +
>>>>>>>> + /* for each served device */
>>>>>>>> + AMDIOMMUAddressSpace **address_spaces[PCI_BUS_MAX];
>>>>>>>> +};
>>>>>>>> +
>>>>>>>> +typedef struct AMDIOMMUEvent {
>>>>>>>> + uint16_t devfn;
>>>>>>>> + uint16_t reserved;
>>>>>>>> + uint16_t domid;
>>>>>>>> + uint16_t info;
>>>>>>>> + uint16_t addr;
>>>>>>>> +} __attribute__((packed)) AMDIOMMUEvent;
>>>>>>>> +
>>>>>>>> +AddressSpace *bridge_host_amd_iommu(PCIBus *bus, void *opaque, int
>>>>>>>> devfn);
>>>>>>>> +
>>>>>>>> +#endif
>>>>>>>>
>>>>>>>
>>>>>>> Regards,
>>>>>>> Valentine
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> С уважением,
>>>>> Синицын Валентин
Re: [Qemu-devel] [RFC 2/4] hw/i386: Introduce AMD IOMMU, Valentine Sinitsyn, 2015/09/12