Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 398111 - net-fs/openafs-kernel-1.6.0 fails against linux 3.2
Summary: net-fs/openafs-kernel-1.6.0 fails against linux 3.2
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Andrej Filipcic
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-08 02:07 UTC by Markus Peloquin
Modified: 2012-01-09 16:38 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Peloquin 2012-01-08 02:07:54 UTC
In linux's 'struct inode', i_nlink is now read-only for some reason.  They provide an inline function to set it: set_nlink(struct inode *, unsigned)

Attaching a patch...

Reproducible: Always
Comment 1 Markus Peloquin 2012-01-08 02:11:03 UTC
Okay i'm having trouble with uploading files... probably due to my KDE update.  Here's the patch:

--- openafs-1.6.0/src/afs/LINUX/osi_vfsops.c	2012-01-07 17:50:01.437528708 -0800
+++ openafs-1.6.0/src/afs/LINUX/osi_vfsops.c	2012-01-07 17:50:39.531045712 -0800
@@ -24,6 +24,10 @@
 
 #include "osi_compat.h"
 
+#ifndef LINUX_VERSION_CODE
+#	include <linux/version.h>
+#endif
+
 struct vcache *afs_globalVp = 0;
 struct vfs *afs_globalVFS = 0;
 struct vfsmount *afs_cacheMnt;
@@ -439,7 +443,11 @@
 vattr2inode(struct inode *ip, struct vattr *vp)
 {
     ip->i_ino = vp->va_nodeid;
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(3,2,0)
+    set_nlink(ip, vp->va_nlink);
+#else
     ip->i_nlink = vp->va_nlink;
+#endif
     ip->i_blocks = vp->va_blocks;
     ip->i_blkbits = AFS_BLKBITS;
 #endif
Comment 2 Markus Peloquin 2012-01-08 02:15:10 UTC
Here's the commit that broke openafs-kernel:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=a78ef704a8dd430225955f0709b22d4a6ba21deb

It doesn't seem like a big deal...
Comment 3 Andrej Filipcic 2012-01-09 15:30:51 UTC
please try the latest 1.6.1 pre1 ebuild in portage and report.
Comment 4 Mike Hammill 2012-01-09 15:53:16 UTC
Failed for me using gentoo-sources-3.2.0-r1, openafs-1.6.1_pre1, and heimdal-1.5.1-r1.  If this was supposed to work against a 3.2 kernel, what test case *was* found to work?  I would guess it was probably tested against mit-krb but not heimdal. (Just guessing.)  More at Bug #398253.
Comment 5 Markus Peloquin 2012-01-09 16:38:47 UTC
I can't do it right now; 1.6.1_pre1 has the same fix according to git.openafs.org so I'm satisfied.

The heimdal stuff isn't relevant to this bug, and probably linux-3.2 isn't relevant to bug #398253.