lkml.org 
[lkml]   [2012]   [Apr]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: vmevent: question?
    From
    Hi Minchan,

    On Mon, Apr 30, 2012 at 10:06 AM, Minchan Kim <minchan@kernel.org> wrote:
    > vmevent_smaple gathers all registered values to report to user if vmevent match.
    > But the time gap between vmevent match check and vmevent_sample_attr could make error
    > so user could confuse.
    >
    > Q 1. Why do we report _all_ registered vmstat value?
    >     In my opinion, it's okay just to report _a_ value vmevent_match happens.

    It makes the userspace side simpler for "lowmem notification" use
    case. I'm open to changing the ABI if it doesn't make the userspace
    side too complex.

    > Q 2. Is it okay although value when vmevent_match check happens is different with
    >     vmevent_sample_attr in vmevent_sample's for loop?
    >     I think it's not good.

    Yeah, that's just silly and needs fixing.

    > Q 3. Do you have any plan to change getting value's method?
    >     Now it's IRQ context so we have limitation to get a vmstat values so that
    >     It couldn't be generic. IMHO, To merge into mainline, we should solve this problem.

    Yes, that needs fixing as well. I was hoping to reuse perf sampling
    code for this.

    > Q 4. Do you have any plan for this patchset to merge into mainline?

    Yes, I'm interested in pushing it forward if we can show that the ABI
    makes sense, is stable and generic enough, and fixes real world
    problems.

    Pekka
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-04-30 10:01    [W:0.037 / U:1.224 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site