Messages in this thread | | | From | Keith Owens <> | Subject | Re: [PATCH] iget-locked [2/6] | Date | Sat, 11 May 2002 10:22:39 +1000 |
| |
On Fri, 10 May 2002 15:57:03 -0400 (EDT), Alexander Viro <viro@math.psu.edu> wrote: >On Fri, 10 May 2002, Jan Harkes wrote: >> Now we introduce iget_locked and iget5_locked. These are similar to >> iget, but return a locked inode and read_inode has not been called. So >> the FS has to call read_inode to initialize the inode and then unlock >> it with unlock_new_inode(). > >No problems, except for putting exports in inode.c. ISTR Linus saying that >additional files with exports seriously increase the build time... Linus?
Build time is the least of your worries here. All objects that export symbols must have unique basenames, all the modversion crud goes in include/linux/modules under the object's basename. This is the main reason that many subsystems have a subsystem_ksyms.c file, to get a unique base name.
There are 34 files called inode in 2.4.18. None currently export symbols so adding EXPORT_SYMBOL to fs/inode.c is safe, until somebody else decides they want their inode.c to export symbols. You will not notice until you build both systems with MODVERSIONS=y.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |