[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 2 Feb 2024 18:44:01 +0100
From: Marco Pagani <marpagan@...hat.com>
To: Xu Yilun <yilun.xu@...ux.intel.com>
Cc: Moritz Fischer <mdf@...nel.org>, Wu Hao <hao.wu@...el.com>,
Xu Yilun <yilun.xu@...el.com>, Tom Rix <trix@...hat.com>,
Jonathan Corbet <corbet@....net>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Alan Tull <atull@...nsource.altera.com>, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, linux-fpga@...r.kernel.org
Subject: Re: [RFC PATCH v5 1/1] fpga: add an owner and use it to take the
low-level module's refcount
On 2024-01-30 05:31, Xu Yilun wrote:
>> +#define fpga_mgr_register_full(parent, info) \
>> + __fpga_mgr_register_full(parent, info, THIS_MODULE)
>> struct fpga_manager *
>> -fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info);
>> +__fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info,
>> + struct module *owner);
>>
>> +#define fpga_mgr_register(parent, name, mops, priv) \
>> + __fpga_mgr_register(parent, name, mops, priv, THIS_MODULE)
>> struct fpga_manager *
>> -fpga_mgr_register(struct device *parent, const char *name,
>> - const struct fpga_manager_ops *mops, void *priv);
>> +__fpga_mgr_register(struct device *parent, const char *name,
>> + const struct fpga_manager_ops *mops, void *priv, struct module *owner);
>> +
>> void fpga_mgr_unregister(struct fpga_manager *mgr);
>>
>> +#define devm_fpga_mgr_register_full(parent, info) \
>> + __devm_fpga_mgr_register_full(parent, info, THIS_MODULE)
>> struct fpga_manager *
>> -devm_fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info);
>> +__devm_fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info,
>> + struct module *owner);
>
> Add a line here. I can do it myself if you agree.
Sure, that is fine by me. I also spotted a typo in the commit log body
(in taken -> is taken). Do you want me to send a v6, or do you prefer
to fix that in place?
>
> There is still a RFC prefix for this patch. Are you ready to get it merged?
> If yes, Acked-by: Xu Yilun <yilun.xu@...el.com>
I'm ready for the patch to be merged. However, I recently sent an RFC
to propose a safer implementation of try_module_get() that would
simplify the code and may also benefit other subsystems. What do you
think?
https://lore.kernel.org/linux-modules/20240130193614.49772-1-marpagan@redhat.com/
> Next time if you think patches are ready for serious review and merge, drop
> the RFC prefix. That avoids an extra query.
Okay, I'll do it like that next time.
Thanks,
Marco
Powered by blists - more mailing lists