From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755315Ab3EVJD3 (ORCPT ); Wed, 22 May 2013 05:03:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:63685 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751495Ab3EVJDZ (ORCPT ); Wed, 22 May 2013 05:03:25 -0400 Date: Wed, 22 May 2013 12:03:40 +0300 From: "Michael S. Tsirkin" To: Stephen Rothwell Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: linux-next: build failure after merge of the vhost tree Message-ID: <20130522090340.GA19482@redhat.com> References: <20130522143704.3431658328f8f879f55b0910@canb.auug.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20130522143704.3431658328f8f879f55b0910@canb.auug.org.au> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 22, 2013 at 02:37:04PM +1000, Stephen Rothwell wrote: > Hi Michael, > > After merging the vhost tree, today's linux-next build (powerpc > ppc64_defconfig) failed like this: > > In file included from arch/powerpc/include/asm/uaccess.h:12:0, > from arch/powerpc/lib/checksum_wrappers_64.c:24: > include/asm-generic/uaccess.h: In function 'might_fault': > include/asm-generic/uaccess.h:36:2: error: implicit declaration of function 'segment_eq' [-Werror=implicit-function-declaration] > if (_might_fault()) > ^ > include/asm-generic/uaccess.h:36:2: error: implicit declaration of function 'get_fs' [-Werror=implicit-function-declaration] > include/asm-generic/uaccess.h:29:47: error: 'KERNEL_DS' undeclared (first use in this function) > #define _might_fault() (!segment_eq(get_fs(), KERNEL_DS) && !in_atomic()) > ^ > include/asm-generic/uaccess.h:36:6: note: in expansion of macro '_might_fault' > if (_might_fault()) > ^ > > and many, many more. > > Caused by commit d198a8acf543 ("kernel: might_fault does not imply > might_sleep"). > > I have used the version of the vhost tree from next-20130521 for today. Aha I see what happened. I fixed it and made sure the changes build and work, but didn't commit and pushed from a dirty tree. > > It was unexpected that such patches would turn up in the "vhost" tree ... > > -- > Cheers, > Stephen Rothwell sfr@canb.auug.org.au vhost wants to use these changes but these patches are still work in progress. Since the changes are all over the tree I thought looking at how they interact with everyone's work at an early stage is a good idea. Of course breaking builds for everyone is not acceptable, I'm very sorry about that. I'll make double sure by tree is clean before push next time, will also look at whether git push can check this automatically. -- MST