[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region
From: |
Eric Auger |
Subject: |
Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region |
Date: |
Tue, 8 Feb 2022 18:58:40 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
Hi Stefan,
On 2/8/22 6:16 PM, Stefan Berger wrote:
>
> On 2/8/22 08:38, Eric Auger wrote:
>> Representing the CRB cmd/response buffer as a standard
>> RAM region causes some trouble when the device is used
>> with VFIO. Indeed VFIO attempts to DMA_MAP this region
>> as usual RAM but this latter does not have a valid page
>> size alignment causing such an error report:
>> "vfio_listener_region_add received unaligned region".
>> To allow VFIO to detect that failing dma mapping
>> this region is not an issue, let's use a ram_device
>> memory region type instead.
>>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> Tested-by: Stefan Berger <stefanb@linux.ibm.com>
>> Acked-by: Stefan Berger <stefanb@linux.ibm.com>
>> [PMD: Keep tpm_crb.c in meson's softmmu_ss]
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
>
> v4 doesn't build for me:
>
> ../hw/tpm/tpm_crb.c: In function ?tpm_crb_realize?:
> ../hw/tpm/tpm_crb.c:297:33: error: implicit declaration of function
> ?HOST_PAGE_ALIGN? [-Werror=implicit-function-declaration]
> 297 | HOST_PAGE_ALIGN(CRB_CTRL_CMD_SIZE));
> | ^~~~~~~~~~~~~~~
> ../hw/tpm/tpm_crb.c:297:33: error: nested extern declaration of
> ?HOST_PAGE_ALIGN? [-Werror=nested-externs]
> cc1: all warnings being treated as errors
Do you have
b269a70810a exec/cpu: Make host pages variables / macros 'target
agnostic' in your tree?
Thanks
Eric
>
>
>
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, (continued)
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Peter Maydell, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Eric Auger, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Peter Maydell, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Alex Williamson, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Peter Maydell, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Eric Auger, 2022/02/09
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Eric Auger, 2022/02/08
Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Dr. David Alan Gilbert, 2022/02/08
Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region, Stefan Berger, 2022/02/08
- Re: [PATCH v4 1/2] tpm: CRB: Use ram_device for "tpm-crb-cmd" region,
Eric Auger <=
[PATCH v4 2/2] hw/vfio/common: Silence ram device offset alignment error traces, Eric Auger, 2022/02/08