lkml.org 
[lkml]   [2010]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [Ksummit-2010-discuss] checkpoint-restart: naked patch
    On 11/19/2010 05:38 PM, Alexey Dobriyan wrote:
    > On Fri, Nov 19, 2010 at 6:32 PM, Tejun Heo <tj@kernel.org> wrote:
    >> On 11/19/2010 05:27 PM, Alexey Dobriyan wrote:
    >>> On Fri, Nov 19, 2010 at 6:19 PM, Tejun Heo <tj@kernel.org> wrote:
    >>>>> The paragon of absurdity is struct task_struct::did_exec .
    >>>>
    >>>> Yeah, then go and figure how to do that in a way which would be useful
    >>>> for other purposes too instead of trying to shove the whole
    >>>> checkpointer inside the kernel. It sure would be harder but hey
    >>>> that's the way it is.
    >>>
    >>> System call for one bit? This is ridiculous.
    >>
    >> Why not just a flag in proc entry? It's a frigging single bit.
    >
    > Because /proc/*/did_exec useless to anyone but C/R (even for reading!).

    I don't think you'll need a full file. Just shove it in status or
    somewhere. Your argument is completely absurd. So, because exporting
    single bit is so horrible to everyone else, you want to shove the
    whole frigging checkpointer inside the kernel?

    > Because code is much simpler:
    >
    > tsk->did_exec = !!tsk_img->did_exec;
    > +
    > __u8 did_exec;

    Sigh, yeah, except for the horror show to create tsk_img. Your
    "paragon of absurdity" is did_exec which is only ever used to decide
    whether setpgid() should fail with -EACCES, seriously? Here's a
    thought. Ignore it for now and concentrate on more relevant problems.
    I'm fairly sure CR'd program malfunctioning over did_exec wouldn't
    mark the beginning of the end of our civilization. You gotta be
    kidding me.

    --
    tejun


    \
     
     \ /
      Last update: 2010-11-19 17:53    [W:5.973 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site