[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v3 3/3] softmmu/physmem: Introduce MemTxAttrs::memory fie
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [RFC PATCH v3 3/3] softmmu/physmem: Introduce MemTxAttrs::memory field and MEMTX_BUS_ERROR |
Date: |
Sat, 18 Dec 2021 12:26:27 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 |
On 12/18/21 03:07, Richard Henderson wrote:
> On 12/17/21 3:18 PM, Philippe Mathieu-Daudé wrote:
>>>> #define MEMTX_OK 0
>>>> #define MEMTX_ERROR (1U << 0) /* device returned an
>>>> error */
>>>> #define MEMTX_DECODE_ERROR (1U << 1) /* nothing at that
>>>> address */
>>>> +#define MEMTX_BUS_ERROR (1U << 2) /* bus returned an error */
>>>
>>> This is kind of odd naming, because MEMTX_DECODE_ERROR already means
>>> "bus/interconnect returned an error" and it generally translates
>>> into what at the OS level gets called a "bus error"...
>>
>> MEMTX_DECODE_ERROR is "nothing at that address". We want a name
>> for "there is something, but you don't have access to it".
>> Maybe MEMTX_ILLEGAL_ERROR?
>
> ILLEGAL doesn't convey much. MEMTX_ACCESS_ERROR?
OK, such:
#define MEMTX_ACCESS_ERROR (1U << 2) /* access denied */
alternatively MEMTX_PERM_ERROR.