From: Wen Congyang <we...@cn.fujitsu.com>
Subject: [RFC][PATCH 08/14 v9] target-i386: Add API to write cpu status to core 
file
Date: Wed, 14 Mar 2012 10:09:26 +0800

> +    memset(note, 0, note_size);
> +    if (type == 0) {
> +        note32 = note;
> +        note32->n_namesz = cpu_to_le32(name_size);
> +        note32->n_descsz = cpu_to_le32(descsz);
> +        note32->n_type = 0;
> +    } else {
> +        note64 = note;
> +        note64->n_namesz = cpu_to_le32(name_size);
> +        note64->n_descsz = cpu_to_le32(descsz);
> +        note64->n_type = 0;
> +    }

Why not give new type for this note information an explicit name?
Like NT_QEMUCPUSTATE? There might be another type in the future. This
way there's also a merit that we can know all the existing notes
relevant to qemu dump by looking at the names in a header file.

Thanks.
HATAYAMA, Daisuke


Reply via email to