Re: [Qemu-devel] [PATCH v2 5/9] block: vhdx - break endian translation functions out

[Jeff Cody <jcody@redhat.com>]

On Wed, Aug 07, 2013 at 05:29:04PM +0200, Kevin Wolf wrote:
> Am 01.08.2013 um 05:23 hat Jeff Cody geschrieben:
> > This moves the endian translation functions out from the vhdx.c source,
> > into a separate source file. In addition to the previously defined
> > endian functions, new endian translation functions for log support are
> > added as well.
> > 
> > Signed-off-by: Jeff Cody <jcody@redhat.com>
> 
> > +void vhdx_log_entry_hdr_le_import(VHDXLogEntryHeader *hdr)
> > +{
> > +    assert(hdr != NULL);
> > +
> > +    le32_to_cpus(&hdr->signature);
> > +    le32_to_cpus(&hdr->checksum);
> > +    le32_to_cpus(&hdr->entry_length);
> > +    le32_to_cpus(&hdr->tail);
> > +    le64_to_cpus(&hdr->sequence_number);
> > +    le32_to_cpus(&hdr->descriptor_count);
> > +    leguid_to_cpus(&hdr->log_guid);
> > +    le64_to_cpus(&hdr->flushed_file_offset);
> > +    le64_to_cpus(&hdr->last_file_offset);
> > +}
> > +
> > +void vhdx_log_entry_hdr_le_export(VHDXLogEntryHeader *hdr)
> > +{
> > +    assert(hdr != NULL);
> > +
> > +    cpu_to_le32s(&hdr->signature);
> > +    cpu_to_le32s(&hdr->checksum);
> > +    cpu_to_le32s(&hdr->entry_length);
> > +    cpu_to_le32s(&hdr->tail);
> > +    cpu_to_le64s(&hdr->sequence_number);
> > +    cpu_to_le32s(&hdr->descriptor_count);
> > +    cpu_to_le64s(&hdr->flushed_file_offset);
> > +    cpu_to_le64s(&hdr->last_file_offset);
> > +    cpu_to_leguids(&hdr->log_guid);
> > +}
> 
> Almost as critical as Stefan's comment: Can you keep the order of fields
> consistent between import and export?
> 

Absolutely! :)