diff mbox

VFIO: platform: enable ARM64 build

Message ID 1434359632-20278-1-git-send-email-eric.auger@linaro.org
State Accepted
Commit e6bcd47f8f676bd03205c8ffe3fe194936a3e2ca
Headers show

Commit Message

Auger Eric June 15, 2015, 9:13 a.m. UTC
This patch enables building VFIO platform and derivatives on ARM64.

Signed-off-by: Eric Auger <eric.auger@linaro.org>
---
 drivers/vfio/platform/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Auger Eric June 22, 2015, 3:56 p.m. UTC | #1
Hi Baptiste,

No unfortunately I don't have any HW to test this currently. I just
test-compiled this. Up to you to decide whether you prefer waiting for a
functional test for this. I sent that patch since I expect ARM64 to be
the main platform where the VFIO platform drivermight be used for
virtualization use case.

Best Regards

Eric

On 06/22/2015 05:49 PM, Baptiste Reynal wrote:
> Acked-by: Baptiste Reynal <b.reynal@virtualopensystems.com>
> Tested-by: Baptiste Reynal <b.reynal@virtualopensystems.com>
> 
> Out of curiosity, have you get VFIO running on ARM64 without any problems ?
> 
> On Mon, Jun 15, 2015 at 11:13 AM, Eric Auger <eric.auger@linaro.org> wrote:
>> This patch enables building VFIO platform and derivatives on ARM64.
>>
>> Signed-off-by: Eric Auger <eric.auger@linaro.org>
>> ---
>>  drivers/vfio/platform/Kconfig | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/vfio/platform/Kconfig b/drivers/vfio/platform/Kconfig
>> index 1df7477..bb30128 100644
>> --- a/drivers/vfio/platform/Kconfig
>> +++ b/drivers/vfio/platform/Kconfig
>> @@ -1,6 +1,6 @@
>>  config VFIO_PLATFORM
>>         tristate "VFIO support for platform devices"
>> -       depends on VFIO && EVENTFD && ARM
>> +       depends on VFIO && EVENTFD && (ARM || ARM64)
>>         select VFIO_VIRQFD
>>         help
>>           Support for platform devices with VFIO. This is required to make
>> --
>> 1.9.1
>>
Auger Eric June 23, 2015, 7:06 a.m. UTC | #2
Hi Baptiste, Alex,
On 06/23/2015 08:56 AM, Baptiste Reynal wrote:
> On Mon, Jun 22, 2015 at 6:26 PM, Alex Williamson
> <alex.williamson@redhat.com> wrote:
>> On Mon, 2015-06-22 at 17:56 +0200, Eric Auger wrote:
>>> Hi Baptiste,
>>>
>>> No unfortunately I don't have any HW to test this currently. I just
>>> test-compiled this. Up to you to decide whether you prefer waiting for a
>>> functional test for this. I sent that patch since I expect ARM64 to be
>>> the main platform where the VFIO platform drivermight be used for
>>> virtualization use case.
> 
> 
> On my side, VFIO has been tested on ARM64 using Fastmodels without
> problems, we can go ahead with the patch.

OK great

> 
>>
>> FWIW, I've been occasionally removing the arch condition entirely to
>> build test on x86 as well.  The Intel VT-d code certainly allows for
>> devices described by ACPI, which I assume would fall under the platform
>> category, but I don't know of any systems that do this, let alone any
>> x86 platform devices that we'd actually want to expose to the user.
>> Eventually we might want to consider decoupling this from an arch-based
>> Kconfig option, but we could just as easily let the first non-ARM user
>> think about that problem.  Thanks,
>>
>> Alex
>>
> 
> Indeed, I'd rather wait for someone to show up with some use cases and
> tests before removing this condition.
OK

Eric
> 
> Regards,
> Baptiste
> 
>>> On 06/22/2015 05:49 PM, Baptiste Reynal wrote:
>>>> Acked-by: Baptiste Reynal <b.reynal@virtualopensystems.com>
>>>> Tested-by: Baptiste Reynal <b.reynal@virtualopensystems.com>
>>>>
>>>> Out of curiosity, have you get VFIO running on ARM64 without any problems ?
>>>>
>>>> On Mon, Jun 15, 2015 at 11:13 AM, Eric Auger <eric.auger@linaro.org> wrote:
>>>>> This patch enables building VFIO platform and derivatives on ARM64.
>>>>>
>>>>> Signed-off-by: Eric Auger <eric.auger@linaro.org>
>>>>> ---
>>>>>  drivers/vfio/platform/Kconfig | 2 +-
>>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/vfio/platform/Kconfig b/drivers/vfio/platform/Kconfig
>>>>> index 1df7477..bb30128 100644
>>>>> --- a/drivers/vfio/platform/Kconfig
>>>>> +++ b/drivers/vfio/platform/Kconfig
>>>>> @@ -1,6 +1,6 @@
>>>>>  config VFIO_PLATFORM
>>>>>         tristate "VFIO support for platform devices"
>>>>> -       depends on VFIO && EVENTFD && ARM
>>>>> +       depends on VFIO && EVENTFD && (ARM || ARM64)
>>>>>         select VFIO_VIRQFD
>>>>>         help
>>>>>           Support for platform devices with VFIO. This is required to make
>>>>> --
>>>>> 1.9.1
>>>>>
>>>
>>
>>
>>
diff mbox

Patch

diff --git a/drivers/vfio/platform/Kconfig b/drivers/vfio/platform/Kconfig
index 1df7477..bb30128 100644
--- a/drivers/vfio/platform/Kconfig
+++ b/drivers/vfio/platform/Kconfig
@@ -1,6 +1,6 @@ 
 config VFIO_PLATFORM
 	tristate "VFIO support for platform devices"
-	depends on VFIO && EVENTFD && ARM
+	depends on VFIO && EVENTFD && (ARM || ARM64)
 	select VFIO_VIRQFD
 	help
 	  Support for platform devices with VFIO. This is required to make