lkml.org 
[lkml]   [2020]   [Feb]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 3/6] hwrng: imx-rngc - use devres for registration
    Hi PrasannaKumar,

    Thus wrote PrasannaKumar Muralidharan (prasannatsmkumar@gmail.com):

    > After imx_rngc_remove function hwrng_unregister will get called. This
    > leaves a window where the clock to rng hardware block is disabled but
    > still user space can access it via /dev/hwrng.

    thanks for spotting this issue. I see that in __device_release_driver,
    the driver's remove function is called before the devres cleanup.

    > This does not look right, please revisit the patch.

    I checked again how other hwrng drivers use devres. Some don't have to
    disable a clock and need no remove function at all. Others enable the
    clock in the hwrng init routine and disable it in the cleanup routine.

    Both of these approaches don't work here. I should disable the clock
    eventually and I need it in the probe function to run the selftest
    before hwrng init is called.

    Therefore, I suggest to drop this patch, at least for the moment.
    Herbert, should I resend the series without this patch or is it ok for
    you to take the remaining patches as-is?

    BTW, 3e75241be808 ("hwrng: drivers - Use device-managed registration
    API") makes the same change that I proposed here for a couple of other
    hwrng drivers and seems to introduce the same race condition in som
    drivers e.g. drivers/char/hw_random/exynos-trng.c. Should we try to fix
    this?

    Thanks,
    Martin

    \
     
     \ /
      Last update: 2020-02-17 10:37    [W:3.287 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site