Ext4 Jaunty SRU tracking bug

Bug #418197 reported by Leann Ogasawara
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Unassigned
Jaunty
Fix Released
Medium
Stefan Bader

Bug Description

SRU Justification: After examining the ext4 maintainer's 2.6.28-stable tree repository there were a few candidate patches we should consider backporting. These will prevent some cases of Oops', lock contention, file corruption, CPU lockups, memory leaks, and fsck errors.

Impact: https://lists.ubuntu.com/archives/kernel-team/2009-June/006142.html

Fix: Backport the following 7 patches

fb68407b0d9efba962c03f55009c797e22f024bc jbd2: Call journal commit callback without holding j_list_lock
9c1ee184a30394e54165fa4c15923cabd952c106 ext4: Fix sub-block zeroing for writes into preallocated extents
485c26ec70f823f2a9cf45982b724893e53a859e ext4: Fix softlockup caused by illegal i_file_acl value in on-disk inode
a7b19448ddbdc34b2b8fedc048ba154ca798667b ext4: fix typo which causes a memory leak on error path
8d03c7a0c550e7ab24cadcef5e66656bfadec8b9 ext4: fix bogus BUG_ONs in in mballoc code
7ce9d5d1f3c8736511daa413c64985a05b2feee3 ext4: fix ext4_free_inode() vs. ext4_claim_inode() race
8f64b32eb73fbfe9f38c4123121b63ee409278a7 ext4: don't call jbd2_journal_force_commit_nested without journal

Testcase: While unable to reproduce the cases, I ran the fs test suite proposed in https://lists.ubuntu.com/archives/kernel-team/2009-June/006147.html before and after applying the patchset and could see no regression.

Changed in linux (Ubuntu):
status: New → Fix Released
Changed in linux (Ubuntu Jaunty):
importance: Undecided → Medium
status: New → Triaged
Stefan Bader (smb)
Changed in linux (Ubuntu Jaunty):
assignee: nobody → Stefan Bader (stefan-bader-canonical)
status: Triaged → In Progress
Stefan Bader (smb)
description: updated
Stefan Bader (smb)
Changed in linux (Ubuntu Jaunty):
status: In Progress → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted linux into jaunty-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

I have been running this on an affected system all weekend and have not seen any regressions. Furthermore, I tried to trigger this today by looping through an untar/rm -rf cycle on some large and fairly deep tar files while doing a kernel compile and there were no lockups (I used to be able to reproduce a lockup fairly easily with this procedure).

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

Thanks, Jamie!

tags: added: hw-specifc
tags: added: verification-done
removed: hw-specifc verification-needed
tags: added: verification-needed
removed: verification-done
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

I've been running the aforementioned loop for over 3 hours with no lockups or other problems. Before, doing this would lock the machine up in less than 5 minutes. This system seems very solid now.

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

This bug was fixed in the package linux - 2.6.28-15.52

---------------
linux (2.6.28-15.52) jaunty-proposed; urgency=low

  [ Stefan Bader ]

  * Revert "SAUCE: ACPI: Populate DIDL before registering ACPI video device
    on Intel"
    - LP: #423296
  * SAUCE: Allow less restrictive acpi video detection
    - LP: #333386

  [ Upstream Kernel Changes ]

  * include drivers/pci/hotplug/* in -virtual package
    - LP: #364916
  * ext4: don't call jbd2_journal_force_commit_nested without journal
    - LP: #418197
  * ext4: fix ext4_free_inode() vs. ext4_claim_inode() race
    - LP: #418197
  * ext4: fix bogus BUG_ONs in in mballoc code
    - LP: #418197
  * ext4: fix typo which causes a memory leak on error path
    - LP: #418197
  * ext4: Fix softlockup caused by illegal i_file_acl value in on-disk
    inode
    - LP: #418197
  * ext4: Fix sub-block zeroing for writes into preallocated extents
    - LP: #418197
  * jbd2: Call journal commit callback without holding j_list_lock
    - LP: #418197
  * ext4: Print the find_group_flex() warning only once
    - LP: #367065
  * ext4: really print the find_group_flex fallback warning only once
    - LP: #367065

linux (2.6.28-15.51) jaunty-proposed; urgency=low

  [ Colin Ian King ]

  * SAUCE: wireless: hostap, fix oops due to early probing interrupt
    - LP: #254837

  [ Leann Ogasawara ]

  * Add the atl1c driver to support Atheros AR8132
    - LP: #415358
  * Updating configs to enable the atl1c driver
    - LP: #415358

  [ Stefan Bader ]

  * Revert "SAUCE: input: Blacklist digitizers from joydev.c"
    - LP: #300143
  * SAUCE: Fix the exported name for e1000e-next
    - LP: #402890
  * SAUCE: Fix incorrect stable backport to bas_gigaset
    - LP: #417732
  * SAUCE: Remove the atl2 driver from the ubuntu subdirectory
    - LP: #419438

linux (2.6.28-15.50) jaunty-proposed; urgency=low

  [ Colin Ian King ]

  * SAUCE: radio-maestro: fix panics on probe failure
    - LP: #357724
  * SAUCE: HDA Intel, sigmatel: Enable speakers on HP Mini 1000
    - LP: #318942

  [ Jerone Young ]

  * SAUCE: Fix Soltech TA12 volume hotkeys not sending key release in
    Jaunty
    - LP: #397499

  [ John Johansen ]

  * SAUCE: remove AppArmor debug check for calls from interrupt context
    - LP: #350789

  [ Manoj Iyer ]

  * SAUCE: Fix kernel panic when SELinux is enabled.
    - LP: #395219

  [ Matthew Garrett ]

  * SAUCE: ACPI: Populate DIDL before registering ACPI video device on
    Intel

  [ Michael Frey (Senior Manager, MID ]

  * SAUCE: Fix for internal microphone for Dell Mini10V
    - LP: #394793

  [ Tim Gardner ]

  * SAUCE: Added e1000e from sourceforge.
    - LP: #402890

  [ Upstream Kernel Changes ]

  * Input: synaptics - report multi-taps only if supported by the device
    - LP: #399787
  * ftdi_sio: fix kref leak
    - LP: #396930, #376128
  * IPv6: add "disable" module parameter support to ipv6.ko
    - LP: #351656

 -- Stefan Bader <email address hidden> Thu, 27 Aug 2009 15:09:06 +0200

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

Other bug subscribers

Remote bug watches

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