Ubuntu
linux package

OpenVZ kernel: non-POSIX behavior in mmap functions

Bug #231400 reported by Daniel Hahler
12
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Medium
Tim Gardner
Hardy
Fix Released
Medium
Tim Gardner
Ubuntu ubuntu-8.04.1
Intrepid
Fix Released
Medium
Tim Gardner

Bug Description

I had upstart crashes in an OpenVZ container, when upstart was reading new/changed event files.
(This caused the whole container to crash!)

Scott has debugged this to be caused by non-POSIX behavior in mmap related functions in the kernel.

There are two patches in the OpenVZ patchset, which cause this:
1. "Return addr from mmap(2) when len is zero"
    http://git.openvz.org/?p=linux-2.6.24-openvz;a=commit;h=8d7a5ca70e9913a71ec0ac9dccdb6ce85c23ac4b
2. "IA64: mmap returns EINVAL if len==0"
    http://git.openvz.org/?p=linux-2.6.24-openvz;a=commit;h=8ddefba8a401d78484899d05c0f5d3b45a4e8e0b

I've added a patch to debian/binary-custom.d/openvz/patchset/, which reverts those patches from the OpenVZ patchset and the upstart crashes are fixed now.

This is with 2.6.24-18-openvz (from Git) on Ubuntu Hardy. The host system is amd64, the containers x86.

Revision history for this message
Daniel Hahler (blueyed) wrote :
Bug Watch Updater (bug-watch-updater)
Changed in openvz-kernel:
status: Unknown → Confirmed
Revision history for this message
Daniel Hahler (blueyed) wrote :

Subscribing ubuntu-sru.

This has been included now in the upstream patchset (ovz005.1):
http://download.openvz.org/kernel/branches/2.6.24/2.6.24-ovz005.1/patches/

See also upstream git at: http://git.openvz.org/?p=ubuntu-hardy-openvz;a=blob;f=debian/binary-custom.d/openvz/patchset/0074-mmap-0-einval.patch;h=a2a4c8a0330850787334465ebf211c66a4cbda43;hb=a625c1cb036f635124c67d4745da52516805b225

More information is available in the upstream bug report.

Revision history for this message
Daniel Hahler (blueyed) wrote :

Sorry, subscribing motu-sru instead, since linux-openvz is part of Universe (multiverse).

Revision history for this message
Martin Pitt (pitti) wrote :

Approved for SRU, simple patch.

Changed in linux:
assignee: nobody → ubuntu-kernel-team
status: New → In Progress
status: In Progress → Triaged
Revision history for this message
Tim Gardner (timg-tpi) wrote :

http://kernel.ubuntu.com/git?p=ubuntu/ubuntu-hardy.git;a=commit;h=6a85f3003f86810e982be1f93fc4256f3e821698

Changed in linux:
assignee: ubuntu-kernel-team → timg-tpi
importance: Undecided → Medium
milestone: none → ubuntu-8.04.1
status: Triaged → Fix Committed
Bug Watch Updater (bug-watch-updater)
Changed in openvz-kernel:
status: Confirmed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here

Changed in linux:
status: New → Fix Committed
Steve Langasek (vorlon)
Changed in linux:
assignee: nobody → timg-tpi
importance: Undecided → Medium
milestone: none → ubuntu-8.04.1
milestone: ubuntu-8.04.1 → none
Revision history for this message
Martin Pitt (pitti) wrote :

Copied to hardy-updates. The new kernel was tested extensively by many people, who reported back in other bug reports. Due to lack of feedback, this particular bug was not confirmed to be tested, though. Please report back here if the bug still occurs for you with the new kernel packages, then we will reopen this bug.

Changed in linux:
status: Fix Committed → Fix Released
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

This is "Fix Released" for Intrepid.

Changed in linux:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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