From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Tinguely Subject: Re: new 64 bit math link fail in xfs in linux-next today Date: Wed, 16 Apr 2014 15:59:51 -0500 Message-ID: <534EEF47.4000706@sgi.com> References: <534EAB41.60901@windriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <534EAB41.60901@windriver.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Paul Gortmaker Cc: "linux-next@vger.kernel.org" , xfs@oss.sgi.com List-Id: linux-next.vger.kernel.org On 04/16/14 11:09, Paul Gortmaker wrote: > Hi Dave, > > Not sure if this has been reported yet, but this new failure showed > up in the xtensa link of today's linux-next builds. > > http://kisskb.ellerman.id.au/kisskb/buildresult/10938384/ > > Paul. > Try this works on i386. sorry if my mailer wraps the line: The roundup in commit 68c1fb5d should be a roundup_64() because it is desired to round a 64 bit type by an integer and that will result in a 64 bit value. On 32 bit machines using roundup() in this case will result in the error: ERROR: "__divdi3" [fs/xfs/xfs.ko] undefined! Signed-off-by: Mark Tinguely --- fs/xfs/xfs_aops.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: b/fs/xfs/xfs_aops.c =================================================================== --- a/fs/xfs/xfs_aops.c +++ b/fs/xfs/xfs_aops.c @@ -1365,7 +1365,7 @@ __xfs_get_blocks( if (offset < i_size_read(inode) && offset + mapping_size >= i_size_read(inode)) { /* limit mapping to block that spans EOF */ - mapping_size = roundup(i_size_read(inode) - offset, + mapping_size = roundup_64(i_size_read(inode) - offset, 1 << inode->i_blkbits); } if (mapping_size > LONG_MAX) _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs