lkml.org 
[lkml]   [2020]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/2] Block layer filter - second version
    EXPORT_SYMBOL_GPL - ok.

    #ifdef CONFIG_BLK_FILTER or IS_ENABLED() - It's a matter of habit.

    > double blank line
    Ok, I did.
    Looks like a candidate for ./scripts/checkpatch.pl.

    > Separate into multiple patches: one that introduces the filter
    > functions/ops code and another that changes the block layer where needed.

    I'll think about it. Personally, it seems to me that this separation
    does not make it easier to understand the code.
    It is important for me to know immediately where the function is called,
    and this determines its behavior.

    --
    Sergei Shtepa
    Veeam Software developer.

    \
     
     \ /
      Last update: 2020-10-21 12:27    [W:6.167 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site