open-vm-dkms 2012.10.14-874563-0ubuntu1: open-vm-tools kernel module failed to build

Bug #1100096 reported by Sidnei da Silva
30
This bug affects 4 people
Affects Status Importance Assigned to Milestone
open-vm-tools (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Not much to add.

ProblemType: Package
DistroRelease: Ubuntu 13.04
Package: open-vm-dkms 2012.10.14-874563-0ubuntu1
ProcVersionSignature: Ubuntu 3.8.0-0.3-generic 3.8.0-rc3
Uname: Linux 3.8.0-0-generic x86_64
ApportVersion: 2.8-0ubuntu1
Architecture: amd64
DKMSKernelVersion: 3.8.0-0-generic
Date: Tue Jan 15 22:02:35 2013
InstallationDate: Installed on 2012-09-04 (133 days ago)
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha amd64 (20120901)
MarkForUpload: True
PackageArchitecture: all
PackageVersion: 2012.10.14-874563-0ubuntu1
SourcePackage: open-vm-tools
Title: open-vm-dkms 2012.10.14-874563-0ubuntu1: open-vm-tools kernel module failed to build
UpgradeStatus: Upgraded to raring on 2012-12-26 (20 days ago)

Related branches

Revision history for this message
Sidnei da Silva (sidnei) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in open-vm-tools (Ubuntu):
status: New → Confirmed
Revision history for this message
Hugo van der Wijst (hugwijst) wrote :

I'm seeing the same compilation errors with a fresh and up to date installation of RR x64.

Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

I've linked this bug report to the my proposal.

This is the packaging: https://launchpad.net/~n-muench/+archive/virtual-test/+packages

NOTE: This has been tested on the 3.7 kernel, but not on the 3.8 kernel.

tags: removed: need-duplicate-check
Revision history for this message
Hugo van der Wijst (hugwijst) wrote :

The package from Nate fixes most compilation errors, except for the following:

/var/lib/dkms/open-vm-tools/2012.12.26/build/vmhgfs/inode.c: In function ‘HgfsTruncatePages’:
/var/lib/dkms/open-vm-tools/2012.12.26/build/vmhgfs/inode.c:888:4: error: implicit declaration of function ‘vmtruncate’ [-Werror=implicit-function-declaration]
cc1: some warnings being treated as errors

Apparently, the vmtruncate method got removed in the following kernel commit: 7898575fc81bd707ce0844cb06874d48e39bbe09

I've got the tools compiling by copying the contents of the vmtruncate method before it was removed. See the attached patch. Please note that I have remove the call to the truncate method of the inode, as it does not exist anymore. I don't know if this is correct.

Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

I might bring that patch in on the next proposal (I want to do one more before the freeze). That proposal will bring stuff from Debian (including kmod which was withheld from last cycle because it wasn't supported)

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "open-vm-tools.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

Here's an idea, add the patch to my existing proposal because I'm waiting another month (it's been almost one already) for current proposal to reviewed again

Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

For the record, I added a comment to my proposal, so that whoever reviews it, needs to add the patch to the proposal (and probably rename the patch)

Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

I had to redo your patch Hugo, your path was wrong

Revision history for this message
Hugo van der Wijst (hugwijst) wrote :

Thanks Nate, I've been busy the past few days so I didn't have time to look into it.

Revision history for this message
Nate Muench (Mink) (n-muench) wrote :

Sorry about all the activity, there were issues with my proposal. Stuff from my original proposal weren't brought into the newer branches.

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

This bug was fixed in the package open-vm-tools - 2012.12.26-958366-0ubuntu1

---------------
open-vm-tools (2012.12.26-958366-0ubuntu1) raring-proposed; urgency=high

  * Merging upstream version 2012.12.26-958366.

  * Added 04-dkms-vmsync.patch: An additional to 03-dkms.patch
    - Deletes lines involving vmsync, adjust other numbers.
    - Module compile still fails if we don't add this patch.
    - This keeps the 03-dkms.patch unmodified.

  * Added 05-vmhgfs.patch:
    - Fixes vmxnet compile issues with 3.8 kernel (LP: #1100096)

  * Stop dkms from building vmblock module.
    - vmblock-fuse takes care of this now.
      + Modules still build, but this gets rid of the vmblock-related
        warning message
 -- Nate Muench <email address hidden> Mon, 28 Jan 2013 16:11:13 -0600

Changed in open-vm-tools (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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