Re: [PATCH] VMI paravirt-ops bugfix for 2.6.21



On Saturday 31 March 2007 10:45, Zachary Amsden wrote:
So lazy MMU mode is vulnerable to interrupts coming in and issuing
kmap_atomic, which does not work when under lazy MMU mode. The window
for this is small, but it means highmem kernels, especially with heavy
network, USB, or AIO workloads are vulnerable to getting invariably
fatal pagefaults in interrupt handlers. For now, the best fix is to
simply disable and re-enable interrupts when entering and exiting lazy
mode (which, btw, is already guaranteed to have preempt disabled). For
the future, a better fix is to simply exit lazy mode when issuing
kmap_atomic, but I do not want to touch any generic code now for 2.6.21.

I think I would prefer you touch the generic code. This new hook is ugly.

And the lazy mode is currently only used by VMI anyways, isn't it? So you shouldn't
impact anybody else

-Andi

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



Relevant Pages

  • Re: Clue bats
    ... This time, it's scaffolding; but I ... For me, it's not that I'm too lazy to go back and fix earlier chapters, ... I try not to unless I'm really stuck. ...
    (rec.arts.sf.composition)
  • Re: What the hack? (Getaway Fliptronics board)
    ... Yes, I'd say whoever did it was too lazy to pull the board, remove ... the bad components and fix it properly. ... Replace with the proper components, ...
    (rec.games.pinball)
  • Re: DRM Security
    ... has a pointer to the KB that details how to fix this. ... Sorry for being lazy on my part. ... Speaking for myself only. ...
    (microsoft.public.windows.vista.music_pictures_video)
  • Re: /proc/cpuinfo mistake
    ... Alan Connor wrote in comp.os.linux.misc: ... Your line wrap is broken, fix it. ... lazy. ...
    (comp.os.linux.misc)
  • Re: 3rd time is the charm! Linking CountIF formula
    ... Maybe you could use a different function (multiple times!): ... (You'll have to fix all the references--I got lazy!) ... BenJAMMIN wrote: ...
    (microsoft.public.excel.misc)