Re: [PATCH] shmem: add shmem_size option for full filesystem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>From Qian:

> Hide it under the EXPERT does not mean the bar is lower to justify when introduced a new config option. What’s the benefits of this vs squashfs/resize?

>From Vlastimil:

> Users can override the default via a mount option, no?

> Can the initramfs boot parameter specify the needed mount option
> somehow? If not, could we detect that we are mounting an initramfs, and
> size it accordingly over 50%? Do we know in advance how much we need?
> I'd rather avoid a new config option for this, if possible.

Thank you for clarifying all of that, Qian and Vlastimil.  I have been making the assumption, apparently mistaken, that adding kernel configuration options is a trivial change.  I stand educated.

Thank you for your ongoing patience with my passion regarding this.  I am *not* a kernel developer, understand and appreciate the work involved, and know there are many other important things which need attention.

I concede, but am going to submit one final patch which simply makes the value a define.

My fundamental issue/fix here is that it is a magic token buried in the code, and it should at least be a clearly defined default, as a matter of principle.  This will also make pre-compile sed/awk hacking much more reliable, for those so inclined.

If even that change is unpalatable, I will graciously accept a simple "reject, based on previous discussion", with gratitude.  <^)

Thank you again for your consideration, and all of your hard work and contributions to the community.

Happy Hacking!

-- Gary





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux