[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 14/32] pc-dimm: drop the prefix of pc-dimm
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v3 14/32] pc-dimm: drop the prefix of pc-dimm |
Date: |
Mon, 12 Oct 2015 10:43:58 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 10/10/2015 09:52 PM, Xiao Guangrong wrote:
> This patch is generated by this script:
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/PC_DIMM/DIMM/g"
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/PCDIMM/DIMM/g"
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/pc_dimm/dimm/g"
>
> find ./ -name "trace-events" -type f | xargs sed -i "s/pc-dimm/dimm/g"
>
> It prepares the work which abstracts dimm device type for both pc-dimm and
> nvdimm
>
> Signed-off-by: Xiao Guangrong <address@hidden>
> ---
> hmp.c | 2 +-
> +++ b/qapi-schema.json
> @@ -3684,9 +3684,9 @@
> { 'command': 'query-memdev', 'returns': ['Memdev'] }
>
> ##
> -# @PCDIMMDeviceInfo:
> +# @DIMMDeviceInfo:
> #
> -# PCDIMMDevice state information
> +# DIMMDevice state information
> #
> # @id: #optional device's ID
> #
> @@ -3706,7 +3706,7 @@
> #
> # Since: 2.1
> ##
> -{ 'struct': 'PCDIMMDeviceInfo',
> +{ 'struct': 'DIMMDeviceInfo',
> 'data': { '*id': 'str',
> 'addr': 'int',
> 'size': 'int',
> @@ -3725,7 +3725,7 @@
> #
> # Since: 2.1
> ##
> -{ 'union': 'MemoryDeviceInfo', 'data': {'dimm': 'PCDIMMDeviceInfo'} }
> +{ 'union': 'MemoryDeviceInfo', 'data': {'dimm': 'DIMMDeviceInfo'} }
Struct names are not ABI, so this change is safe.
I have not reviewed the rest of the patch, but I don't see any problems
from the qapi perspective.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, (continued)
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Xiao Guangrong, 2015/10/12
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Dan Williams, 2015/10/13
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Xiao Guangrong, 2015/10/13
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Dan Williams, 2015/10/13
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Xiao Guangrong, 2015/10/14
- Re: [Qemu-devel] [PATCH v3 00/32] implement vNVDIMM, Dan Williams, 2015/10/14
[Qemu-devel] [PATCH v3 03/32] acpi: add aml_create_field, Xiao Guangrong, 2015/10/10
[Qemu-devel] [PATCH v3 14/32] pc-dimm: drop the prefix of pc-dimm, Xiao Guangrong, 2015/10/10
- Re: [Qemu-devel] [PATCH v3 14/32] pc-dimm: drop the prefix of pc-dimm,
Eric Blake <=
[Qemu-devel] [PATCH v3 04/32] acpi: add aml_mutex, aml_acquire, aml_release, Xiao Guangrong, 2015/10/10
[Qemu-devel] [PATCH v3 10/32] hostmem-file: clean up memory allocation, Xiao Guangrong, 2015/10/10
[Qemu-devel] [PATCH v3 12/32] pc-dimm: remove DEFAULT_PC_DIMMSIZE, Xiao Guangrong, 2015/10/10
[Qemu-devel] [PATCH v3 22/32] nvdimm: init the address region used by NVDIMM ACPI, Xiao Guangrong, 2015/10/10