Hi Paul,
After merging the rcu tree, today's linux-next build (powerpc64
allnoconfig) failed like this:
In file included from arch/powerpc/include/asm/kvm_ppc.h:30:0,
from arch/powerpc/kernel/setup_64.c:68:
include/linux/kvm_host.h:366:21: error: field 'srcu' has incomplete type
struct srcu_struct srcu;
^
include/linux/kvm_host.h:367:21: error: field 'irq_srcu' has incomplete type
struct srcu_struct irq_srcu;
^
Presumably caused by commit 6e8ef258b669 ("srcu: Isolate srcu sections
using CONFIG_SRCU").
I have reverted that commit again for today - more work required ...
--
Cheers,
Stephen Rothwell [email protected]
On 12/10/2014 03:09 AM, Stephen Rothwell wrote:
> Hi Paul,
>
> After merging the rcu tree, today's linux-next build (powerpc64
> allnoconfig) failed like this:
>
> In file included from arch/powerpc/include/asm/kvm_ppc.h:30:0,
> from arch/powerpc/kernel/setup_64.c:68:
> include/linux/kvm_host.h:366:21: error: field 'srcu' has incomplete type
> struct srcu_struct srcu;
> ^
> include/linux/kvm_host.h:367:21: error: field 'irq_srcu' has incomplete type
> struct srcu_struct irq_srcu;
> ^
>
> Presumably caused by commit 6e8ef258b669 ("srcu: Isolate srcu sections
> using CONFIG_SRCU").
>
> I have reverted that commit again for today - more work required ...
>
Looks like there are other places which use KVM unconditionally. I will try to look and fix such uses.
Paul, can you drop the patch until then? Thanks!
--
Pranith
On Wed, Dec 10, 2014 at 10:03:57AM -0500, Pranith Kumar wrote:
> On 12/10/2014 03:09 AM, Stephen Rothwell wrote:
> > Hi Paul,
> >
> > After merging the rcu tree, today's linux-next build (powerpc64
> > allnoconfig) failed like this:
> >
> > In file included from arch/powerpc/include/asm/kvm_ppc.h:30:0,
> > from arch/powerpc/kernel/setup_64.c:68:
> > include/linux/kvm_host.h:366:21: error: field 'srcu' has incomplete type
> > struct srcu_struct srcu;
> > ^
> > include/linux/kvm_host.h:367:21: error: field 'irq_srcu' has incomplete type
> > struct srcu_struct irq_srcu;
> > ^
> >
> > Presumably caused by commit 6e8ef258b669 ("srcu: Isolate srcu sections
> > using CONFIG_SRCU").
> >
> > I have reverted that commit again for today - more work required ...
> >
>
> Looks like there are other places which use KVM unconditionally. I will try to look and fix such uses.
>
> Paul, can you drop the patch until then? Thanks!
Done!
Thanx, Paul