From: Mark Rutland <firstname.lastname@example.org> To: Russell King - ARM Linux <email@example.com> Cc: Alexander Stein <firstname.lastname@example.org>, Peter Zijlstra <email@example.com>, Ingo Molnar <firstname.lastname@example.org>, Arnaldo Carvalho de Melo <email@example.com>, Alexander Shishkin <firstname.lastname@example.org>, Will Deacon <email@example.com>, firstname.lastname@example.org, email@example.com Subject: Re: [PATCH v2 2/2] arm: perf: Mark as non-removable Date: Wed, 4 Jan 2017 11:46:13 +0000 [thread overview] Message-ID: <20170104114612.GF8329@leverpostej> (raw) In-Reply-To: <20170104114056.GO14217@n2100.armlinux.org.uk> On Wed, Jan 04, 2017 at 11:40:56AM +0000, Russell King - ARM Linux wrote: > On Wed, Jan 04, 2017 at 11:30:25AM +0000, Mark Rutland wrote: > > On Wed, Jan 04, 2017 at 10:19:46AM +0100, Alexander Stein wrote: > > > I'm not sure if the change above works with remove functions set in struct > > > bus_type too. > > > But on the other hand this would hide errors in drivers which are actually > > > removable but do not cleanup properly which DEBUG_TEST_DRIVER_REMOVE tries to > > > detect. > > > By setting .suppress_bind_attrs = true explicitely you state "This > > > driver cannot be removed!", so the remove callback is not missing by accident. > > > > I'm not sure I follow. If the remove callback is accidentally missing, > > the driver is not "actually removable" today -- there's either no remove > > code, or it's not been wired up (the latter of which will likely result > > in a compiler warning about an unused function). > > > > Aborting the remove early in those cases is much safer than forcefully > > removing a driver without a remove callback. > > Drivers without a remove function may be removable - there's more layers > than just the driver - there's the bus layer as well, which may or may > not direct to a private-bus pointer. Sure; which is why I initially suggested doing something at the bus layer. That way, each layer could do any necessary check, and/or delegate to a callback for the layer below. > There's no real way for the core driver model code to know whether the > lack of the ->remove in the struct device_driver is something that > prevents a driver being removed, or whether it's handled via some other > method. Eg, platform drivers. While true today, my suggestion is to add the infrastrucutre such that it can. That seems nicer to me than each driver having to retain (redundant) state. Regardless, this patch itself is fine. Thanks, Mark.
next prev parent reply other threads:[~2017-01-04 11:47 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-12-21 15:03 [PATCH v2 0/2] mark driver " Alexander Stein 2016-12-21 15:03 ` [PATCH v2 1/2] drivers/perf: arm_pmu: Use devm_ allocators Alexander Stein 2016-12-21 15:03 ` [PATCH v2 2/2] arm: perf: Mark as non-removable Alexander Stein 2016-12-22 22:48 ` Mark Rutland 2017-01-04 9:19 ` Alexander Stein 2017-01-04 11:30 ` Mark Rutland 2017-01-04 11:40 ` Russell King - ARM Linux 2017-01-04 11:46 ` Mark Rutland [this message] 2017-01-04 18:17 ` Will Deacon 2016-12-21 23:17 ` [PATCH v2 0/2] mark driver " Russell King - ARM Linux
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20170104114612.GF8329@leverpostej \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH v2 2/2] arm: perf: Mark as non-removable' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).