From owner-svn-src-all@FreeBSD.ORG Tue Aug 13 17:12:24 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 24B78655; Tue, 13 Aug 2013 17:12:24 +0000 (UTC) (envelope-from sfourman@gmail.com) Received: from mail-vb0-x235.google.com (mail-vb0-x235.google.com [IPv6:2607:f8b0:400c:c02::235]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id A0F2A2593; Tue, 13 Aug 2013 17:12:23 +0000 (UTC) Received: by mail-vb0-f53.google.com with SMTP id i3so6857013vbh.12 for ; Tue, 13 Aug 2013 10:12:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=EOQlYQLRcedVDkw3RUlB6m4G92gVDXVl3/w3sDtjhMY=; b=O/ExjfqUcYKNPktQWv1X+YiaL/OV42HDTblxnKAgyy4pSj3GgkHf2jpPeTz3Gub8Ma KVqsof1zxtjSJciIK+JfablCfw0WgCIZwCTPN1KGGZjedAAbknpROAcpREzI3i3goe/T GHVxRNHOqw9Xa7MV5Y3hp+qy60q3xW3IbZSPByzMEK8Ga1cSHklKpjwWqG3IxWHiQAYv uUtlHBRjTVHVUYOFKnYjIgAyfNJ5XXeEnBj6o9Cz8SRa1eFrhQzVL6MuirX9eA4bTQy7 jye8/o6V1K1eSN5CKvtNZxD32Wmmhe0cRwcUKFVxJbSSHv9+y+GvKNLVCMMnQOBqxFIK rJjw== MIME-Version: 1.0 X-Received: by 10.58.208.130 with SMTP id me2mr5319412vec.13.1376413941622; Tue, 13 Aug 2013 10:12:21 -0700 (PDT) Received: by 10.220.96.78 with HTTP; Tue, 13 Aug 2013 10:12:21 -0700 (PDT) In-Reply-To: <201308131540.r7DFehRl071120@svn.freebsd.org> References: <201308131540.r7DFehRl071120@svn.freebsd.org> Date: Tue, 13 Aug 2013 13:12:21 -0400 Message-ID: Subject: Re: svn commit: r254283 - head/sys/fs/ext2fs From: "Sam Fourman Jr." To: "Pedro F. Giffuni" Content-Type: text/plain; charset=ISO-8859-1 Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 Aug 2013 17:12:24 -0000 On Tue, Aug 13, 2013 at 11:40 AM, Pedro F. Giffuni wrote: > Author: pfg > Date: Tue Aug 13 15:40:43 2013 > New Revision: 254283 > URL: http://svnweb.freebsd.org/changeset/base/254283 > > Log: > Define ext2fs local types and use them. > > Add definitions for e2fs_daddr_t, e4fs_daddr_t in addition > to the already existing e2fs_lbn_t and adjust them for ext4. > Other than making the code more readable these changes should > fix problems related to big filesystems. > > Setting the proper types can be tricky so the process was > helped by looking at UFS. In our implementation, logical block > numbers can be negative and the code depends on it. In ext2, > block numbers are unsigned so it is convenient to keep > e2fs_daddr_t unsigned and use the complete 32 bits. In the > case of e4fs_daddr_t, while the value should be unsigned, for > ext4 we only need to support 48 bits so preserving an extra > bit from the sign is not an issue. > > While here also drop the ext2_setblock() prototype that was > never used. > > Discussed with: mckusick, bde > MFC after: 3 weeks > I think this change broke the build..... root@Border:/usr/src # svnlite info Path: . Working Copy Root Path: /usr/src URL: http://svn0.us-west.freebsd.org/base/head Relative URL: ^/head Repository Root: http://svn0.us-west.freebsd.org/base Repository UUID: ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f Revision: 254284 Node Kind: directory Schedule: normal Last Changed Author: pfg Last Changed Rev: 254283 Last Changed Date: 2013-08-13 15:40:43 +0000 (Tue, 13 Aug 2013) ssing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -c /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_inode_cnv.c --- ext2_extents.o --- ctfconvert -L VERSION -g ext2_extents.o --- ext2_lookup.o --- --- ext2_subr.o --- --- ext2_htree.o --- ctfconvert -L VERSION -g ext2_htree.o --- ext2_lookup.o --- cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include /usr/obj/usr/src/sys/BORDER/opt_global.h -I. -I@ -I@/contrib/altq -fno-common -g -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -I/usr/obj/usr/src/sys/BORDER -mno-aes -mno-avx -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fstack-protector -std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -c /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_lookup.c --- ext2_subr.o --- cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include /usr/obj/usr/src/sys/BORDER/opt_global.h -I. -I@ -I@/contrib/altq -fno-common -g -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -I/usr/obj/usr/src/sys/BORDER -mno-aes -mno-avx -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fstack-protector -std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -c /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_subr.c --- ext2_vfsops.o --- cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include /usr/obj/usr/src/sys/BORDER/opt_global.h -I. -I@ -I@/contrib/altq -fno-common -g -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -I/usr/obj/usr/src/sys/BORDER -mno-aes -mno-avx -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fstack-protector -std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -c /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_vfsops.c --- ext2_subr.o --- /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_subr.c:154:4: error: format specifies type 'int' but the argument has type 'e4fs_daddr_t' (aka 'long') [-Werror,-Wformat] start, last, (long long)ep->b_blkno, ^~~~~ /usr/src/sys/modules/ext2fs/../../fs/ext2fs/ext2_subr.c:154:11: error: format specifies type 'int' but the argument has type 'e4fs_daddr_t' (aka 'long') [-Werror,-Wformat] start, last, (long long)ep->b_blkno, ^~~~ 2 errors generated. *** [ext2_subr.o] Error code 1 make[4]: stopped in /usr/src/sys/modules/ext2fs --- ext2_inode_cnv.o --- ctfconvert -L VERSION -g ext2_inode_cnv.o --- ext2_inode.o --- ctfconvert -L VERSION -g ext2_inode.o --- ext2_alloc.o --- ctfconvert -L VERSION -g ext2_alloc.o --- ext2_lookup.o --- ctfconvert -L VERSION -g ext2_lookup.o --- ext2_vfsops.o --- ctfconvert -L VERSION -g ext2_vfsops.o 1 error make[4]: stopped in /usr/src/sys/modules/ext2fs *** Error code 2 Stop. make[3]: stopped in /usr/src/sys/modules *** [modules-all] Error code 1 make[2]: stopped in /usr/obj/usr/src/sys/BORDER 1 error make[2]: stopped in /usr/obj/usr/src/sys/BORDER *** [buildkernel] Error code 2 make[1]: stopped in /usr/src 1 error make[1]: stopped in /usr/src *** [buildkernel] Error code 2 make: stopped in /usr/src 1 error make: stopped in /usr/src root@Border:/usr/src # -- Sam Fourman Jr.