Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751883Ab3IMQfl (ORCPT ); Fri, 13 Sep 2013 12:35:41 -0400 Received: from mail-pd0-f178.google.com ([209.85.192.178]:35302 "EHLO mail-pd0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751201Ab3IMQfj (ORCPT ); Fri, 13 Sep 2013 12:35:39 -0400 Date: Fri, 13 Sep 2013 09:35:35 -0700 From: Guenter Roeck To: Geert Uytterhoeven Cc: "linux-kernel@vger.kernel.org" , linux-btrfs , Chris Mason , Mark Fasheh , Zach Brown Subject: Re: Build failures due to commit 416161db (btrfs: offline dedupe) Message-ID: <20130913163535.GA5681@roeck-us.net> References: <20130913133314.GA20461@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1465 Lines: 34 On Fri, Sep 13, 2013 at 03:52:43PM +0200, Geert Uytterhoeven wrote: > On Fri, Sep 13, 2013 at 3:33 PM, Guenter Roeck wrote: > > fs/btrfs/ioctl.c: In function 'btrfs_ioctl_file_extent_same': > > fs/btrfs/ioctl.c:2802:3: error: implicit declaration of function '__put_user_unaligned' [-Werror=implicit-function-declaration] > > cc1: some warnings being treated as errors > > make[2]: *** [fs/btrfs/ioctl.o] Error 1 > > make[2]: *** Waiting for unfinished jobs.... > > > > Seen with alpha:allmodconfig, arm:allmodconfig, m68k:allmodconfig, and > > xtensa:allmodconfig. > > Known issue, cfr. my early warning 10 days ago: > > "Btrfs is the first user of __put_user_unaligned() outside the compat code, > hence now all 32-bit architectures should make sure to implement this, too." > > http://marc.info/?l=linux-arch&m=137820065929216&w=2 > > and today's thread https://lkml.org/lkml/2013/9/12/814 > It doesn't seem right that a patch breaks the build for several platforms, and the problem is then blamed on the platform code instead of the code that is introducing the problem. Maybe we should add BROKEN to the btrfs dependencies for the affected platforms. After all, it _is_ broken. Guenter -- 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/