Messages in this thread | | | From | "Michael Kerrisk (man-pages)" <> | Date | Fri, 12 Oct 2012 07:16:12 +0200 | Subject | Re: [PATCH 1/4] module: add syscall to load module from fd |
| |
Rusty,
On Fri, Oct 12, 2012 at 12:16 AM, Rusty Russell <rusty@rustcorp.com.au> wrote: > "H. Peter Anvin" <hpa@zytor.com> writes: > >> On 10/10/2012 06:03 AM, Michael Kerrisk (man-pages) wrote: >>> Good point. A "whole hog" openat()-style interface is worth thinking about too. >> >> *Although* you could argue that you can always simply open the module >> file first, and that finit_module() is really what we should have had in >> the first place. Then you don't need the flags since those would come >> from openat(). > > There's no fundamental reason that modules have to be in a file. I'm > thinking of compressed modules, or an initrd which simply includes all > the modules it wants to load in one linear file. > > Also, --force options manipulate the module before loading (as did the > now-obsolete module rename option).
Sure. But my point that started this subthread was: should we take the opportunity now to add a 'flags' argument to the new finit_module() system call, so as to allow flexibility in extending the behavior in future? There have been so many cases of revised system calls in the past few years that replaced calls without a 'flags' argument that it seems worth at least some thought before the API is cast in stone.
Thanks,
Michael
| |