lkml.org 
[lkml]   [2008]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [RFC][PATCH 1/3] PM: Introduce new top level suspend and hibernation callbacks (rev. 4)
    Date
    On Thursday, 27 of March 2008, Benjamin Herrenschmidt wrote:
    >
    > > > There is absolutely no point getting a second struct anymore.
    > >
    > > I obviously disagree with that opinion, so please elaborate.
    >
    > Well, what does it bring you ? Why can't it be one struct ? To save
    > space in the data area ?

    Mostly, but not only that.

    There are users of 'struct pm_ops' that aren't even supposed to define the
    _noirq callbacks (device types and device classes), so I thought it would be
    better to introduce a separate _noirq struct after all.

    > I don't think it makes things much cleaner. But I won't fight a war for it,
    > now that they are clearly named differently and things like prepare/complete
    > are no longer in "noirq", it's semantically the same thing as having the
    > fields in one structure, so it's mostly cosmetic.

    Well, I'm not going to fight for having the two separate stuctures either.
    Also, it wouldn't be difficult to rearrange thigs so that all of the callbacks
    are in one structure, so if other people think it's better to do it this way,
    I'll go for it.

    Thanks,
    Rafael


    \
     
     \ /
      Last update: 2008-03-27 02:25    [W:2.535 / U:0.184 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site