Date: Wed, 9 Jul 2008 21:47:53 +0200
From: Ingo Molnar <mingo@elte.hu>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: LKML <linux-kernel@vger.kernel.org>, x86@kernel.org,
       Stephen Tweedie <sct@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>,
       Mark McLoughlin <markmc@redhat.com>, "Rafael J. Wysocki" <rjw@sisk.pl>
Subject: Re: [patch] power, xen64: fix PM_SLEEP build dependencies (was:
	Re: [PATCH 00 of 55] xen64: implement 64-bit Xen support)
Message-ID: <20080709194753.GA8224@elte.hu>
References: <patchbomb.1215554782@localhost> <20080709111219.GA24874@elte.hu> <20080709111621.GA2780@elte.hu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20080709111621.GA2780@elte.hu>
User-Agent: Mutt/1.5.18 (2008-05-17)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1032
Lines: 30


-tip testing found another PM/PM_SLEEP victim:

 arch/x86/kernel/built-in.o: In function `suspend':
 apm_32.c:(.text+0xedb5): undefined reference to `save_processor_state'
 apm_32.c:(.text+0xedd0): undefined reference to `restore_processor_state'

with this config:
 
 http://redhat.com/~mingo/misc/config-Wed_Jul__9_18_53_45_CEST_2008.bad

havent figured out yet what it's about, but the core scenario seems to 
be this combination (and only this combination) of config values:

  # CONFIG_PM is not set
  CONFIG_PM_SLEEP=y
  CONFIG_APM=y
  CONFIG_PARAVIRT=y
  CONFIG_XEN=y

( accordingly, this build failure only hit two testsystems - the others
  were able to do 250 successful builds without ever hitting this. It
  needs to be fixed nevertheless, to keep the tests going. )

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/