Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932263AbVLPNNh (ORCPT ); Fri, 16 Dec 2005 08:13:37 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932402AbVLPNNg (ORCPT ); Fri, 16 Dec 2005 08:13:36 -0500 Received: from TYO201.gate.nec.co.jp ([210.143.35.51]:27124 "EHLO tyo201.gate.nec.co.jp") by vger.kernel.org with ESMTP id S932401AbVLPNNf (ORCPT ); Fri, 16 Dec 2005 08:13:35 -0500 From: "Takashi Sato" To: Cc: , , "'Trond Myklebust'" Subject: [PATCH 3/3] Fix problems on multi-TB filesystem and file Date: Fri, 16 Dec 2005 22:11:09 +0900 Message-ID: <000201c60242$318eff70$4168010a@bsd.tnes.nec.co.jp> X-Priority: 3 (Normal) X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook, Build 10.0.6626 Importance: Normal X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.2180 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1774 Lines: 55 This fix was proposed by Trond Myklebust. He says: The type "sector_t" is heavily tied in to the block layer interface as an offset/handle to a block, and is subject to a supposedly block-specific configuration option: CONFIG_LBD. Despite this, it is used in struct kstatfs to save a couple of bytes on the stack whenever we call the filesystems' ->statfs(). So kstatfs's entries related to blocks are invalid on statfs64 for a network filesystem which has more than 2^32-1 blocks when CONFIG_LBD is disabled. The content of the patch attached to this mail is below. - struct kstatfs Change the type of following entries from sector_t to u64. f_blocks f_bfree f_bavail f_files f_ffree Any feedback and comments are welcome. Signed-off-by: Trond Myklebust Signed-off-by: Takashi Sato diff -uprN -X /home/sho/blocks/linux-2.6.15-rc5-blocks/Documentation/dontdiff linux-2.6.15-rc5-blocks/include/linux/statfs.h linux-2.6.15-rc5-kstatfs/include/linux/statfs.h --- linux-2.6.15-rc5-blocks/include/linux/statfs.h 2005-10-28 09:02:08.000000000 +0900 +++ linux-2.6.15-rc5-kstatfs/include/linux/statfs.h 2005-12-16 18:29:51.000000000 +0900 @@ -8,11 +8,11 @@ struct kstatfs { long f_type; long f_bsize; - sector_t f_blocks; - sector_t f_bfree; - sector_t f_bavail; - sector_t f_files; - sector_t f_ffree; + u64 f_blocks; + u64 f_bfree; + u64 f_bavail; + u64 f_files; + u64 f_ffree; __kernel_fsid_t f_fsid; long f_namelen; long f_frsize; -- Takashi Sato - 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/