openafs 1.6.20-2ubuntu2 ADT test failure with linux 4.12.0-12.13

Bug #1711835 reported by Seth Forshee
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openafs (Ubuntu)
Fix Released
Undecided
Unassigned
Revision history for this message
Seth Forshee (sforshee) wrote :
tags: added: kernel-adt-failure
Revision history for this message
Seth Forshee (sforshee) wrote :

In file included from /var/lib/dkms/openafs/1.6.20/build/src/afs/LINUX/osi_machdep.h:129:0,
                 from /var/lib/dkms/openafs/1.6.20/build/src/afs/afs_osi.h:440,
                 from /var/lib/dkms/openafs/1.6.20/build/src/libafs/MODLOAD-4.12.0-11-generic-SP/afs_dir.c:70:
./arch/s390/include/asm/uaccess.h: In function ‘set_fs’:
./arch/s390/include/asm/uaccess.h:41:6: error: implicit declaration of function ‘uaccess_kernel’; did you mean ‘getname_kernel’? [-Werror=implicit-function-declaration]
  if (uaccess_kernel()) {
      ^~~~~~~~~~~~~~
      getname_kernel

It looks like this would be fixed by including uaccess.h instead of asm/uaccess.h, need to determine whether there's some kernel version limitation on this change though.

Revision history for this message
Seth Forshee (sforshee) wrote :

Sorry that should be linux/uaccess.h. Looks like it was introduced in 2.6.18.

Revision history for this message
Seth Forshee (sforshee) wrote :

This patch ought to fix the issue.

Revision history for this message
Seth Forshee (sforshee) wrote :

Note that the patch is against upstream git, it requires some context adjustments for the Debian/Ubuntu packages.

tags: added: patch
Revision history for this message
Seth Forshee (sforshee) wrote :

I applied this patch to a 1.6.21-2ubuntu1 package that I've uploaded to the canonical-kernel-team/unstable ppa and started ADT against that package. s390x passed:

https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-artful-canonical-kernel-team-unstable/artful/s390x/o/openafs/20170823_163112_7793c@/log.gz

i386 also passed, still waiting armhf and amd64 to complete.

Revision history for this message
Anders Kaseorg (andersk) wrote :

Do you want to submit this upstream (https://gerrit.openafs.org/), or do you want me to do it?

Revision history for this message
Seth Forshee (sforshee) wrote :
Revision history for this message
Seth Forshee (sforshee) wrote :

Attaching debdiff for openafs 1.6.21-2ubuntu1 to fix this in Ubuntu until the fix is available in Debian.

Revision history for this message
Nish Aravamudan (nacc) wrote :

Sponsored to artful, thanks Seth!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openafs - 1.6.21-2ubuntu1

---------------
openafs (1.6.21-2ubuntu1) artful; urgency=medium

  * debian/patches:
    - Linux-Include-linux-uaccess.h-rather-than-asm-uacces.patch (LP: #1711835)

 -- Seth Forshee <email address hidden> Thu, 24 Aug 2017 11:00:58 -0700

Changed in openafs (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.