[v1] accel: Allow targets to use Kconfig, disable semihosting by default

[PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Philippe Mathieu-Daudé 5 years, 10 months ago

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
Cc: Reviewed-by: Nikita Leshenko <nikita.leshchenko@oracle.com>
Cc: Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>
Cc: Roman Bolshakov <r.bolshakov@yadro.com>
Cc: Patrick Colp <patrick.colp@oracle.com>
Cc: Cameron Esfahani <dirty@apple.com>
Cc: Liran Alon <liran.alon@oracle.com>
Cc: Heiher <r@hev.cc>
---
 MAINTAINERS | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 7ec42a18f7..bcf40afb85 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -420,6 +420,12 @@ F: accel/stubs/hax-stub.c
 F: target/i386/hax-all.c
 F: include/sysemu/hax.h
 
+HVF Accelerator
+S: Orphan
+F: accel/stubs/hvf-stub.c
+F: target/i386/hvf/hvf.c
+F: include/sysemu/hvf.h
+
 WHPX CPUs
 M: Sunil Muthuswamy <sunilmut@microsoft.com>
 S: Supported
-- 
2.21.1

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Roman Bolshakov 5 years, 10 months ago

Hi Philippe,

I can take the ownership if nobody wants it. At the moment I'm working
on APIC for HVF to get kvm-unit-tests fixed.

Next items on the list (in no particular order):
* MMX emulation
* SSE emulation
* qxl display
* gdb stub
* virtio-gpu/virgil running on metal
* VFIO-PCI based on macOS user-space DriverKit framework

Best regards,
Roman

On Mon, Mar 16, 2020 at 01:00:42PM +0100, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> Cc: Reviewed-by: Nikita Leshenko <nikita.leshchenko@oracle.com>
> Cc: Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>
> Cc: Roman Bolshakov <r.bolshakov@yadro.com>
> Cc: Patrick Colp <patrick.colp@oracle.com>
> Cc: Cameron Esfahani <dirty@apple.com>
> Cc: Liran Alon <liran.alon@oracle.com>
> Cc: Heiher <r@hev.cc>
> ---
>  MAINTAINERS | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7ec42a18f7..bcf40afb85 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -420,6 +420,12 @@ F: accel/stubs/hax-stub.c
>  F: target/i386/hax-all.c
>  F: include/sysemu/hax.h
>  
> +HVF Accelerator
> +S: Orphan
> +F: accel/stubs/hvf-stub.c
> +F: target/i386/hvf/hvf.c
> +F: include/sysemu/hvf.h
> +
>  WHPX CPUs
>  M: Sunil Muthuswamy <sunilmut@microsoft.com>
>  S: Supported
> -- 
> 2.21.1
>

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Philippe Mathieu-Daudé 5 years, 10 months ago

Hi Roman,

On 3/16/20 1:12 PM, Roman Bolshakov wrote:
> Hi Philippe,
> 
> I can take the ownership if nobody wants it. At the moment I'm working
> on APIC for HVF to get kvm-unit-tests fixed.
> 
> Next items on the list (in no particular order):
> * MMX emulation
> * SSE emulation
> * qxl display
> * gdb stub
> * virtio-gpu/virgil running on metal
> * VFIO-PCI based on macOS user-space DriverKit framework

Glad to hear :)
I suppose Paolo will be happy to have someone caring about HVF.
Do you mind sending a patch to step in?

Thanks,

Phil.

> 
> Best regards,
> Roman
> 
> On Mon, Mar 16, 2020 at 01:00:42PM +0100, Philippe Mathieu-Daudé wrote:
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> Cc: Reviewed-by: Nikita Leshenko <nikita.leshchenko@oracle.com>
>> Cc: Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>
>> Cc: Roman Bolshakov <r.bolshakov@yadro.com>
>> Cc: Patrick Colp <patrick.colp@oracle.com>
>> Cc: Cameron Esfahani <dirty@apple.com>
>> Cc: Liran Alon <liran.alon@oracle.com>
>> Cc: Heiher <r@hev.cc>
>> ---
>>   MAINTAINERS | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7ec42a18f7..bcf40afb85 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -420,6 +420,12 @@ F: accel/stubs/hax-stub.c
>>   F: target/i386/hax-all.c
>>   F: include/sysemu/hax.h
>>   
>> +HVF Accelerator
>> +S: Orphan
>> +F: accel/stubs/hvf-stub.c
>> +F: target/i386/hvf/hvf.c
>> +F: include/sysemu/hvf.h
>> +
>>   WHPX CPUs
>>   M: Sunil Muthuswamy <sunilmut@microsoft.com>
>>   S: Supported
>> -- 
>> 2.21.1
>>
>

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Cameron Esfahani via 5 years, 10 months ago

Sorry I didn't see this yesterday.

We've (Apple) signed up for taking over HVF ownership.  I didn't realize I needed to add to the MAINTAINERS list.

Roman, we also have a bunch of pending fixes for some of the issues you've listed.  We're in the process of upstreaming them.

Cameron Esfahani
dirty@apple.com

"All that is necessary for the triumph of evil is that good men do nothing."

Edmund Burke



> On Mar 16, 2020, at 5:12 AM, Roman Bolshakov <r.bolshakov@yadro.com> wrote:
> 
> Hi Philippe,
> 
> I can take the ownership if nobody wants it. At the moment I'm working
> on APIC for HVF to get kvm-unit-tests fixed.
> 
> Next items on the list (in no particular order):
> * MMX emulation
> * SSE emulation
> * qxl display
> * gdb stub
> * virtio-gpu/virgil running on metal
> * VFIO-PCI based on macOS user-space DriverKit framework
> 
> Best regards,
> Roman
> 
> On Mon, Mar 16, 2020 at 01:00:42PM +0100, Philippe Mathieu-Daudé wrote:
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> ---
>> Cc: Reviewed-by: Nikita Leshenko <nikita.leshchenko@oracle.com>
>> Cc: Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>
>> Cc: Roman Bolshakov <r.bolshakov@yadro.com>
>> Cc: Patrick Colp <patrick.colp@oracle.com>
>> Cc: Cameron Esfahani <dirty@apple.com>
>> Cc: Liran Alon <liran.alon@oracle.com>
>> Cc: Heiher <r@hev.cc>
>> ---
>> MAINTAINERS | 6 ++++++
>> 1 file changed, 6 insertions(+)
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7ec42a18f7..bcf40afb85 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -420,6 +420,12 @@ F: accel/stubs/hax-stub.c
>> F: target/i386/hax-all.c
>> F: include/sysemu/hax.h
>> 
>> +HVF Accelerator
>> +S: Orphan
>> +F: accel/stubs/hvf-stub.c
>> +F: target/i386/hvf/hvf.c
>> +F: include/sysemu/hvf.h
>> +
>> WHPX CPUs
>> M: Sunil Muthuswamy <sunilmut@microsoft.com>
>> S: Supported
>> -- 
>> 2.21.1
>> 
>

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Roman Bolshakov 5 years, 10 months ago

On Tue, Mar 17, 2020 at 04:24:44PM -0700, Cameron Esfahani wrote:
> Sorry I didn't see this yesterday.
> 
> We've (Apple) signed up for taking over HVF ownership.  I didn't realize I needed to add to the MAINTAINERS list.
> 
> Roman, we also have a bunch of pending fixes for some of the issues you've listed.  We're in the process of upstreaming them.
> 

Hi Cameron,

That's great news. What exactly are you planning to upstream first?
BTW many thanks for the fixes you made last December.

Best regards,
Roman

Re: [PATCH 04/11] MAINTAINERS: Add an entry for the HVF accelerator

Posted by Cameron Esfahani via 5 years, 10 months ago

Please add me to the HVF maintainers as well.

Cameron Esfahani
dirty@apple.com

"In the elder days of Art, Builders wrought with greatest care each minute and unseen part; For the gods see everywhere."

"The Builders", H. W. Longfellow



> On Mar 16, 2020, at 5:00 AM, Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> Cc: Reviewed-by: Nikita Leshenko <nikita.leshchenko@oracle.com>
> Cc: Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>
> Cc: Roman Bolshakov <r.bolshakov@yadro.com>
> Cc: Patrick Colp <patrick.colp@oracle.com>
> Cc: Cameron Esfahani <dirty@apple.com>
> Cc: Liran Alon <liran.alon@oracle.com>
> Cc: Heiher <r@hev.cc>
> ---
> MAINTAINERS | 6 ++++++
> 1 file changed, 6 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7ec42a18f7..bcf40afb85 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -420,6 +420,12 @@ F: accel/stubs/hax-stub.c
> F: target/i386/hax-all.c
> F: include/sysemu/hax.h
> 
> +HVF Accelerator
> +S: Orphan
> +F: accel/stubs/hvf-stub.c
> +F: target/i386/hvf/hvf.c
> +F: include/sysemu/hvf.h
> +
> WHPX CPUs
> M: Sunil Muthuswamy <sunilmut@microsoft.com>
> S: Supported
> -- 
> 2.21.1
> 
>