lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2024101410-turf-junior-7739@gregkh>
Date: Mon, 14 Oct 2024 18:00:51 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc: shiju.jose@...wei.com, linux-edac@...r.kernel.org,
	linux-cxl@...r.kernel.org, linux-acpi@...r.kernel.org,
	linux-mm@...ck.org, linux-kernel@...r.kernel.org, bp@...en8.de,
	tony.luck@...el.com, rafael@...nel.org, lenb@...nel.org,
	mchehab@...nel.org, dan.j.williams@...el.com, dave@...olabs.net,
	dave.jiang@...el.com, alison.schofield@...el.com,
	vishal.l.verma@...el.com, ira.weiny@...el.com, david@...hat.com,
	Vilas.Sridharan@....com, leo.duran@....com, Yazen.Ghannam@....com,
	rientjes@...gle.com, jiaqiyan@...gle.com, Jon.Grimm@....com,
	dave.hansen@...ux.intel.com, naoya.horiguchi@....com,
	james.morse@....com, jthoughton@...gle.com, somasundaram.a@....com,
	erdemaktas@...gle.com, pgonda@...gle.com, duenwen@...gle.com,
	gthelen@...gle.com, wschwartz@...erecomputing.com,
	dferguson@...erecomputing.com, wbs@...amperecomputing.com,
	nifan.cxl@...il.com, tanxiaofei@...wei.com,
	prime.zeng@...ilicon.com, roberto.sassu@...wei.com,
	kangkang.shen@...urewei.com, wanghuiqiang@...wei.com,
	linuxarm@...wei.com
Subject: Re: [PATCH v13 12/18] platform: Add __free() based cleanup function
 for platform_device_put

On Mon, Oct 14, 2024 at 04:43:39PM +0100, Jonathan Cameron wrote:
> On Wed, 9 Oct 2024 13:41:13 +0100
> <shiju.jose@...wei.com> wrote:
> 
> > From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
> > 
> > Add __free() based cleanup function for platform_device_put().
> > 
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@...wei.com>
> > Signed-off-by: Shiju Jose <shiju.jose@...wei.com>
> > ---
> >  include/linux/platform_device.h | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/include/linux/platform_device.h b/include/linux/platform_device.h
> > index d422db6eec63..606533b88f44 100644
> > --- a/include/linux/platform_device.h
> > +++ b/include/linux/platform_device.h
> > @@ -232,6 +232,7 @@ extern int platform_device_add_data(struct platform_device *pdev,
> >  extern int platform_device_add(struct platform_device *pdev);
> >  extern void platform_device_del(struct platform_device *pdev);
> >  extern void platform_device_put(struct platform_device *pdev);
> > +DEFINE_FREE(platform_device_put, struct platform_device *, if (_T) platform_device_put(_T))
> >  
> >  struct platform_driver {
> >  	int (*probe)(struct platform_device *);
> 
> +CC Greg KH and Rafael.
> 
> Makes sure to include them on v14 as this needs review from a driver core point
> of view I think.

Why is this needed for a platform device?  This feels like you will have
to do more work to "keep" the reference on the normal path than you to
today to release the reference on the error path, right?  Have a pointer
to a patch that uses this?

thanks,

greg k-h

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ