[vbox-dev] Error in building latest SVN

[Larry Finger]

Frank Mehnert wrote:
> On Friday 08 February 2008, Larry Finger wrote:
>> In the new kernel, __attribute_used__ is not defined anywhere. It is truly
>> deprecated in 2.6.25.
> 
> Please could you try the latest SVN.

Yes, the above problem is fixed.

There are at least 2 more changes in the 2.6.25 kernel that affect VirtualBox:

(1) Struct super_operations no longer has member read_inode.
(2) The routine change_page_attr no longer exists and the module build has a missing global.

The patch file below fixes #1. I'm still working on #2.

Index: vbox/src/VBox/Additions/linux/sharedfolders/vfsmod.c
===================================================================
--- vbox.orig/src/VBox/Additions/linux/sharedfolders/vfsmod.c
+++ vbox/src/VBox/Additions/linux/sharedfolders/vfsmod.c
@@ -296,10 +296,12 @@ sf_clear_inode (struct inode *inode)
     the only thing that is known about inode at this point is its index
     hence we can't do anything here, and let lookup/whatever with the
     job to properly fill then [inode] */
+#if LINUX_VERSION_CODE < KERNEL_VERSION (2, 6, 25)
  static void
  sf_read_inode (struct inode *inode)
  {
  }
+#endif

  /* vfs is done with [sb] (umount called) call [sf_glob_free] to unmap
     the folder and free [sf_g] */
@@ -337,7 +339,9 @@ sf_remount_fs (struct super_block *sb, i

  static struct super_operations sf_super_ops = {
  	.clear_inode = sf_clear_inode,
+#if LINUX_VERSION_CODE < KERNEL_VERSION (2, 6, 25)
  	.read_inode  = sf_read_inode,
+#endif
  	.put_super   = sf_put_super,
  	.statfs      = sf_statfs,
  	.remount_fs  = sf_remount_fs

Larry