From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Tue, 09 Apr 2013 00:50:10 +0000
Subject: Re: [RFC PATCH v3 1/6] kvm: add device control API
Message-Id: <1365468610.28843.29@snotra>
List-Id: <kvm-ppc.vger.kernel.org>
References: <20130408053313.GA8932@iris.ozlabs.ibm.com>
In-Reply-To: <20130408053313.GA8932@iris.ozlabs.ibm.com> (from
	paulus@samba.org on Mon Apr  8 00:33:13 2013)
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Paul Mackerras <paulus@samba.org>
Cc: Alexander Graf <agraf@suse.de>, kvm-ppc@vger.kernel.org, kvm@vger.kernel.org

On 04/08/2013 12:33:13 AM, Paul Mackerras wrote:
> On Tue, Apr 02, 2013 at 08:57:48PM -0500, Scott Wood wrote:
> 
> [snip]
> 
> > +static int kvm_ioctl_create_device(struct kvm *kvm,
> > +				   struct kvm_create_device *cd)
> > +{
> > +	bool test = cd->flags & KVM_CREATE_DEVICE_TEST;
> > +
> > +	switch (cd->type) {
> > +	default:
> > +		return -ENODEV;
> > +	}
> > +}
> 
> This gives a compile error saying "error: unused variable `test'",
> which is fatal since this gets compiled under arch/powerpc/kvm, and we
> treat all warnings as errors there.
> 
> This still gives a compile error at the end of your series if you try
> to compile with CONFIG_KVM_MPIC=n.

Ah, right.  Will mark as __maybe_unused.

-Scott