lkml.org 
[lkml]   [2008]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] add typecheck on irqsave and friends for correct flags
    On Thu, 1 May 2008 21:16:48 -0400 (EDT) Steven Rostedt <rostedt@goodmis.org> wrote:

    >
    > On Thu, 1 May 2008, Andrew Morton wrote:
    > > > This patch adds a typecheck inside the irqsave and restore functions
    > > > to flag these cases.
    > >
    > > hm, not exactly a thing of beauty, but it could have been worse.
    >
    > Beauty is in the eye of the beholder.
    > (a mother always thinks their kid is beautiful ;-).

    As long as your patch doesn't soil its diaper.

    > >
    > >
    > > If we had implemeted these things properly, as
    > >
    > > unsigned long spin_lock_irqsave(spinlock_t *lock);
    >
    > Then we would have been forced to do the
    >
    > irqflags_t spin_lock_irqsave(spinlock_t *lock);
    >
    > trick. Which would probably be the better long term solution, but the
    > biggest PITA for you in the short term.

    We should have done that from day one. If there's an architecture out
    there which cannot fit its interrupt state into its unsigned long (hard to
    believe) then it'll need to pull stunts with cookies and lookups or
    something. And on 64-bit architectures we're using 4 more bytes of local
    storage than is needed.

    But I don't think we've had enough problems with this particular issue to
    justify a kernel-wide edit like that. And this patch should settle the
    issue.




    \
     
     \ /
      Last update: 2008-05-02 03:53    [W:0.024 / U:30.304 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site