Re: [PATCH 3/3] vTPM: support little endian guests

["Joy M. Latten" <jmlatten@linux.vnet.ibm.com>]

Hi Ashley,
On Fri, 2015-05-08 at 17:31 -0500, Ashley Lai wrote:
> > The event log in ppc64 arch is always in big endian format. PowerPC
> > supports both little endian and big endian guests. This patch converts
> > the event log entries to guest format.
> 
> I'm a little confused here.  If this patch is to convert the event log 
> entries why are we convert in the conditional statements?  One example 
> below:
> 
> +       if (((convert_to_host_format(event->event_type) == 0) &&
> +            (convert_to_host_format(event->event_size) == 0))
> +           ||
> +           ((v + sizeof(struct tcpa_event) +
> +             convert_to_host_format(event->event_size)) > limit))
> 
Ah, ok... I see what you are saying.. description may be a bit
confusing... we are not converting the event log entries.
We are just making sure that raw integers are correctly interpreted on 
both big and little endian platforms. 
Phype sends info (including logs) in big endian. 
Thus any raw integers in that data, like event_size and 
event_type may become "garbaged" when OS is ppc64-LE. 
This will ensure raw integers are correctly interpreted for both
LE and BE platforms.

> >
> > We defined a macro to convert to guest format. In addition,
> > tpm_binary_bios_measurements_show() is modified to parse the event
> > and print each field individually.
> 
> It's nice to have human readable format but it may break existing tools 
> that parse or understand the machine readable format.  Any comments on 
> this anyone?
> 
Again, description...
We have not changed the format. We are just making sure that raw
integers are correctly interpreted on both big and little endian
platforms for vtpm.

> >>
> > diff --git a/drivers/char/tpm/tpm_eventlog.c b/drivers/char/tpm/tpm_eventlog.c
> > index e77d8c1..1b62c52 100644
> > --- a/drivers/char/tpm/tpm_eventlog.c
> > +++ b/drivers/char/tpm/tpm_eventlog.c
> > @@ -28,6 +28,11 @@
> > #include "tpm.h"
> > #include "tpm_eventlog.h"
> >
> > +#ifdef CONFIG_PPC64
> > +#define convert_to_host_format(x) be32_to_cpu(x)
> > +#else
> > +#define convert_to_host_format(x) x
> > +#endif
> 
> This can go in the header file tpm_eventlog.h
> 
> 
>