lkml.org 
[lkml]   [2022]   [Jun]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [cgroup] 3c87862ca1: WARNING:at_kernel/softirq.c:#__local_bh_enable_ip
    Hello Tadeusz.

    On Thu, Jun 09, 2022 at 07:30:41AM -0700, Tadeusz Struk <tadeusz.struk@linaro.org> wrote:
    > Are you interested in fixing this at syzbot issue all?

    The (original) syzbot report is conditioned by allocation failure that's
    unlikely under normal conditions (AFAIU). Hence I don't treat it extra
    high urgent.
    OTOH, it's interesting and it points to some disparity worth fixing --
    so I try helping (as time permits, so far I can only run the reproducers
    via the syzbot).

    > Do you have any more feedback on this?

    Ad the patch v2 with spinlock per css -- that looks like an overkill to
    me, I didn't look deeper into it.

    Ad the in-thread patch with ancestry css_get(), the ->parent ref:
    - is inc'd in init_and_link_css(),
    - is dec'd in css_free_rwork_fn()
    and thanks to ->online_cnt offlining is ordered (child before parent).

    Where does your patch dec these ancestry references? (Or why would they
    be missing in the first place?)

    Thanks for digging into this!
    Michal

    \
     
     \ /
      Last update: 2022-06-09 16:57    [W:3.190 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site