lkml.org 
[lkml]   [2008]   [Feb]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: acpi dsts loading and populate_rootfs
    [skipping the populate_rootfs discussion as it seems you have a better
    handle on that than me]

    On Sun, Feb 10, 2008 at 12:58:09PM +0100, Eric Piel wrote:
    > >And while we're at it the file reading thing in there is utter crap
    > >aswell. You really should be using the firmware loader which works
    > >perfectly fine if you initramfs is set up for it. So please folks,
    > >back to the drawing board, do it properly and send it out to lkml
    > >for review please.
    > Christoph, if you have seen this part of the code, you have probably
    > also read the big fat warning explaining why this cannot be done by
    > firmware loader (ie: userspace cannot be run at this early time,
    > corresponding to acpi_early_init()). However, you probably know the
    > kernel ten times better than me. Could you explain what I misunderstood
    > when writing this warning, and give me some hints about how to use the
    > firmware loader in this case?

    Sorry, I misparsed the comment. I took it for the usual I'm too lazy
    to put something that could load firmware into initramfs excuse.

    But thinking about it is there a reason acpi initialization needs to
    happen so early that we can't even have userspace in initramfs running?

    But if we can't use real userspace this could should at least be written
    like the pseudo-userspace in init/do_mounts*.c, using the sys_ syscall
    implementations.

    As an additional comment the stat + open approach is racy and not a good
    idea. Please just open the file using sys_open, it will tell you
    if the file doesn't exist and then use fstat on it to find the
    length. It would also be useful if this kind of code is not hidden
    inside acpi but rather done somewhere close to the early init code
    because that's where people would expect this kind of nastiness._
    syscall
    implementations.

    As an additional comment the stat + open approach is racy and not a good
    idea. Please just open the file using sys_open, it will tell you
    if the file doesn't exist and then use fstat on it to find the
    length. It would also be useful if this kind of code is not hidden
    inside acpi but rather done somewhere close to the early init code
    because that's where people would expect this kind of nastiness.


    \
     
     \ /
      Last update: 2008-02-12 06:43    [W:0.023 / U:30.220 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site