lkml.org 
[lkml]   [2022]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[RFC PATCH v1] iio: trigger: move trig->owner init to trigger allocate() stage
    Date
    To provide a new IIO trigger to the IIO core, usually driver executes the
    following pipeline: allocate()/register()/get(). Before, IIO core assigned
    trig->owner as a pointer to the module which registered this trigger at
    the register() stage. But actually the trigger object is owned by the
    module earlier, on the allocate() stage, when trigger object is
    successfully allocated for the driver.

    This patch moves trig->owner initialization from register()
    stage of trigger initialization pipeline to allocate() stage to
    eliminate all misunderstandings and time gaps between trigger object
    creation and owner acquiring.

    Signed-off-by: Dmitry Rokosov <ddrokosov@sberdevices.ru>
    ---
    drivers/iio/industrialio-trigger.c | 46 ++++++++++++++++--------------
    include/linux/iio/iio.h | 10 +++++--
    include/linux/iio/trigger.h | 21 +++++++-------
    3 files changed, 42 insertions(+), 35 deletions(-)

    diff --git a/drivers/iio/industrialio-trigger.c b/drivers/iio/industrialio-trigger.c
    index f504ed351b3e..143c8f360499 100644
    --- a/drivers/iio/industrialio-trigger.c
    +++ b/drivers/iio/industrialio-trigger.c
    @@ -64,13 +64,10 @@ ATTRIBUTE_GROUPS(iio_trig_dev);

    static struct iio_trigger *__iio_trigger_find_by_name(const char *name);

    -int __iio_trigger_register(struct iio_trigger *trig_info,
    - struct module *this_mod)
    +int iio_trigger_register(struct iio_trigger *trig_info)
    {
    int ret;

    - trig_info->owner = this_mod;
    -
    trig_info->id = ida_simple_get(&iio_trigger_ida, 0, 0, GFP_KERNEL);
    if (trig_info->id < 0)
    return trig_info->id;
    @@ -101,7 +98,7 @@ int __iio_trigger_register(struct iio_trigger *trig_info,
    ida_simple_remove(&iio_trigger_ida, trig_info->id);
    return ret;
    }
    -EXPORT_SYMBOL(__iio_trigger_register);
    +EXPORT_SYMBOL(iio_trigger_register);

    void iio_trigger_unregister(struct iio_trigger *trig_info)
    {
    @@ -552,8 +549,9 @@ static void iio_trig_subirqunmask(struct irq_data *d)
    trig->subirqs[d->irq - trig->subirq_base].enabled = true;
    }

    -static __printf(2, 0)
    +static __printf(3, 0)
    struct iio_trigger *viio_trigger_alloc(struct device *parent,
    + struct module *this_mod,
    const char *fmt,
    va_list vargs)
    {
    @@ -581,6 +579,8 @@ struct iio_trigger *viio_trigger_alloc(struct device *parent,
    if (trig->name == NULL)
    goto free_descs;

    + trig->owner = this_mod;
    +
    trig->subirq_chip.name = trig->name;
    trig->subirq_chip.irq_mask = &iio_trig_subirqmask;
    trig->subirq_chip.irq_unmask = &iio_trig_subirqunmask;
    @@ -601,8 +601,9 @@ struct iio_trigger *viio_trigger_alloc(struct device *parent,
    }

    /**
    - * iio_trigger_alloc - Allocate a trigger
    + * __iio_trigger_alloc - Allocate a trigger
    * @parent: Device to allocate iio_trigger for
    + * @this_mod: module allocating the trigger
    * @fmt: trigger name format. If it includes format
    * specifiers, the additional arguments following
    * format are formatted and inserted in the resulting
    @@ -610,18 +611,20 @@ struct iio_trigger *viio_trigger_alloc(struct device *parent,
    * RETURNS:
    * Pointer to allocated iio_trigger on success, NULL on failure.
    */
    -struct iio_trigger *iio_trigger_alloc(struct device *parent, const char *fmt, ...)
    +struct iio_trigger *__iio_trigger_alloc(struct device *parent,
    + struct module *this_mod,
    + const char *fmt, ...)
    {
    struct iio_trigger *trig;
    va_list vargs;

    va_start(vargs, fmt);
    - trig = viio_trigger_alloc(parent, fmt, vargs);
    + trig = viio_trigger_alloc(parent, this_mod, fmt, vargs);
    va_end(vargs);

    return trig;
    }
    -EXPORT_SYMBOL(iio_trigger_alloc);
    +EXPORT_SYMBOL(__iio_trigger_alloc);

    void iio_trigger_free(struct iio_trigger *trig)
    {
    @@ -636,10 +639,11 @@ static void devm_iio_trigger_release(struct device *dev, void *res)
    }

    /**
    - * devm_iio_trigger_alloc - Resource-managed iio_trigger_alloc()
    + * __devm_iio_trigger_alloc - Resource-managed iio_trigger_alloc()
    * Managed iio_trigger_alloc. iio_trigger allocated with this function is
    * automatically freed on driver detach.
    * @parent: Device to allocate iio_trigger for
    + * @this_mod: module allocating the trigger
    * @fmt: trigger name format. If it includes format
    * specifiers, the additional arguments following
    * format are formatted and inserted in the resulting
    @@ -649,7 +653,9 @@ static void devm_iio_trigger_release(struct device *dev, void *res)
    * RETURNS:
    * Pointer to allocated iio_trigger on success, NULL on failure.
    */
    -struct iio_trigger *devm_iio_trigger_alloc(struct device *parent, const char *fmt, ...)
    +struct iio_trigger *__devm_iio_trigger_alloc(struct device *parent,
    + struct module *this_mod,
    + const char *fmt, ...)
    {
    struct iio_trigger **ptr, *trig;
    va_list vargs;
    @@ -661,7 +667,7 @@ struct iio_trigger *devm_iio_trigger_alloc(struct device *parent, const char *fm

    /* use raw alloc_dr for kmalloc caller tracing */
    va_start(vargs, fmt);
    - trig = viio_trigger_alloc(parent, fmt, vargs);
    + trig = viio_trigger_alloc(parent, this_mod, fmt, vargs);
    va_end(vargs);
    if (trig) {
    *ptr = trig;
    @@ -672,7 +678,7 @@ struct iio_trigger *devm_iio_trigger_alloc(struct device *parent, const char *fm

    return trig;
    }
    -EXPORT_SYMBOL_GPL(devm_iio_trigger_alloc);
    +EXPORT_SYMBOL_GPL(__devm_iio_trigger_alloc);

    static void devm_iio_trigger_unreg(void *trigger_info)
    {
    @@ -680,10 +686,9 @@ static void devm_iio_trigger_unreg(void *trigger_info)
    }

    /**
    - * __devm_iio_trigger_register - Resource-managed iio_trigger_register()
    + * devm_iio_trigger_register - Resource-managed iio_trigger_register()
    * @dev: device this trigger was allocated for
    * @trig_info: trigger to register
    - * @this_mod: module registering the trigger
    *
    * Managed iio_trigger_register(). The IIO trigger registered with this
    * function is automatically unregistered on driver detach. This function
    @@ -693,19 +698,18 @@ static void devm_iio_trigger_unreg(void *trigger_info)
    * RETURNS:
    * 0 on success, negative error number on failure.
    */
    -int __devm_iio_trigger_register(struct device *dev,
    - struct iio_trigger *trig_info,
    - struct module *this_mod)
    +int devm_iio_trigger_register(struct device *dev,
    + struct iio_trigger *trig_info)
    {
    int ret;

    - ret = __iio_trigger_register(trig_info, this_mod);
    + ret = iio_trigger_register(trig_info);
    if (ret)
    return ret;

    return devm_add_action_or_reset(dev, devm_iio_trigger_unreg, trig_info);
    }
    -EXPORT_SYMBOL_GPL(__devm_iio_trigger_register);
    +EXPORT_SYMBOL_GPL(devm_iio_trigger_register);

    bool iio_trigger_using_own(struct iio_dev *indio_dev)
    {
    diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h
    index 07025d6b3de1..853daa191b3c 100644
    --- a/include/linux/iio/iio.h
    +++ b/include/linux/iio/iio.h
    @@ -669,9 +669,13 @@ static inline void *iio_priv(const struct iio_dev *indio_dev)

    void iio_device_free(struct iio_dev *indio_dev);
    struct iio_dev *devm_iio_device_alloc(struct device *parent, int sizeof_priv);
    -__printf(2, 3)
    -struct iio_trigger *devm_iio_trigger_alloc(struct device *parent,
    - const char *fmt, ...);
    +
    +#define devm_iio_trigger_alloc(parent, fmt, ...) \
    + __devm_iio_trigger_alloc((parent), THIS_MODULE, (fmt), ##__VA_ARGS__)
    +__printf(3, 4)
    +struct iio_trigger *__devm_iio_trigger_alloc(struct device *parent,
    + struct module *this_mod,
    + const char *fmt, ...);
    /**
    * iio_buffer_enabled() - helper function to test if the buffer is enabled
    * @indio_dev: IIO device structure for device
    diff --git a/include/linux/iio/trigger.h b/include/linux/iio/trigger.h
    index 4a008b952710..5abec8df5a18 100644
    --- a/include/linux/iio/trigger.h
    +++ b/include/linux/iio/trigger.h
    @@ -135,16 +135,10 @@ static inline void *iio_trigger_get_drvdata(struct iio_trigger *trig)
    * iio_trigger_register() - register a trigger with the IIO core
    * @trig_info: trigger to be registered
    **/
    -#define iio_trigger_register(trig_info) \
    - __iio_trigger_register((trig_info), THIS_MODULE)
    -int __iio_trigger_register(struct iio_trigger *trig_info,
    - struct module *this_mod);
    +int iio_trigger_register(struct iio_trigger *trig_info);

    -#define devm_iio_trigger_register(dev, trig_info) \
    - __devm_iio_trigger_register((dev), (trig_info), THIS_MODULE)
    -int __devm_iio_trigger_register(struct device *dev,
    - struct iio_trigger *trig_info,
    - struct module *this_mod);
    +int devm_iio_trigger_register(struct device *dev,
    + struct iio_trigger *trig_info);

    /**
    * iio_trigger_unregister() - unregister a trigger from the core
    @@ -172,8 +166,13 @@ void iio_trigger_poll_chained(struct iio_trigger *trig);

    irqreturn_t iio_trigger_generic_data_rdy_poll(int irq, void *private);

    -__printf(2, 3)
    -struct iio_trigger *iio_trigger_alloc(struct device *parent, const char *fmt, ...);
    +#define iio_trigger_alloc(parent, fmt, ...) \
    + __iio_trigger_alloc((parent), THIS_MODULE, (fmt), ##__VA_ARGS__)
    +
    +__printf(3, 4)
    +struct iio_trigger *__iio_trigger_alloc(struct device *parent,
    + struct module *this_mod,
    + const char *fmt, ...);
    void iio_trigger_free(struct iio_trigger *trig);

    /**
    --
    2.36.0
    \
     
     \ /
      Last update: 2022-06-01 19:50    [W:4.323 / U:0.168 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site