From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 In-Reply-To: Sender: linux-btrfs-owner@vger.kernel.org List-ID: 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