Received: by 2002:a05:6358:4e97:b0:b3:742d:4702 with SMTP id ce23csp566283rwb; Thu, 18 Aug 2022 08:24:26 -0700 (PDT) X-Google-Smtp-Source: AA6agR6qntn9DdIqeqaid2kXd1kIE0s3vRvwPoRulZek35QVqlCetYcP9zVyz3IFgI6Cglf4+IIu X-Received: by 2002:aa7:d88a:0:b0:440:916e:706d with SMTP id u10-20020aa7d88a000000b00440916e706dmr2607384edq.167.1660836265869; Thu, 18 Aug 2022 08:24:25 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1660836265; cv=pass; d=google.com; s=arc-20160816; b=keOaFisMnmxxT0YaSBPxvrkzTj30Q31rUkS4H2bT1bQ5xdMqZ9abAS550vVyFU2UhB iRqJilBjwE7ig0L0lNfU2V54NWVML5Wnwju8MvsUnCc5UkJya4Od/JlomHDo6hBq7ZbB i2PmpW6kdyHOUhVWFKf4priwEAGFJvaUNCvZX9falzGsKZ/BVp+zwd5zX66kJWFRlSbW wqOBn6Y61yNLeeETjU6oCmYMowXylEHUJ0vpy0TsygB+8Wbvo+icOX10myxADZHP6KPm LRxloU+hUnS5opMfzwc8X/OlPxrOZxawJdPh9KNuNCIbRYxPW0T/hgRJ67Bl5Xgf22Va 9nuQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:importance:content-transfer-encoding :mime-version:subject:references:in-reply-to:message-id:cc:to:from :date:dkim-signature; bh=VU/9NQq/tY4qE6lA3If/jM8YrDr0mazxoc1R8N5jJzM=; b=SspbVwWnIg68MXpmW8/SnqjtLpNw7xGz21X6fxWXX+Am0o/1+9yzL/WS/+BLjKgZtR lSmRYK7pLoyVuBw2K5ktdjkNDMrzhmyHGphP06Ppls/Rabujoxvpv+Nl+2ILZg2vfjbP wysPbCSk5t44c1PY0g4hVzaxAxiMqKf/jtDKPMVNihX5W6torlwRfu63wVlCwrReWQQ9 saiJr6jYReC3lyOgdYtxTd3ixciHppkYTN68Nh0ksbJicQfoU1osmIPC64+1QSgxgiJg DsQCRW++SHGD/XZdPyR6acswFZJCJPHGjb/D1DaxGn/PvpRQs+bcsuW5EkopWPsNWeB0 rIrQ== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@siddh.me header.s=zmail header.b=RMmO85lS; arc=pass (i=1 spf=pass spfdomain=siddh.me dkim=pass dkdomain=siddh.me dmarc=pass fromdomain=siddh.me>); spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=siddh.me Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id dt7-20020a170907728700b00722e51021b3si1207390ejc.74.2022.08.18.08.23.58; Thu, 18 Aug 2022 08:24:25 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@siddh.me header.s=zmail header.b=RMmO85lS; arc=pass (i=1 spf=pass spfdomain=siddh.me dkim=pass dkdomain=siddh.me dmarc=pass fromdomain=siddh.me>); spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=siddh.me Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1343601AbiHRPWF (ORCPT + 99 others); Thu, 18 Aug 2022 11:22:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37394 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245691AbiHRPWD (ORCPT ); Thu, 18 Aug 2022 11:22:03 -0400 Received: from sender-of-o50.zoho.in (sender-of-o50.zoho.in [103.117.158.50]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 39AD57539F; Thu, 18 Aug 2022 08:21:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1660836089; cv=none; d=zohomail.in; s=zohoarc; b=ANI7IPRR551PJTpRtyF/vzZcTFfEc84mHr+GpDLqExtKSLVOTV+sY42RwadRJxrSjig2PIz1/9oyCgz5rEwez47HmXxfvJtH//rJbf4yF7P3BdQVkuCPLC1GulheaMi2rb73QoeR2shRpq4ivJZ0JgeqGfB/TK9oK/u0Q0LECTk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.in; s=zohoarc; t=1660836089; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:To; bh=VU/9NQq/tY4qE6lA3If/jM8YrDr0mazxoc1R8N5jJzM=; b=Gm4TZbzpKIAdX5WgaCIyI3RaM2qsBnQa2OrbXs99nBA5w4Xz5BQpNk+zUQHXfJTdPL+WFIF4FJIPAU7pT3sGIab9e0OoJ2r4yC4XwQmAis2awbxmm8tXrMVdfTggiKgz5QezJz1dx1KzyfGv8xUeK1/BbjCtn4mKPvJdFQxK53M= ARC-Authentication-Results: i=1; mx.zohomail.in; dkim=pass header.i=siddh.me; spf=pass smtp.mailfrom=code@siddh.me; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1660836089; s=zmail; d=siddh.me; i=code@siddh.me; h=Date:Date:From:From:To:To:Cc:Cc:Message-ID:In-Reply-To:References:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Reply-To; bh=VU/9NQq/tY4qE6lA3If/jM8YrDr0mazxoc1R8N5jJzM=; b=RMmO85lSetrrrFF2xKPs/f6gejNaB5fSXzegfG46VK5pASIAXAWgJTmJl/TbEZyW SYieXMe1kcpvE9mG1rek2hKz/TFiAC6pHbIRMeyKfcilX6BPzCnBuP24yAMYhN8ROKY 6duj9GjcU7N5BttFAwNM6og9uNbI0ScOme2elwzk= Received: from mail.zoho.in by mx.zoho.in with SMTP id 1660836076966185.1731467950858; Thu, 18 Aug 2022 20:51:16 +0530 (IST) Date: Thu, 18 Aug 2022 20:51:16 +0530 From: Siddh Raman Pant To: "Matthew Wilcox" Cc: "david" , "djwong" , "fgheet255t" , "hch" , "linux-ext4" , "linux-fsdevel" , "linux-kernel" , "linux-xfs" , "riteshh" , "syzbot+a8e049cd3abd342936b6" , "syzkaller-bugs" Message-ID: <182b18b5d92.7a2e2b1623166.1514589417142553905@siddh.me> In-Reply-To: References: <20220818110031.89467-1-code@siddh.me> <20220818111117.102681-1-code@siddh.me> Subject: Re: [syzbot] WARNING in iomap_iter MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Importance: Medium User-Agent: Zoho Mail X-Mailer: Zoho Mail X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org On Thu, 18 Aug 2022 20:20:02 +0530 Matthew Wilcox wrote: > I don't think changing these from u64 to s64 is the right way to go. Why do you think so? Is there somnething I overlooked? I think it won't intorduce regression, since if something is working, it will continue to work. If something does break, then they were relying on overflows, which is anyways an incorrect way to go about. Also, it seems even the 32-bit compatibility structure uses signed types. Thanks, Siddh