From: Suzuki K Poulose <suzuki.poulose@arm.com> To: Kim Phillips <kim.phillips@arm.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Mathieu Poirier <mathieu.poirier@linaro.org>, Leo Yan <leo.yan@linaro.org>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Alex Williamson <alex.williamson@redhat.com>, Andrew Morton <akpm@linux-foundation.org>, David Howells <dhowells@redhat.com>, Eric Auger <eric.auger@redhat.com>, Eric Biederman <ebiederm@xmission.com>, Gargi Sharma <gs051095@gmail.com>, Geert Uytterhoeven <geert+renesas@glider.be>, Kefeng Wang <wangkefeng.wang@huawei.com>, Kirill Tkhai <ktkhai@virtuozzo.com>, Mike Rapoport <rppt@linux.vnet.ibm.com>, Oleg Nesterov <oleg@redhat.com>, Pavel Tatashin <pasha.tatashin@oracle.com>, Rik van Riel <riel@redhat.com>, Robin Murphy <robin.murphy@arm.com>, Russell King <linux@armlinux.org.uk>, Thierry Reding <treding@nvidia.com>, Todd Kjos <tkjos@google.com>, Randy Dunlap <rdunlap@infradead.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org> Subject: Re: [PATCH v4 05/14] coresight: get/put module in coresight_build/release_path Date: Thu, 7 Jun 2018 10:43:26 +0100 [thread overview] Message-ID: <30e98462-237f-df2f-39d9-f0414f52b950@arm.com> (raw) In-Reply-To: <20180606155501.704583e1412996a1a2c6fa61@arm.com> On 06/06/2018 09:55 PM, Kim Phillips wrote: > On Wed, 6 Jun 2018 10:46:36 +0100 > Suzuki K Poulose <suzuki.poulose@arm.com> wrote: > >> And while we are at this, I also realised that we hold references to the >> parent devices for each connection (via bus_find_device() from >> of_coresight_get_endpoint_device()), while parsing the platform data, >> which is never released. > > Would this fix that?: Not completely. We store the dev_name() as a reference, which itself can be free'd, when the device is gone. I have a fix for this in my next version of the DT clean up series [0], where I clean up most of the platform parsing code. [0] http://lists.infradead.org/pipermail/linux-arm-kernel/2018-June/582904.html Cheers Suzuki > > diff --git a/drivers/hwtracing/coresight/of_coresight.c b/drivers/hwtracing/coresight/of_coresight.c > index a33a92ebe74b..a43ab078c85e 100644 > --- a/drivers/hwtracing/coresight/of_coresight.c > +++ b/drivers/hwtracing/coresight/of_coresight.c > @@ -181,6 +181,8 @@ of_get_coresight_platform_data(struct device *dev, > pdata->child_names[i] = dev_name(rdev); > pdata->child_ports[i] = rendpoint.id; > > + put_device(rdev); > + > i++; > } while (ep); > }
WARNING: multiple messages have this Message-ID (diff)
From: suzuki.poulose@arm.com (Suzuki K Poulose) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 05/14] coresight: get/put module in coresight_build/release_path Date: Thu, 7 Jun 2018 10:43:26 +0100 [thread overview] Message-ID: <30e98462-237f-df2f-39d9-f0414f52b950@arm.com> (raw) In-Reply-To: <20180606155501.704583e1412996a1a2c6fa61@arm.com> On 06/06/2018 09:55 PM, Kim Phillips wrote: > On Wed, 6 Jun 2018 10:46:36 +0100 > Suzuki K Poulose <suzuki.poulose@arm.com> wrote: > >> And while we are at this, I also realised that we hold references to the >> parent devices for each connection (via bus_find_device() from >> of_coresight_get_endpoint_device()), while parsing the platform data, >> which is never released. > > Would this fix that?: Not completely. We store the dev_name() as a reference, which itself can be free'd, when the device is gone. I have a fix for this in my next version of the DT clean up series [0], where I clean up most of the platform parsing code. [0] http://lists.infradead.org/pipermail/linux-arm-kernel/2018-June/582904.html Cheers Suzuki > > diff --git a/drivers/hwtracing/coresight/of_coresight.c b/drivers/hwtracing/coresight/of_coresight.c > index a33a92ebe74b..a43ab078c85e 100644 > --- a/drivers/hwtracing/coresight/of_coresight.c > +++ b/drivers/hwtracing/coresight/of_coresight.c > @@ -181,6 +181,8 @@ of_get_coresight_platform_data(struct device *dev, > pdata->child_names[i] = dev_name(rdev); > pdata->child_ports[i] = rendpoint.id; > > + put_device(rdev); > + > i++; > } while (ep); > }
next prev parent reply other threads:[~2018-06-07 9:43 UTC|newest] Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-06-05 21:06 [PATCH v4 00/14] coresight: allow to build components as modules Kim Phillips 2018-06-05 21:06 ` Kim Phillips 2018-06-05 21:06 ` [PATCH v4 01/14] coresight: cpu_debug: minor module fixups Kim Phillips 2018-06-05 21:06 ` Kim Phillips 2018-06-06 8:25 ` Greg Kroah-Hartman 2018-06-06 8:25 ` Greg Kroah-Hartman 2018-06-05 21:06 ` [PATCH v4 02/14] coresight: use IS_ENABLED for CONFIGs that may be modules Kim Phillips 2018-06-05 21:06 ` Kim Phillips 2018-06-05 21:06 ` [PATCH v4 03/14] coresight: move shared barrier_pkt[] to coresight_priv.h Kim Phillips 2018-06-05 21:06 ` Kim Phillips 2018-06-06 8:27 ` Greg Kroah-Hartman 2018-06-06 8:27 ` Greg Kroah-Hartman 2018-06-05 21:07 ` [PATCH v4 04/14] coresight: export coresight_timeout and etm_perf_symlink Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 05/14] coresight: get/put module in coresight_build/release_path Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:14 ` Suzuki K Poulose 2018-06-05 21:14 ` Suzuki K Poulose 2018-06-06 8:24 ` Greg Kroah-Hartman 2018-06-06 8:24 ` Greg Kroah-Hartman 2018-06-06 9:46 ` Suzuki K Poulose 2018-06-06 9:46 ` Suzuki K Poulose 2018-06-06 20:55 ` Kim Phillips 2018-06-06 20:55 ` Kim Phillips 2018-06-07 8:34 ` Greg Kroah-Hartman 2018-06-07 8:34 ` Greg Kroah-Hartman 2018-06-07 9:04 ` Suzuki K Poulose 2018-06-07 9:04 ` Suzuki K Poulose 2018-06-07 9:13 ` Greg Kroah-Hartman 2018-06-07 9:13 ` Greg Kroah-Hartman 2018-06-07 9:32 ` Suzuki K Poulose 2018-06-07 9:32 ` Suzuki K Poulose 2018-06-07 9:34 ` Suzuki K Poulose 2018-06-07 9:34 ` Suzuki K Poulose 2018-06-07 9:53 ` Greg Kroah-Hartman 2018-06-07 9:53 ` Greg Kroah-Hartman 2018-06-07 10:07 ` Suzuki K Poulose 2018-06-07 10:07 ` Suzuki K Poulose 2018-06-07 17:13 ` Kim Phillips 2018-06-07 17:13 ` Kim Phillips 2018-06-07 21:10 ` Suzuki K Poulose 2018-06-07 21:10 ` Suzuki K Poulose 2018-06-07 21:40 ` Mathieu Poirier 2018-06-07 21:40 ` Mathieu Poirier 2018-06-07 21:47 ` Kim Phillips 2018-06-07 21:47 ` Kim Phillips 2018-06-07 21:59 ` Mathieu Poirier 2018-06-07 21:59 ` Mathieu Poirier 2018-06-08 9:22 ` Suzuki K Poulose 2018-06-08 9:22 ` Suzuki K Poulose 2018-06-07 9:43 ` Suzuki K Poulose [this message] 2018-06-07 9:43 ` Suzuki K Poulose 2018-06-05 21:07 ` [PATCH v4 06/14] coresight: allow stm to be built as a module Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 07/14] coresight: allow dynamic-replicator " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 08/14] coresight: allow etm3x " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 09/14] coresight: allow etm4x " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 10/14] coresight: allow etb " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 11/14] coresight: allow tpiu " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 12/14] coresight: allow tmc " Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 13/14] coresight: allow funnel and replicator drivers to be built as modules Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-05 21:07 ` [PATCH v4 14/14] coresight: allow the coresight core driver to be built as a module Kim Phillips 2018-06-05 21:07 ` Kim Phillips 2018-06-06 8:33 ` Greg Kroah-Hartman 2018-06-06 8:33 ` Greg Kroah-Hartman 2018-06-06 9:57 ` Suzuki K Poulose 2018-06-06 9:57 ` Suzuki K Poulose 2018-06-07 21:46 ` Mathieu Poirier 2018-06-07 21:46 ` Mathieu Poirier
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=30e98462-237f-df2f-39d9-f0414f52b950@arm.com \ --to=suzuki.poulose@arm.com \ --cc=akpm@linux-foundation.org \ --cc=alex.williamson@redhat.com \ --cc=alexander.shishkin@linux.intel.com \ --cc=dhowells@redhat.com \ --cc=ebiederm@xmission.com \ --cc=eric.auger@redhat.com \ --cc=geert+renesas@glider.be \ --cc=gregkh@linuxfoundation.org \ --cc=gs051095@gmail.com \ --cc=kim.phillips@arm.com \ --cc=ktkhai@virtuozzo.com \ --cc=leo.yan@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mathieu.poirier@linaro.org \ --cc=oleg@redhat.com \ --cc=pasha.tatashin@oracle.com \ --cc=rdunlap@infradead.org \ --cc=riel@redhat.com \ --cc=robin.murphy@arm.com \ --cc=rppt@linux.vnet.ibm.com \ --cc=tkjos@google.com \ --cc=treding@nvidia.com \ --cc=wangkefeng.wang@huawei.com \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.