[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:
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.
--
