diff mbox series

[v4,19/19] mac_newworld: Document deprecation

Message ID 7832e9b6d79c5243d21f54b0679f487c32d968dd.1666715145.git.balaton@eik.bme.hu
State New
Headers show
Series Misc ppc/mac machines clean up | expand

Commit Message

BALATON Zoltan Oct. 25, 2022, 4:44 p.m. UTC
Also update PowerMac family docs with some more recent info.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
---
 docs/about/deprecated.rst    |  7 +++++++
 docs/system/ppc/powermac.rst | 12 ++++++++----
 2 files changed, 15 insertions(+), 4 deletions(-)

Comments

Mark Cave-Ayland Oct. 28, 2022, 9:41 a.m. UTC | #1
On 25/10/2022 17:44, BALATON Zoltan wrote:

> Also update PowerMac family docs with some more recent info.
> 
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> ---
>   docs/about/deprecated.rst    |  7 +++++++
>   docs/system/ppc/powermac.rst | 12 ++++++++----
>   2 files changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 93affe3669..07661af7fe 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -248,6 +248,13 @@ These old machine types are quite neglected nowadays and thus might have
>   various pitfalls with regards to live migration. Use a newer machine type
>   instead.
>   
> +``mac99`` variants other than the default qemu-system-ppc version (since 7.2)
> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
> +
> +The ``mac99`` machine emulates different hardware depending on using
> +qemu-system-ppc64 or ``via`` property. To avoid confusion new machine
> +types has been added for these variants which are now preferred over
> +``mac99``.
>   
>   Backend options
>   ---------------
> diff --git a/docs/system/ppc/powermac.rst b/docs/system/ppc/powermac.rst
> index 04334ba210..9a37e69b1b 100644
> --- a/docs/system/ppc/powermac.rst
> +++ b/docs/system/ppc/powermac.rst
> @@ -4,8 +4,12 @@ PowerMac family boards (``g3beige``, ``mac99``)
>   Use the executable ``qemu-system-ppc`` to simulate a complete PowerMac
>   PowerPC system.
>   
> -- ``g3beige``              Heathrow based PowerMAC
> -- ``mac99``                Mac99 based PowerMAC
> +- ``g3beige``           Heathrow based old world Power Macintosh G3
> +- ``mac99``             Core99 based generic PowerMac
> +- ``powermac3_1``       Power Mac G4 AGP (Sawtooth)
> +- ``powerbook3_2``      PowerBook G4 Titanium (Mercury)
> +- ``powermac7_3``       Power Mac G5 (Niagara) (only in ``qemu-system-ppc64``)
> +
>   
>   Supported devices
>   -----------------
> @@ -15,9 +19,9 @@ QEMU emulates the following PowerMac peripherals:
>    *  UniNorth or Grackle PCI Bridge
>    *  PCI VGA compatible card with VESA Bochs Extensions
>    *  2 PMAC IDE interfaces with hard disk and CD-ROM support
> - *  NE2000 PCI adapters
> + *  Sungem PCI network adapter
>    *  Non Volatile RAM
> - *  VIA-CUDA with ADB keyboard and mouse.
> + *  VIA-CUDA or VIA-PMU99 with ot without ADB or USB keyboard and mouse.
>   
>   
>   Missing devices

Documentation updates are always useful, but until there is consensus as to how the 
32-bit and 64-bit targets should be handled then I don't think we should go ahead 
with a potential compatibility break/deprecation until we have a clear path forward.

Given that freeze is so close, I suggest leaving this for 7.2 and resurrecting the 
appropriate thread from earlier in the year at the start of the 8.0 development cycle.


ATB,

Mark.
BALATON Zoltan Oct. 28, 2022, 12:20 p.m. UTC | #2
On Fri, 28 Oct 2022, Mark Cave-Ayland wrote:
> On 25/10/2022 17:44, BALATON Zoltan wrote:
>> Also update PowerMac family docs with some more recent info.
>> 
>> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
>> ---
>>   docs/about/deprecated.rst    |  7 +++++++
>>   docs/system/ppc/powermac.rst | 12 ++++++++----
>>   2 files changed, 15 insertions(+), 4 deletions(-)
>> 
>> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
>> index 93affe3669..07661af7fe 100644
>> --- a/docs/about/deprecated.rst
>> +++ b/docs/about/deprecated.rst
>> @@ -248,6 +248,13 @@ These old machine types are quite neglected nowadays 
>> and thus might have
>>   various pitfalls with regards to live migration. Use a newer machine type
>>   instead.
>>   +``mac99`` variants other than the default qemu-system-ppc version (since 
>> 7.2)
>> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>> +
>> +The ``mac99`` machine emulates different hardware depending on using
>> +qemu-system-ppc64 or ``via`` property. To avoid confusion new machine
>> +types has been added for these variants which are now preferred over
>> +``mac99``.
>>     Backend options
>>   ---------------
>> diff --git a/docs/system/ppc/powermac.rst b/docs/system/ppc/powermac.rst
>> index 04334ba210..9a37e69b1b 100644
>> --- a/docs/system/ppc/powermac.rst
>> +++ b/docs/system/ppc/powermac.rst
>> @@ -4,8 +4,12 @@ PowerMac family boards (``g3beige``, ``mac99``)
>>   Use the executable ``qemu-system-ppc`` to simulate a complete PowerMac
>>   PowerPC system.
>>   -- ``g3beige``              Heathrow based PowerMAC
>> -- ``mac99``                Mac99 based PowerMAC
>> +- ``g3beige``           Heathrow based old world Power Macintosh G3
>> +- ``mac99``             Core99 based generic PowerMac
>> +- ``powermac3_1``       Power Mac G4 AGP (Sawtooth)
>> +- ``powerbook3_2``      PowerBook G4 Titanium (Mercury)
>> +- ``powermac7_3``       Power Mac G5 (Niagara) (only in 
>> ``qemu-system-ppc64``)
>> +
>>     Supported devices
>>   -----------------
>> @@ -15,9 +19,9 @@ QEMU emulates the following PowerMac peripherals:
>>    *  UniNorth or Grackle PCI Bridge
>>    *  PCI VGA compatible card with VESA Bochs Extensions
>>    *  2 PMAC IDE interfaces with hard disk and CD-ROM support
>> - *  NE2000 PCI adapters
>> + *  Sungem PCI network adapter
>>    *  Non Volatile RAM
>> - *  VIA-CUDA with ADB keyboard and mouse.
>> + *  VIA-CUDA or VIA-PMU99 with ot without ADB or USB keyboard and mouse.
>>       Missing devices
>
> Documentation updates are always useful, but until there is consensus as to 
> how the 32-bit and 64-bit targets should be handled then I don't think we 
> should go ahead with a potential compatibility break/deprecation until we 
> have a clear path forward.
>
> Given that freeze is so close, I suggest leaving this for 7.2 and 
> resurrecting the appropriate thread from earlier in the year at the start of 
> the 8.0 development cycle.

Please don't postpone patches because you were not able to review in time. 
A better approach would be to merge these now and drop them during the 
freeze if any problem that can't be fixed is found. The deprecation 
process itself is also to allow backing off if this turns out to be a bad 
idea so no need to wait for more votes now and postpone this further.

Regards,
BALATON Zoltan
Mark Cave-Ayland Oct. 29, 2022, 8:09 a.m. UTC | #3
On 28/10/2022 13:20, BALATON Zoltan wrote:

> On Fri, 28 Oct 2022, Mark Cave-Ayland wrote:
>> On 25/10/2022 17:44, BALATON Zoltan wrote:
>>> Also update PowerMac family docs with some more recent info.
>>>
>>> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
>>> ---
>>>   docs/about/deprecated.rst    |  7 +++++++
>>>   docs/system/ppc/powermac.rst | 12 ++++++++----
>>>   2 files changed, 15 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
>>> index 93affe3669..07661af7fe 100644
>>> --- a/docs/about/deprecated.rst
>>> +++ b/docs/about/deprecated.rst
>>> @@ -248,6 +248,13 @@ These old machine types are quite neglected nowadays and thus 
>>> might have
>>>   various pitfalls with regards to live migration. Use a newer machine type
>>>   instead.
>>>   +``mac99`` variants other than the default qemu-system-ppc version (since 7.2)
>>> +'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
>>> +
>>> +The ``mac99`` machine emulates different hardware depending on using
>>> +qemu-system-ppc64 or ``via`` property. To avoid confusion new machine
>>> +types has been added for these variants which are now preferred over
>>> +``mac99``.
>>>     Backend options
>>>   ---------------
>>> diff --git a/docs/system/ppc/powermac.rst b/docs/system/ppc/powermac.rst
>>> index 04334ba210..9a37e69b1b 100644
>>> --- a/docs/system/ppc/powermac.rst
>>> +++ b/docs/system/ppc/powermac.rst
>>> @@ -4,8 +4,12 @@ PowerMac family boards (``g3beige``, ``mac99``)
>>>   Use the executable ``qemu-system-ppc`` to simulate a complete PowerMac
>>>   PowerPC system.
>>>   -- ``g3beige``              Heathrow based PowerMAC
>>> -- ``mac99``                Mac99 based PowerMAC
>>> +- ``g3beige``           Heathrow based old world Power Macintosh G3
>>> +- ``mac99``             Core99 based generic PowerMac
>>> +- ``powermac3_1``       Power Mac G4 AGP (Sawtooth)
>>> +- ``powerbook3_2``      PowerBook G4 Titanium (Mercury)
>>> +- ``powermac7_3``       Power Mac G5 (Niagara) (only in ``qemu-system-ppc64``)
>>> +
>>>     Supported devices
>>>   -----------------
>>> @@ -15,9 +19,9 @@ QEMU emulates the following PowerMac peripherals:
>>>    *  UniNorth or Grackle PCI Bridge
>>>    *  PCI VGA compatible card with VESA Bochs Extensions
>>>    *  2 PMAC IDE interfaces with hard disk and CD-ROM support
>>> - *  NE2000 PCI adapters
>>> + *  Sungem PCI network adapter
>>>    *  Non Volatile RAM
>>> - *  VIA-CUDA with ADB keyboard and mouse.
>>> + *  VIA-CUDA or VIA-PMU99 with ot without ADB or USB keyboard and mouse.
>>>       Missing devices
>>
>> Documentation updates are always useful, but until there is consensus as to how the 
>> 32-bit and 64-bit targets should be handled then I don't think we should go ahead 
>> with a potential compatibility break/deprecation until we have a clear path forward.
>>
>> Given that freeze is so close, I suggest leaving this for 7.2 and resurrecting the 
>> appropriate thread from earlier in the year at the start of the 8.0 development cycle.
> 
> Please don't postpone patches because you were not able to review in time. A better 
> approach would be to merge these now and drop them during the freeze if any problem 
> that can't be fixed is found. The deprecation process itself is also to allow backing 
> off if this turns out to be a bad idea so no need to wait for more votes now and 
> postpone this further.

As I've previously mentioned, there is no commercial sponsorship for the QEMU Mac 
machines which is why the Mac machines are marked as Maintained, and so patch review 
is often limited by my available spare time. This does not mean that we should bypass 
the entire QEMU review process though.


ATB,

Mark.
diff mbox series

Patch

diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 93affe3669..07661af7fe 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -248,6 +248,13 @@  These old machine types are quite neglected nowadays and thus might have
 various pitfalls with regards to live migration. Use a newer machine type
 instead.
 
+``mac99`` variants other than the default qemu-system-ppc version (since 7.2)
+'''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+The ``mac99`` machine emulates different hardware depending on using
+qemu-system-ppc64 or ``via`` property. To avoid confusion new machine
+types has been added for these variants which are now preferred over
+``mac99``.
 
 Backend options
 ---------------
diff --git a/docs/system/ppc/powermac.rst b/docs/system/ppc/powermac.rst
index 04334ba210..9a37e69b1b 100644
--- a/docs/system/ppc/powermac.rst
+++ b/docs/system/ppc/powermac.rst
@@ -4,8 +4,12 @@  PowerMac family boards (``g3beige``, ``mac99``)
 Use the executable ``qemu-system-ppc`` to simulate a complete PowerMac
 PowerPC system.
 
-- ``g3beige``              Heathrow based PowerMAC
-- ``mac99``                Mac99 based PowerMAC
+- ``g3beige``           Heathrow based old world Power Macintosh G3
+- ``mac99``             Core99 based generic PowerMac
+- ``powermac3_1``       Power Mac G4 AGP (Sawtooth)
+- ``powerbook3_2``      PowerBook G4 Titanium (Mercury)
+- ``powermac7_3``       Power Mac G5 (Niagara) (only in ``qemu-system-ppc64``)
+
 
 Supported devices
 -----------------
@@ -15,9 +19,9 @@  QEMU emulates the following PowerMac peripherals:
  *  UniNorth or Grackle PCI Bridge
  *  PCI VGA compatible card with VESA Bochs Extensions
  *  2 PMAC IDE interfaces with hard disk and CD-ROM support
- *  NE2000 PCI adapters
+ *  Sungem PCI network adapter
  *  Non Volatile RAM
- *  VIA-CUDA with ADB keyboard and mouse.
+ *  VIA-CUDA or VIA-PMU99 with ot without ADB or USB keyboard and mouse.
 
 
 Missing devices