From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B7420946F for ; Thu, 20 Apr 2023 23:21:40 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 18A60C433EF; Thu, 20 Apr 2023 23:21:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1682032900; bh=UWWS88ckAbbbv7dT9lyPBDOyOQ1c5gbIG2dd4Nr6Q+0=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=B5DM14YjM78BDBqq7yEgPATkm4MtgUek6GaOMEfGK9Pjp+V0YWlkOduUZRjSm17uE eAqt2B4v+jubZW/DSCeFbDawX9Xt09JIi0mGusdOI9On+ffZKoP8vM7K4I3uy/mjjk G4festEsXoPPfRZhhPa6cnPxjORqpDhTr+XKMjCYsHvwYKPeILb8M5uQtlbqw5kg9D BIla9vjLuYOZLA4ThEhVTE7cjhESKy4TQJhcr3Cjedv3brQ1sc2U5vBQgABK2JfRqn kV3DGCiHHgEJyqAGZ9rodOnBisLmwX8EPoXeAH4xug+ehdy9o888LybrDN+Z4HCJOh HxMlvfwU96ocw== Date: Thu, 20 Apr 2023 16:21:39 -0700 From: Jakub Kicinski To: Jay Vosburgh Cc: kernel test robot , Hangbin Liu , netdev@vger.kernel.org, oe-kbuild-all@lists.linux.dev, "David S . Miller" , Paolo Abeni , Eric Dumazet , Liang Li , Vincent Bernat Subject: Re: [PATCH net 1/4] bonding: fix send_peer_notif overflow Message-ID: <20230420162139.3926e85c@kernel.org> In-Reply-To: <27709.1682006380@famine> References: <20230420082230.2968883-2-liuhangbin@gmail.com> <202304202222.eUq4Xfv8-lkp@intel.com> <27709.1682006380@famine> Precedence: bulk X-Mailing-List: oe-kbuild-all@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Thu, 20 Apr 2023 08:59:40 -0700 Jay Vosburgh wrote: > >All errors (new ones prefixed by >>, old ones prefixed by <<): > > > >>> ERROR: modpost: "__umoddi3" [drivers/net/bonding/bonding.ko] undefined! > > I assume this is related to send_peer_notif now being u64 in the > modulus at: > > static bool bond_should_notify_peers(struct bonding *bond) > { > [...] > if (!slave || !bond->send_peer_notif || > bond->send_peer_notif % > max(1, bond->params.peer_notif_delay) != 0 || > > but I'm unsure if this is a real coding error, or some issue > with the parisc arch specifically? Coding error, I think. An appropriate helper from linux/math64.h should be used.