From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f47.google.com (mail-wr1-f47.google.com [209.85.221.47]) (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 0131A205AA7 for ; Wed, 5 Mar 2025 09:55:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741168509; cv=none; b=SV79ajP3nonYL0b7g2jahigD+t7NJLulRGwQoEsCpCQQg3t/R1HYtoQpwRPwKrRmcWrXyhEcH+jq8D2Nv8QR0hGCgNGArwPPPz1zmPNNJ46ADA+QmbUoOUrVZdSgB1KWUIoiByy18qhg16AzqILZc7objBCs1grdcKlYEHVrTqs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741168509; c=relaxed/simple; bh=PTcgVomCwM2FB96hrOlD05ZxuTqCJVF3mwzVAg+5LPo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=W+c7lMvh5ynPZTvXwA4c7ySloJeuK2729X2msfe3fBQQ3xQuhq+V6ijOtPe3k7u0k+SN3zMk0792N6e5g7qaQqi+yKhBYUXU39+ue2sg2OYvf6MtZN++EwOk1OXADk0Xu4vrE0pmmv7QevuYnRz+he/iCcSz4lPTtlQr6J4qAME= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=A3bhkRyp; arc=none smtp.client-ip=209.85.221.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="A3bhkRyp" Received: by mail-wr1-f47.google.com with SMTP id ffacd0b85a97d-38a8b17d7a7so3902714f8f.2 for ; Wed, 05 Mar 2025 01:55:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1741168505; x=1741773305; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ydQTMdZO9ERtZacB96542Tb/ntd2ZAMm3PztulH/kfE=; b=A3bhkRypX+HzWwhud1wdgsTdo8XuRYzQrjtF3EkiXReb1OUg7nBvkBfmW3dm7hoR4c AQQmCUKyEjonET6KnD1UpUKlLhnD8YpfdrUf0tDxIjX/2bNvwf11aKP70tUb+L/uwqIV Vhc0+jIoyVrcLz15MvP6SV+IlHY/GWjxAVtAbTP3LTagjTffRcUmPleiYnYOE09VzvFO 9G2fH4bDyHcCQ/fgXs1hvwQhxwMTVxsH/HOpxIFce47FkWuRWltMyk7r5a2Ysxw60Bol aA25IGYnYKKUBX+gpTzrFkkdSKBGgjFkhTiqAkhRGIE5mnye+2AhpndD7/oDCx7CPmxt oB6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741168505; x=1741773305; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ydQTMdZO9ERtZacB96542Tb/ntd2ZAMm3PztulH/kfE=; b=NipS+41KXeGl/CPGXOCslwThA4J9/KF19zJPl6WQfUxpflDk/YYa2nyHotYjBQhZLa i2cURdTE0kI4zJ77HdutijnMZI0FNezzld+/SOSNew6uWepkLFMEVVExoh8fgLzVeEdm SqkL4cqEuPJlCjNTFLpKzljOnOhWm6iMAMDZEfyEybUoXQ0BdaS9bIcaAJPuQrSfnu3D XV8fE/pc8KwrCMDKJnLMNwQM0ETZLxjoco4MiTfgww1D3p/TdCeZ5komiHzdVtDBPBRl tYwmRb2iYVlubd7D/986sdqlx+cABIEgzAhb+dsal9z1Bkxu6ftEZTvKn9PL+JsY5Kcp jx6A== X-Gm-Message-State: AOJu0YzaV44fpdOwSzjS9Fzq6XEFRqQhBoE+J8KUD5NcyUJY/E0vs7YD Lajp1Jk9Fjwho0rIsFdxX/AswjjI9HKj/DvTRcHlq/6o+OSd1owJ15jg8OetUTM= X-Gm-Gg: ASbGnct2YTTRMThljGLxddUdMFQ/mADLRmg5n65SRbin6R4zvSTJ2eAlCm9DxgyPcgH EdoOyQ/vQsE7gWFKgOz6btdfO+1E3oauYrCwpLZkXYxtQESygED3Nnkhd29dF9qTzmmjs0uiCRc rPtavqh2sDLD7mi1eBa5nmamAXwc+Smwh5G/ydZ6gKi6NwEYHzhRjGxE2pI650RI8WoOucDHb+z Xbh5HPqUae9th3+QMQKj0Dqz06nkT+L7WyttU4cJzDR+E4vNQBE5YpIHvsYBQuH6aS4Os7H5st8 EqjImGmxblNQzgBbrAb+gZYALjEIeoZ5f8zuMppWgSn95vDu X-Google-Smtp-Source: AGHT+IEK9mhVF3kogJq2vdC9eYsKgrFlFs3HwNqnKMY7zadk2AoV9tzmwVZnh/heIO/idk5U0kjlTA== X-Received: by 2002:a5d:6c65:0:b0:391:34:4fa9 with SMTP id ffacd0b85a97d-3911f58eb5fmr2213679f8f.0.1741168505120; Wed, 05 Mar 2025 01:55:05 -0800 (PST) Received: from [10.100.51.161] ([193.86.92.181]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-390e47b7d69sm20900121f8f.60.2025.03.05.01.55.04 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 05 Mar 2025 01:55:04 -0800 (PST) Message-ID: Date: Wed, 5 Mar 2025 10:55:03 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 0/4] Properly handle module_kobject creation To: Shyam Saini , gregkh@linuxfoundation.org Cc: linux-kernel@vger.kernel.org, linux-modules@vger.kernel.org, code@tyhicks.com, linux@rasmusvillemoes.dk, christophe.leroy@csgroup.eu, hch@infradead.org, mcgrof@kernel.org, frkaya@linux.microsoft.com, vijayb@linux.microsoft.com, linux@weissschuh.net, samitolvanen@google.com, da.gomez@samsung.com, rafael@kernel.org, dakr@kernel.org References: <20250227184930.34163-1-shyamsaini@linux.microsoft.com> Content-Language: en-US From: Petr Pavlu In-Reply-To: <20250227184930.34163-1-shyamsaini@linux.microsoft.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 2/27/25 19:49, Shyam Saini wrote: > Hi Everyone, > > This patch series fixes handling of module_kobject creation. > A driver expect module_kset list populated with its corresponding > module_kobject to create its /sys/module//drivers > directory. > > Since, > [1] commit 96a1a2412acb ("kernel/params.c: defer most of param_sysfs_init() to late_initcall time") > Call to populate module_kset list is deferred to save init time so that > external watchdog doesn't fireup on some boards and Linux can take > responsibility of feeding watchdog before it spuriously resets the > system. However, [1] this fix caused another issue i.e, consumers > of module_kset can't get related module_kobject during driver > initialisation and hence can't create their > /sys/module//drivers directory. > > Consequently, [1] breaks user-space applications for eg: DPDK, which > expects /sys/module/vfio_pci/drivers/pci:vfio-pci/new_id to be present. > > The second issue was reported and the [2] revert of [1] was > proposed. However, [2] the Revert doesn't address the original issue > reported in [1]. > > This patch series addresses both issues reported in [1] and [2]. > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=96a1a2412acb > [2] https://lore.kernel.org/lkml/20250130225803.321004-1-shyamsaini@linux.microsoft.com/ This looks ok to me. I only think the Fixes: tag should have remained solely on the last patch in the series as that is the actual fix. I'll adjust it when picking up the patches. I'm going to wait for a few days if others still would like to comment and then plan to queue this on modules-next. @Greg, could I please get an Acked-by from you on the last patch in the series as this affects the code in the driver core? -- Thanks, Petr