From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f177.google.com (mail-pl1-f177.google.com [209.85.214.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AB622367F3A for ; Wed, 11 Mar 2026 00:47:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773190073; cv=none; b=jp4MeVzxVoskCJKZJ/egI8nA6gZZvFIHSz2PjsRZzM7bqfJtFfokgYptP/mERpOkPCDWKDk3uUgZB+gQWHfphv205MHb/SEl6MDOjSSpvqlypB6qDZ3X128KRs5uJxQWiWKR2yjHxpfJHjP/VuoqeY38D3E42CbOp6ce9thXIwU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773190073; c=relaxed/simple; bh=0NOikj0F4XRAl27fF0T51r1cDQUQLoSd+Y7hpcI8wOg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=rMuj0Ac5GN+L3uy/KMFkqgFJmpDv53EtS6pwrbE86C4rr4/WlNU0VPwezGkq/EhvGzC1uRUE8BV18/PuA3jLPvXQy3D8gVv9UnO9NqnYR3pUfwxt0VEpJ/JVnVgsnnZg8XXvHtf7nT2xPqD6cDW+AfUd1Z0D/aPfvJdR5kTgtX8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=LWTIu7B5; arc=none smtp.client-ip=209.85.214.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="LWTIu7B5" Received: by mail-pl1-f177.google.com with SMTP id d9443c01a7336-2adff872068so64704135ad.1 for ; Tue, 10 Mar 2026 17:47:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773190072; x=1773794872; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=D87ZdyQZ4IabAndi5c77q4G/EVeuF8jpmWClRpdBIok=; b=LWTIu7B5OoWEORAT/02hO5BHbXHKQOCYLsuU3xobAkU4SponXkdaOPupVcWtTPaXpd y4N2k3tJNFeWPDrnkWmXMOp2l1JlyXTZW+8xbDTnyxesys84LPXwpHcFl06eNkLQE4ok QNy2JeKSL90VcTjYaUj1fuBoejEFjDXsQsE5+dZY82COwwVwxnCC1KfCL6bCdKmUxWD/ IRe7FgMf+QR5tW/zzVoo7Nz3TnD148G5T5vW/nDw+4bKnXhurGx0SKxiwrJwn0mTRQLk UMgPIwCvA+yfyZeLuf9xBIPSCA/cUw9iYgmc4f338S2QaHsLaTwd8Qdct9qaWFDZRzfW tuTw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773190072; x=1773794872; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=D87ZdyQZ4IabAndi5c77q4G/EVeuF8jpmWClRpdBIok=; b=jlM+0Z3+lXjsZvfkRzh0p3rDvRwlyHAlYmpdgneE4cbBpUGQ03QxBpe//mIpGl97Jx FOiEIjEOcxWMCLA7VrHke5buSIj6MxL2fP4XG2ESFpqb9ejY0aFApxeB0hm3rZ5zkDbe qOui6hgUnG02aL7fVRyNwa6KmeLMyk2imCra4vL7oqFQudUNpOcjQaL4JcLAEJf1RXYu NkXeO/isjp46mfECnAqLd4VxS/1Odu2wNX83i9TO/l331wp5PP6aqySn99abxqV1EV0L eHAFqpcN7K6rJENpVa3PJ4wjeA3BKLJN3+Ex9bfL1HUzioo5kn6s7aVgh+k5F0yQf1ig cEJw== X-Forwarded-Encrypted: i=1; AJvYcCUp+6cU6TwfpvGlE41Lsnu4Bcpk1YDwal0bib6a953lcLU5/dlUoxyEOmDx/WLT7cF0GOIHp0WnUFbbSFs=@vger.kernel.org X-Gm-Message-State: AOJu0YzncBK/7W6v2/M+kiFlsM1OztPGAqIFzU5WsUzwN75q3x7qpX41 pQNQnpOf4kwe79daXtCa7TOPtlIEgJtCCl5QMCxDDF5Kz+hnDB1s8QBT X-Gm-Gg: ATEYQzzdX9ysnei/ZX4d7uhn/YODpIY+aAu2DeWWeoZ7fXtWg4tpbRFzdIfBmEzJxRE wBaLpttD8WrtCbwhWYuFIkmeWcGo5O62n/u8MP96eyvjSnckP7JjGf6xg58j1rScAqX8v0w6xV3 tdDb2AYpJnBzM3m0Fe7VARMhQnNclOpNU9W67EJFeoxjGNBAUQUjj128aDfB7H3G0caYzFcJnTp anYtlBGntWJco6z2z0dQq5uHjaLptl3nhho9vKnp+m3cWMrIbtmZbrTjU0uJSx7zFgTIA+ipdBT aFQ79sQGoe7Pmh9T0FK5zivZuVvCrrxLNqkuXnTEYfFeOWVMvQrA1cob34MoP5CfXz7OYjOcGyw QKYpTDkQNLR5zeHH7GDXTjj+ljmbhgzrAuYDqmHzg1BwO0rnS9I1xczhHRGqnz/zV5V5wZ/Hjel FkcNieCbHYSDQGS/N2dKqK97nPgoQ= X-Received: by 2002:a17:902:db06:b0:2ae:5005:a26e with SMTP id d9443c01a7336-2aeae78cb97mr7226095ad.2.1773190071971; Tue, 10 Mar 2026 17:47:51 -0700 (PDT) Received: from fedora ([209.132.188.88]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2aeae378974sm4231175ad.82.2026.03.10.17.47.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Mar 2026 17:47:51 -0700 (PDT) Date: Wed, 11 Mar 2026 00:47:41 +0000 From: Hangbin Liu To: Sabrina Dubroca Cc: syzbot ci , andrew@lunn.ch, bridge@lists.linux.dev, davem@davemloft.net, edumazet@google.com, horms@kernel.org, idosch@nvidia.com, jiri@resnulli.us, jv@jvosburgh.net, kuba@kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, pabeni@redhat.com, razor@blackwall.org, sridhar.samudrala@intel.com, syzbot@lists.linux.dev, syzkaller-bugs@googlegroups.com Subject: Re: [syzbot ci] Re: net: move netdev_compute_master_upper_features to ndo_set_features Message-ID: References: <20260310-offload_compute-v1-0-3df79c09ea65@gmail.com> <69b04e91.a70a0220.51e36.0000.GAE@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Tue, Mar 10, 2026 at 08:17:01PM +0100, Sabrina Dubroca wrote: > 2026-03-10, 10:02:09 -0700, syzbot ci wrote: > > batman_adv: batadv0: Not using interface batadv_slave_1 (retrying later): interface not active > > hsr_slave_0: entered promiscuous mode > > hsr_slave_1: entered promiscuous mode > > ------------[ cut here ]------------ > > err == -EMSGSIZE > > WARNING: net/core/rtnetlink.c:4421 at rtmsg_ifinfo_build_skb+0x218/0x260, CPU#0: syz-executor/6496 > > I'm not sure this one is caused by this series, but either way, rtnetlink_event+0x1b7/0x270 notifier_call_chain+0x1be/0x400 netdev_change_features+0x95/0xe0 __netdev_upper_dev_link+0xb20/0xc80 netdev_upper_dev_link+0xb0/0x100 This patch calls netdev_change_features() after __netdev_upper_dev_link(), Which trigger a NETDEV_FEAT_CHANGE notify and calls rtmsg_ifinfo_event() to fill the new link info. Maybe the event is a bit early and macsec has data not ready? Thanks Hangbin > reviewing if_nlmsg_size/rtnl_fill_ifinfo for mismatches is really > unpleasant :/ > > Things I see in rtnl_fill_ifinfo but don't find in if_nlmsg_size: > - IFLA_PARENT_DEV_NAME > - IFLA_PARENT_DEV_BUS_NAME > (both from 00e77ed8e64d ("rtnetlink: add > IFLA_PARENT_[DEV|DEV_BUS]_NAME"), which doesn't include a change to > if_nlmsg_size) > - rtnl_link_slave_info_fill also outputs IFLA_INFO_SLAVE_KIND + the > IFLA_INFO_SLAVE_DATA nest, but rtnl_link_get_slave_info_data_size > only counts the nest, and its caller (rtnl_link_get_size) doesn't > have anything more about the slave info. This may be what syzbot is > tripping on here. > > > But there's a > > + nla_total_size(4) /* IFLA_WEIGHT */ > > that doesn't get filled anywhere. > > > > Modules linked in: > > CPU: 0 UID: 0 PID: 6496 Comm: syz-executor Not tainted syzkaller #0 PREEMPT(full) > > Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS 1.16.2-debian-1.16.2-1 04/01/2014 > > RIP: 0010:rtmsg_ifinfo_build_skb+0x218/0x260 > > Code: f6 ba 01 00 00 00 89 e9 e8 25 15 3a 00 4c 89 f0 48 83 c4 30 5b 41 5c 41 5d 41 5e 41 5f 5d e9 7f 3a 2e 02 cc e8 49 3b 42 f8 90 <0f> 0b 90 eb 90 89 d9 80 e1 07 fe c1 38 c1 0f 8c 95 fe ff ff 48 89 > > RSP: 0018:ffffc9000637e9a0 EFLAGS: 00010293 > > RAX: ffffffff89835e27 RBX: 0000000000000000 RCX: ffff8881b80a57c0 > > RDX: 0000000000000000 RSI: 00000000ffffffa6 RDI: 00000000ffffffa6 > > RBP: 00000000ffffffa6 R08: 0000000000000004 R09: 0000000000000004 > > R10: fffff52000c6fcdc R11: 0000000000000000 R12: 1ffff110235ddc21 > > R13: 0000000000000000 R14: ffff8881133dc780 R15: ffff88811aeee000 > > FS: 0000555557c4a500(0000) GS:ffff88818de65000(0000) knlGS:0000000000000000 > > CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 > > CR2: 000055555e1838c8 CR3: 0000000168b80000 CR4: 00000000000006f0 > > Call Trace: > > > > rtnetlink_event+0x1b7/0x270 > > notifier_call_chain+0x1be/0x400 > > netdev_change_features+0x95/0xe0 > > __netdev_upper_dev_link+0xb20/0xc80 > > netdev_upper_dev_link+0xb0/0x100 > > macsec_newlink+0xb11/0x1200 > > rtnl_newlink_create+0x329/0xb70 > > rtnl_newlink+0x1666/0x1be0 > > -- > Sabrina