[07/12] configure: fix TPM logic

A non-native i386 or x86_64 emulator should not have TPM passthrough
support, since the TPM is only present for those hosts.

Reviewed-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 configure                          | 24 ++++++++++++++++++------
 default-configs/i386-softmmu.mak   |  3 +--
 default-configs/x86_64-softmmu.mak |  3 +--
 tpm/Makefile.objs                  |  4 +---
 4 files changed, 21 insertions(+), 13 deletions(-)

On 04/15/13 15:19, Paolo Bonzini wrote:

> diff --git a/configure b/configure
> index 258c82a..1c1e369 100755
> --- a/configure
> +++ b/configure

> @@ -3436,6 +3445,7 @@ echo "virtio-blk-data-plane $virtio_blk_data_plane"
>  echo "gcov              $gcov_tool"
>  echo "gcov enabled      $gcov"
>  echo "TPM support       $tpm"
> +echo "TPM passthrough   $tpm_passthrough"
>  
>  if test "$sdl_too_old" = "yes"; then
>  echo "-> Your SDL version is too old - please upgrade to have SDL support"

(tiny merge conflict with 0a12ec87)

diff --cc configure
index a97bf31,1c1e369..0000000
--- a/configure
+++ b/configure
@@@ -3503,7 -3445,7 +3513,8 @@@ echo "virtio-blk-data-plane $virtio_blk
  echo "gcov              $gcov_tool"
  echo "gcov enabled      $gcov"
  echo "TPM support       $tpm"
+ echo "TPM passthrough   $tpm_passthrough"
 +echo "libssh2 support   $libssh2"

  if test "$sdl_too_old" = "yes"; then
  echo "-> Your SDL version is too old - please upgrade to have SDL support"

On 15 April 2013 14:19, Paolo Bonzini <pbonzini@redhat.com> wrote:
> +if test "$targetos" = Linux && test "$cpu" = i386 -o "$cpu" = x86_64; then

test -o is deprecated by POSIX; better to use
  if test ... && ( test ... || test ... ); then

in new code.

thanks
-- PMM

Il 15/04/2013 18:09, Peter Maydell ha scritto:
>> > +if test "$targetos" = Linux && test "$cpu" = i386 -o "$cpu" = x86_64; then
> test -o is deprecated by POSIX; better to use
>   if test ... && ( test ... || test ... ); then
> 
> in new code.

True, on the other hand "(.*test" has no match at all in configure; and
since we know that $cpu does not begin with a dash, it is portable in
practice.

I would agree with you, but it looks like convention trumps the
suggested practice.

Paolo

On 15 April 2013 17:14, Paolo Bonzini <pbonzini@redhat.com> wrote:
> Il 15/04/2013 18:09, Peter Maydell ha scritto:
>>> > +if test "$targetos" = Linux && test "$cpu" = i386 -o "$cpu" = x86_64; then
>> test -o is deprecated by POSIX; better to use
>>   if test ... && ( test ... || test ... ); then
>>
>> in new code.
>
> True, on the other hand "(.*test" has no match at all in configure; and
> since we know that $cpu does not begin with a dash, it is portable in
> practice.
>
> I would agree with you, but it looks like convention trumps the
> suggested practice.

I've been consistently saying this in code review for
all new shell code for some time now. You just happen
to have the first example of X && (Y || Z); most
of configure's conditionals are not that complex.

-- PMM

Paolo Bonzini <pbonzini@redhat.com> writes:

> A non-native i386 or x86_64 emulator should not have TPM passthrough
> support, since the TPM is only present for those hosts.
>
> Reviewed-by: Andreas Färber <afaerber@suse.de>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Breaks the build for me:

make: Entering directory `/work/armbru/qemu/bld-x86'
make[1]: *** No rule to make target `../tpm/tpm_tis.o', needed by `qemu-system-x86_64'.
make[1]: Target `all' not remade because of errors.
make: *** [subdir-x86_64-softmmu] Error 2
make: Target `all' not remade because of errors.
make: Leaving directory `/work/armbru/qemu/bld-x86'

Workaround: drop --enable-tpm.

Il 16/04/2013 10:28, Markus Armbruster ha scritto:
> Breaks the build for me:
> 
> make: Entering directory `/work/armbru/qemu/bld-x86'
> make[1]: *** No rule to make target `../tpm/tpm_tis.o', needed by `qemu-system-x86_64'.
> make[1]: Target `all' not remade because of errors.
> make: *** [subdir-x86_64-softmmu] Error 2
> make: Target `all' not remade because of errors.
> make: Leaving directory `/work/armbru/qemu/bld-x86'
> 
> Workaround: drop --enable-tpm.

I had a stale .o in my build directory.  Sending patch, BTW why isn't
TPM enabled by default?

Paolo