lkml.org 
[lkml]   [2015]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v2 2/6] vfio: platform: reset: add vfio_platform_reset_private.h
    Date
    On Thursday 22 October 2015 11:41:58 Eric Auger wrote:
    > v2: creation
    > - this defines the module_vfio_reset_handler macro as suggested by Arnd
    >
    > Although Arnd suggested me to remove the vfio_platform_register_reset
    > symbol_get (since the module manager can handle the case where the
    > vfio-platform driver is not loaded), I prefered to keep it while
    > introducing the macro. The rationale is, when using symbol_get/put
    > we are able to release the hold from the reset module on vfio-platform
    > as soon as the registration is complete and I think this makes sense.

    This makes it highly unusual. I can't think of a good reason to allow
    unloading the vfio platform module while a reset module is registered,
    that just causes a memory leak (or possibly a crash) when you do
    unload the core module while it's used by a reset driver, it
    prevents the reset module from getting loaded without loading the
    vfio module first (because the autoloading doesn't work), and it
    means we don't catch build errors in invalid configurations where you
    have only the reset driver but not the vfio driver enabled.

    Arnd


    \
     
     \ /
      Last update: 2015-10-22 12:41    [W:3.225 / U:0.632 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site