Re: [PATCH 0/3] hw: Constify VMStateDescription

2021-04-08 Thread Keqian Zhu



On 2021/4/8 21:22, Philippe Mathieu-Daudé wrote:
> On 3/15/21 10:05 AM, Keqian Zhu wrote:
>> Hi Philippe,
>>
>> It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
>> Found by .
> 
> Correct (I only searched for the static ones).
> 
> Do you mind sending a patch?
OK, I'll do. :)

BRs,
Keqian

> 
>>
>> Thanks,
>> Keqian
>>
>> On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
>>> VMStateDescription isn't supposed to be modified.
> .
> 



Re: [PATCH 0/3] hw: Constify VMStateDescription

2021-04-08 Thread Philippe Mathieu-Daudé
On 3/15/21 10:05 AM, Keqian Zhu wrote:
> Hi Philippe,
> 
> It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
> Found by .

Correct (I only searched for the static ones).

Do you mind sending a patch?

> 
> Thanks,
> Keqian
> 
> On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
>> VMStateDescription isn't supposed to be modified.



Re: [PATCH 0/3] hw: Constify VMStateDescription

2021-03-15 Thread Keqian Zhu
Hi Philippe,

It seems that vmstate_ecc_state and vmstate_x86_cpu can also be constified.
Found by .

Thanks,
Keqian

On 2021/3/14 1:11, Philippe Mathieu-Daudé wrote:
> VMStateDescription isn't supposed to be modified.
> 
> 
> 
> Philippe Mathieu-Daudé (3):
> 
>   hw/arm: Constify VMStateDescription
> 
>   hw/display/qxl: Constify VMStateDescription
> 
>   hw/usb: Constify VMStateDescription
> 
> 
> 
>  hw/arm/highbank.c | 2 +-
> 
>  hw/arm/pxa2xx_pic.c   | 2 +-
> 
>  hw/arm/spitz.c| 4 ++--
> 
>  hw/arm/strongarm.c| 2 +-
> 
>  hw/arm/z2.c   | 4 ++--
> 
>  hw/display/qxl.c  | 8 
> 
>  hw/dma/pxa2xx_dma.c   | 4 ++--
> 
>  hw/misc/mst_fpga.c| 2 +-
> 
>  hw/usb/ccid-card-passthru.c   | 2 +-
> 
>  hw/usb/dev-smartcard-reader.c | 8 
> 
>  10 files changed, 19 insertions(+), 19 deletions(-)
> 
> 
> 



Re: [PATCH 0/3] hw: Constify VMStateDescription

2021-03-13 Thread Richard Henderson

On 3/13/21 11:11 AM, Philippe Mathieu-Daudé wrote:

Philippe Mathieu-Daudé (3):
   hw/arm: Constify VMStateDescription
   hw/display/qxl: Constify VMStateDescription
   hw/usb: Constify VMStateDescription


Reviewed-by: Richard Henderson 

r~



[PATCH 0/3] hw: Constify VMStateDescription

2021-03-13 Thread Philippe Mathieu-Daudé
VMStateDescription isn't supposed to be modified.

Philippe Mathieu-Daudé (3):
  hw/arm: Constify VMStateDescription
  hw/display/qxl: Constify VMStateDescription
  hw/usb: Constify VMStateDescription

 hw/arm/highbank.c | 2 +-
 hw/arm/pxa2xx_pic.c   | 2 +-
 hw/arm/spitz.c| 4 ++--
 hw/arm/strongarm.c| 2 +-
 hw/arm/z2.c   | 4 ++--
 hw/display/qxl.c  | 8 
 hw/dma/pxa2xx_dma.c   | 4 ++--
 hw/misc/mst_fpga.c| 2 +-
 hw/usb/ccid-card-passthru.c   | 2 +-
 hw/usb/dev-smartcard-reader.c | 8 
 10 files changed, 19 insertions(+), 19 deletions(-)

-- 
2.26.2