Dell Latitude E6220 does not reboot with 64 bit kernel

[Achim Bohnet]

== SRU Justification ==

It's been reported that the Dell Latitude E6220 doesn't reboot unless
using reboot=pci. The following patch resolves this issue and has been
successfully tested. Upstream is unfortunately not accepting any
additional quirk patches like this as they would prefer to migrate to a
more systematic approach. Given that the patch is system specific (ie
low risk of regression), precedence for similar quirk patches already
existing, and having received positive test results, I feel it should be
carried as a SAUCE patch until a more systematic approach can be found
which upstream will accept.

== Impact ==
Affects Dell Latitude E6220 models.

== Test Case ==
Without the patch, those with a Dell Latitude E6220 must use reboot=pci
as a workaround in order for the system to successfully reboot. With
the patch applied, the work around is no longer needed.

== Fix ==
http://people.canonical.com/~ogasawara/lp838402/0001-UBUNTU-SAUCE-x86-reboot-Make-Dell-Latitude-E6220-use.patch

----------

[bug for model E6520 is https://bugs.launchpad.net/ubuntu/+source/linux/+bug/833705]

When one tries to reboot the Dell Latitude E6220 with a 64 bit kernel from natty it hangs at the very end of the shutdown sequence. The console output is:

...
...
* Will now restart
[ xxxx.yyyyyy] Restarting system.

Then nothing happens. One has to force power off.

Sep 12th the E6220 will be back, then I can verify the patch given
at http://people.canonical.com/~ogasawara/lp833705/

Achim

[Brad Figg (brad-figg)]

This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:

apport-collect 838402

and then change the status of the bug to 'Confirmed'.

If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.

This change has been made by an automated script, maintained by the Ubuntu Kernel Team.

[Achim Bohnet (allee)]

Will do when the Laptop is back in two weeks.

[Leann Ogasawara (leannogasawara)]

Achim, thanks for opening this new bug.

Go ahead and disregard comment #1 from our bug bot. Assuming the dmi info for the E6220 is almost identical to the E6520, the patch and test kernel I made just needs verified. I'm still trying to get access to the E6220, so please let me know your testing results if you get access before I do. Thanks.

[Achim Bohnet (allee)]

plain natty kernel 2.6.38-10:
  * laptop hangs on reboot.
  * adding reboot=pci to kern params and laptop reboots without problems.

I'll try a oneiric live CDROM next ...

[Achim Bohnet (allee)]

removing an oneiric disk from another laptop was faster than booting from live CD, so

E6220 with plain 3.0.0-11 oneiric kernel:
  * hangs on reboot
  * with reboot=pci reboots without problems

patched 3.0.0-10 kernel from http://people.canonical.com/~ogasawara/lp833705/ did reboot perfectly.

So IMHO the corresponding patch as for the E6520 can go into the ubuntu kernel.

Achim

[Achim Bohnet (allee)]

Leann: can you create a patched natty 64bit kernel for testing? I think it's worth to include the E6520 and E6520 fix to natty too.

Achim

[Leann Ogasawara (leannogasawara)]

Thanks for the Oneiric testing Achim, I've gone ahead and submitted this to the Ubuntu kernel team mailing list:

 https://lists.ubuntu.com/archives/kernel-team/2011-September/017036.html

I'll try and put together some Natty test kernels for verification. Thanks.

[Achim Bohnet (allee)]

Thx Leann! Same for E6520 too? LP# 833705

[Leann Ogasawara (leannogasawara)]

Hi Achim,

The patch for the E6220 has been applied to the Oneiric git repo so I'm marking this Fix Committed. The patch for the E6520 (bug 833705) was applied and released in the latest 3.0.0-10.16 Oneiric kernel upload.

I've also built a natty test kernel with the patches for both the E6220 and E6520 applied. Please test and let me know your results. I can then get them submitted for Natty SRU. Test kernel is at the following location:

http://people.canonical.com/~ogasawara/lp838402/natty/

I'll post the natty test kernel information to bug 833705 as well for consistency.

[Achim Bohnet (allee)]

Summary: Your patched kernel solves the problem on Latitude E6220.

Tested your 2.6.38-11.51~lp833705lp838402v1_amd64 kernel on the Latitude E6220 without reboot=pci cmdline and it
reboots without problems.

Thx a lot for fixing!

Achim

[Leann Ogasawara (leannogasawara)]

I've sent the Natty SRU request to the Ubuntu kernel team mailing list:

https://lists.ubuntu.com/archives/kernel-team/2011-September/017119.html

[Leann Ogasawara (leannogasawara)]

Applied to Natty, marking Natty nomination Fix Committed.

[Launchpad Janitor (janitor)]

This bug was fixed in the package linux - 3.0.0-12.19

---------------
linux (3.0.0-12.19) oneiric; urgency=low

  [ Alex Bligh ]

  * SAUCE: (drop after v3.1) net/netfilter/nf_conntrack_netlink.c: fix Oops
    on container destroy
    - LP: #843892

  [ Andy Whitcroft ]

  * [Config] standardise on HZ=250
  * SAUCE: headers_install: fix #include "..." usage for userspace
    - LP: #824377
  * make module-inclusion selection retain the left overs
  * add a new linux-image-extras package for virtual

  [ edwin_rong ]

  * SAUCE: Staging: add driver for Realtek RTS5139 cardreader
    - LP: #824273

  [ Greg Kroah-Hartman ]

  * SAUCE: staging: rts5139: add vmalloc.h to some files to fix the build.
    - LP: #824273

  [ Jesse Sung ]

  * SAUCE: Unregister input device only if it is registered
    - LP: #839238

  [ Keng-Yu Lin ]

  * [Config] Enable CONFIG_RTS5139=m on i386/amd64
    - LP: #824273

  [ Leann Ogasawara ]

  * SAUCE: x86: reboot: Make Dell Optiplex 990 use reboot=pci
    - LP: #768039
  * SAUCE: x86: reboot: Make Dell Latitude E6220 use reboot=pci
    - LP: #838402

  [ Ming Lei ]

  * SAUCE: ata: make DVD drive recognisable on systems with Sandybridge CPT
    chipset
    - LP: #794642

  [ Paolo Pisati ]

  * [Config] Compile-in vfat support for armel
    - LP: #853783

  [ Randy Dunlap ]

  * SAUCE: staging: fix rts5139 depends & build
    - LP: #824273

  [ Tim Gardner ]

  * [Config] Fix binary-% build target
  * SAUCE: (drop after 3.0.0) OMAP3 and 4 hwmod I2C units only allow 16 bit
    access
    - LP: #852225

  [ Upstream Kernel Changes ]

  * hfsplus: Fix kfree of wrong pointers in hfsplus_fill_super() error path
    - LP: #854987
  * rt2x00: Serialize TX operations on a queue.
    - LP: #855239
 -- Leann Ogasawara <email address hidden> Wed, 14 Sep 2011 06:14:30 -0700

[Herton R. Krzesinski (herton)]

This bug is awaiting verification that the kernel for Natty in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-natty' to 'verification-done-natty'.

If verification is not done by one week from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

[Achim Bohnet (allee)]

Which natty kernel in proposed? I've add natty-propsed to sources.list. But apt-cache policy was not able to find
linux-image-2.6.38-11-generic or -12-generic in proposed.

Please lemme know the pkg name

Achim

[Leann Ogasawara (leannogasawara)]

Hi Achim,

It should be the 2.6.38-12.51 kernel in the natty-proposed pocket which contains this fix. After adding natty-proposed to your sources.list, make sure to run sudo apt-get update.

https://launchpad.net/ubuntu/+source/linux/2.6.38-12.51

[Achim Bohnet (allee)]

Latitude E6220 reboot with the proposed updates kernel 2.6.38-12.51.
will change tag to verification-done-natty.

Thx,
Achim

[Launchpad Janitor (janitor)]

This bug was fixed in the package linux - 2.6.38-12.51

---------------
linux (2.6.38-12.51) natty-proposed; urgency=low

  [Herton R. Krzesinski]

  * Release Tracking Bug
    - LP: #860832

  [ Alex Bligh ]

  * SAUCE: net/netfilter/nf_conntrack_netlink.c: fix Oops on container
    destroy
    - LP: #843892

  [ Jesse Sung ]

  * SAUCE: Unregister input device only if it is registered
    - LP: #839238

  [ Leann Ogasawara ]

  * SAUCE: x86: reboot: Make Dell Latitude E6220 use reboot=pci
    - LP: #838402
  * SAUCE: x86: reboot: Make Dell Latitude E6520 use reboot=pci
    - LP: #833705

  [ Ming Lei ]

  * SAUCE: fireware: add NO_MSI quirks for o2micro controller
    - LP: #801719

  [ Stefan Bader ]

  * [Config] Include all filesystem modules for virtual
    - LP: #761809

  [ Tim Gardner ]

  * [Config] kernel preparation cannot be parallelized
  * [Config] Linearize module/abi checks
  * [Config] Linearize and simplify tree preparation rules
  * [Config] Build kernel image in parallel with modules
  * [Config] Set concurrency for kmake invocations
  * [Config] Improve install-arch-headers speed
  * [Config] Fix binary-perarch dependencies
  * [Config] Removed stamp-flavours target
  * [Config] Serialize binary indep targets
  * [Config] Use build stamp directly
  * [Config] Restore prepare-% target
  * [Config] Fix binary-% build target

  [ Upstream Kernel Changes ]

  * Revert "drm/i915: disable PCH ports if needed when disabling a CRTC"
    - LP: #814325, #838181
  * drm/i915: restore only the mode of this driver on lastclose (v2)
    - LP: #848687
  * cifs: fix possible memory corruption in CIFSFindNext, CVE-2011-3191
    - LP: #834135
    - CVE-2011-3191
  * befs: Validate length of long symbolic links, CVE-2011-2928
    - LP: #834124
    - CVE-2011-2928
  * gro: Only reset frag0 when skb can be pulled, CVE-2011-2723
    - LP: #844371
    - CVE-2011-2723
  * inet_diag: fix inet_diag_bc_audit(), CVE-2011-2213
    - LP: #838421
    - CVE-2011-2213
  * si4713-i2c: avoid potential buffer overflow on si4713, CVE-2011-2700
    - LP: #844370
    - CVE-2011-2700
  * Bluetooth: Prevent buffer overflow in l2cap config request,
    CVE-2011-2497
    - LP: #838423
    - CVE-2011-2497
  * crypto: Move md5_transform to lib/md5.c, CVE-2011-3188
    - LP: #834129
    - CVE-2011-3188
  * net: Compute protocol sequence numbers and fragment IDs using MD5,
    CVE-2011-3188
    - LP: #834129
    - CVE-2011-3188
  * x86, intel, power: Initialize MSR_IA32_ENERGY_PERF_BIAS
    - LP: #760131
  * x86, intel, power: Correct the MSR_IA32_ENERGY_PERF_BIAS message
    - LP: #760131
  * rt2x00: Serialize TX operations on a queue.
    - LP: #855239
  * ext4: Fix max file size and logical block counting of extent format
    file, CVE-2011-2695
    - LP: #819574
    - CVE-2011-2695
 -- Herton Ronaldo Krzesinski <email address hidden> Tue, 27 Sep 2011 16:19:57 -0300