Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 26 Jul 2019 09:04:29 +0200
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Cc:     Geert Uytterhoeven <geert+renesas@glider.be>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Suzuki K Poulose <suzuki.poulose@arm.com>,
        Leo Yan <leo.yan@linaro.org>, linux-arm-msm@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [Regression] Missing device nodes for ETR, ETF and STM after
 CONFIG_UEVENT_HELPER=n
Message-ID: <20190726070429.GA15714@kroah.com>
References: <cfe09a46-462f-633a-37c2-52f8bfc0ffb2@codeaurora.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <cfe09a46-462f-633a-37c2-52f8bfc0ffb2@codeaurora.org>
User-Agent: Mutt/1.12.1 (2019-06-15)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Jul 26, 2019 at 11:49:19AM +0530, Sai Prakash Ranjan wrote:
> Hi,
> 
> When trying to test my coresight patches, I found that etr,etf and stm
> device nodes are missing from /dev.

I have no idea what those device nodes are.

> Bisection gives this as the bad commit.
> 
> 1be01d4a57142ded23bdb9e0c8d9369e693b26cc is the first bad commit
> commit 1be01d4a57142ded23bdb9e0c8d9369e693b26cc
> Author: Geert Uytterhoeven <geert+renesas@glider.be>
> Date:   Thu Mar 14 12:13:50 2019 +0100
> 
>     driver: base: Disable CONFIG_UEVENT_HELPER by default
> 
>     Since commit 7934779a69f1184f ("Driver-Core: disable /sbin/hotplug by
>     default"), the help text for the /sbin/hotplug fork-bomb says
>     "This should not be used today [...] creates a high system load, or
>     [...] out-of-memory situations during bootup".  The rationale for this
>     was that no recent mainstream system used this anymore (in 2010!).
> 
>     A few years later, the complete uevent helper support was made optional
>     in commit 86d56134f1b67d0c ("kobject: Make support for uevent_helper
>     optional.").  However, if was still left enabled by default, to support
>     ancient userland.
> 
>     Time passed by, and nothing should use this anymore, so it can be
>     disabled by default.
> 
>     Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>     Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
>  drivers/base/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> 
> Any idea on this?

That means that who ever created those device nodes is relying on udev
to do this, and is not doing the correct thing within the kernel and
using devtmpfs.

Any pointers to where in the kernel those devices are trying to be
created?

thanks,

greg k-h