[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 07/11] dump: Add API to write dump_bitmap
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH v6 07/11] dump: Add API to write dump_bitmap |
Date: |
Tue, 07 Jan 2014 22:41:30 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131118 Thunderbird/17.0.11 |
On 01/07/14 15:49, Laszlo Ersek wrote:
>
> On 01/05/14 08:27, Qiao Nuohan wrote:
>> diff --git a/include/sysemu/dump.h b/include/sysemu/dump.h
>> index 9e47b4c..b5eaf8d 100644
>> --- a/include/sysemu/dump.h
>> +++ b/include/sysemu/dump.h
>> @@ -27,11 +27,18 @@
>> #define DUMP_DH_COMPRESSED_LZO (0x2)
>> #define DUMP_DH_COMPRESSED_SNAPPY (0x4)
>>
>> +#define PAGE_SIZE (4096)
>> #define KDUMP_SIGNATURE "KDUMP "
>> #define SIG_LEN (sizeof(KDUMP_SIGNATURE) - 1)
>> #define PHYS_BASE (0)
>> #define DUMP_LEVEL (1)
>> #define DISKDUMP_HEADER_BLOCKS (1)
>> +#define BUFSIZE_BITMAP (PAGE_SIZE)
>> +#define PFN_BUFBITMAP (CHAR_BIT * BUFSIZE_BITMAP)
>> +#define ARCH_PFN_OFFSET (0)
>> +
>> +#define paddr_to_pfn(X, page_shift) \
>> + (((unsigned long long)(X) >> (page_shift)) - ARCH_PFN_OFFSET)
>>
>> typedef struct ArchDumpInfo {
>> int d_machine; /* Architecture */
>>
>
> I think these magic constants are somewhat tied to x86, and therefore
> should be in an arch-specific file rather than a common file, but
> whoever wants to extend this to another architecture can do that.
Stressing the argument a bit more for PAGE_SIZE specifically:
- we already have TARGET_PAGE_SIZE, maybe that would be a better choice,
- PAGE_SIZE is defined *as* TARGET_PAGE_SIZE in kvm-all.c. There's no
actual conflict, but the mental conflict is bad enough.
Anyway my R-b stands.
Laszlo
- Re: [Qemu-devel] [PATCH v6 04/11] dump: Add API to write elf notes to buffer, (continued)
- [Qemu-devel] [PATCH v6 10/11] dump: Make kdump-compressed format available for 'dump-guest-memory', Qiao Nuohan, 2014/01/05
- [Qemu-devel] [PATCH v6 08/11] dump: Add APIs to operate DataCache, Qiao Nuohan, 2014/01/05
- [Qemu-devel] [PATCH v6 09/11] dump: Add API to write dump pages, Qiao Nuohan, 2014/01/05
- [Qemu-devel] [PATCH v6 07/11] dump: Add API to write dump_bitmap, Qiao Nuohan, 2014/01/05
- [Qemu-devel] [PATCH v6 01/11] dump: Add argument to write_elfxx_notes, Qiao Nuohan, 2014/01/05
- Re: [Qemu-devel] [PATCH v6 00/11] Make 'dump-guest-memory' dump in kdump-compressed format, Qiao Nuohan, 2014/01/07