* [PATCH] VFIO: platform: enable ARM64 build @ 2015-06-15 9:13 Eric Auger 2015-06-22 15:49 ` Baptiste Reynal 0 siblings, 1 reply; 6+ messages in thread From: Eric Auger @ 2015-06-15 9:13 UTC (permalink / raw) To: linux-arm-kernel 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 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] VFIO: platform: enable ARM64 build 2015-06-15 9:13 [PATCH] VFIO: platform: enable ARM64 build Eric Auger @ 2015-06-22 15:49 ` Baptiste Reynal 2015-06-22 15:56 ` Eric Auger 0 siblings, 1 reply; 6+ messages in thread From: Baptiste Reynal @ 2015-06-22 15:49 UTC (permalink / raw) To: linux-arm-kernel 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 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] VFIO: platform: enable ARM64 build 2015-06-22 15:49 ` Baptiste Reynal @ 2015-06-22 15:56 ` Eric Auger 2015-06-22 16:26 ` Alex Williamson 0 siblings, 1 reply; 6+ messages in thread From: Eric Auger @ 2015-06-22 15:56 UTC (permalink / raw) To: linux-arm-kernel 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 >> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] VFIO: platform: enable ARM64 build 2015-06-22 15:56 ` Eric Auger @ 2015-06-22 16:26 ` Alex Williamson 2015-06-23 6:56 ` Baptiste Reynal 0 siblings, 1 reply; 6+ messages in thread From: Alex Williamson @ 2015-06-22 16:26 UTC (permalink / raw) To: linux-arm-kernel 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. 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 > 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 > >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] VFIO: platform: enable ARM64 build 2015-06-22 16:26 ` Alex Williamson @ 2015-06-23 6:56 ` Baptiste Reynal 2015-06-23 7:06 ` Eric Auger 0 siblings, 1 reply; 6+ messages in thread From: Baptiste Reynal @ 2015-06-23 6:56 UTC (permalink / raw) To: linux-arm-kernel 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. > > 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. 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 >> >> >> > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] VFIO: platform: enable ARM64 build 2015-06-23 6:56 ` Baptiste Reynal @ 2015-06-23 7:06 ` Eric Auger 0 siblings, 0 replies; 6+ messages in thread From: Eric Auger @ 2015-06-23 7:06 UTC (permalink / raw) To: linux-arm-kernel 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 >>>>> >>> >> >> >> ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-06-23 7:06 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-06-15 9:13 [PATCH] VFIO: platform: enable ARM64 build Eric Auger 2015-06-22 15:49 ` Baptiste Reynal 2015-06-22 15:56 ` Eric Auger 2015-06-22 16:26 ` Alex Williamson 2015-06-23 6:56 ` Baptiste Reynal 2015-06-23 7:06 ` Eric Auger
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).