[HP Z1-G2] Realtek card reader [10ec:5249] does not work after suspended on this system

Bug #1359052 reported by Adam Lee
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
HWE Next
Fix Released
Medium
Adam Lee
Trusty
Fix Released
Medium
Adam Lee
Utopic
Fix Released
Medium
Adam Lee
Vivid
Fix Released
Medium
Adam Lee
linux (Ubuntu)
Fix Released
Medium
Adam Lee
Trusty
Fix Released
Medium
Adam Lee
Utopic
Fix Released
Medium
Adam Lee
Vivid
Fix Released
Medium
Adam Lee

Bug Description

SRU Justification:

Impact: the card reader does not work on this system after s3.
Fix: cherry pick a simple patchset only for the certain driver module.
Testcase: already tested in OEM projects

Before suspend
[ 12.896116] rtsx_pci 0000:03:00.0: irq 47 for MSI/MSI-X
[ 12.896128] rtsx_pci 0000:03:00.0: rtsx_pci_acquire_irq: pcr->msi_en = 1, pci->irq = 47
After resume
[ 89.354583] rtsx_pci 0000:03:00.0: no hotplug settings from platform

rmmod rtsx_pci
modprobe rtsx_pci
doesn't work

Reboot
[ 12.358093] rtsx_pci 0000:03:00.0: irq 47 for MSI/MSI-X
[ 12.358103] rtsx_pci 0000:03:00.0: rtsx_pci_acquire_irq: pcr->msi_en = 1, pci->irq = 47
[ 12.455971] rtsx_pci: probe of 0000:03:00.0 failed with error -110

Then driver cannot be loaded

Changed in linux (Ubuntu Trusty):
status: New → Triaged
importance: Undecided → Medium
tags: added: trusty utopic
Revision history for this message
AceLan Kao (acelankao) wrote :
Revision history for this message
AceLan Kao (acelankao) wrote :
Revision history for this message
AceLan Kao (acelankao) wrote :
Revision history for this message
Adam Lee (adam8157) wrote :

tarball of dmesg logs

dmesg.pass : normal case, SD card is detected, then remove it.
dmesg.resume: dumped after resume.
dmesg.fail: after resume, insert SD card again. SD card cannot be detected.

summary: - Realtek card reader does not work after suspended on a certain HP
- machine
+ [HP Z1-G2] Realtek card reader [10ec:5249] does not work after suspended
summary: [HP Z1-G2] Realtek card reader [10ec:5249] does not work after suspended
+ on this system
Revision history for this message
Adam Lee (adam8157) wrote :

"options rtsx_pci msi_en=0" could workaround this issue.

Revision history for this message
Adam Lee (adam8157) wrote :

Here is a kernel for testing, it should be able to work, http://kernel.ubuntu.com/~adamlee/lp1359052/

Revision history for this message
AceLan Kao (acelankao) wrote :

1. remove the workaround in /etc/modprobe.d/realtek.conf
2. reboot, and reproduce the issue
3. install the new kernel from #6
4. reboot and do S3, the issue can't be reproduced.
5. do S3 again, the SD card can still be recognized.

Revision history for this message
Adam Lee (adam8157) wrote :

waiting vendor submitting this fix to upstream.

Adam Lee (adam8157)
Changed in linux (Ubuntu Trusty):
assignee: nobody → Adam Lee (adam8157)
Revision history for this message
Adam Lee (adam8157) wrote :

https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1366841 looks like the same issue as this, which was fixed by another solution from vendor, I'm contacting vendor for advice.

Revision history for this message
Adam Lee (adam8157) wrote :

Vendor comments the solution #1366841 mentioned:

"The solution in Launchpad is just a workaround, not a fix. We have submitted our patch recently, and please apply our patch to fix this problem.

Our patch set:
https://<email address hidden>/msg723994.html"

Revision history for this message
Adam Lee (adam8157) wrote :

please ignore #9, not the same issue.

Adam Lee (adam8157)
Changed in linux (Ubuntu Utopic):
status: Triaged → In Progress
Changed in linux (Ubuntu Trusty):
status: Triaged → In Progress
Revision history for this message
Adam Lee (adam8157) wrote :
Adam Lee (adam8157)
description: updated
Revision history for this message
Adam Lee (adam8157) wrote :

patchset has been merged into linus's master branch and submitted to kernel-team@

Brad Figg (brad-figg)
Changed in linux (Ubuntu Trusty):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Utopic):
status: In Progress → Fix Committed
Adam Lee (adam8157)
no longer affects: hwe-next/trusty
no longer affects: hwe-next/utopic
Adam Lee (adam8157)
Changed in linux (Ubuntu Vivid):
status: In Progress → Fix Committed
Changed in hwe-next:
status: In Progress → Fix Committed
Revision history for this message
Brad Figg (brad-figg) wrote :

This bug is awaiting verification that the kernel 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-trusty' to 'verification-done-trusty'.

If verification is not done by 5 working days 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!

tags: added: verification-needed-trusty
tags: added: verification-needed-utopic
Revision history for this message
Brad Figg (brad-figg) wrote :

This bug is awaiting verification that the kernel 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-utopic' to 'verification-done-utopic'.

If verification is not done by 5 working days 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!

Revision history for this message
Phidias (phidias-chiang) wrote :

Trusty verified.
The bug can be reproduced after installing Trusty, and after update the bug is fixed.

tags: added: verification-done-trusty
removed: verification-needed-trusty
Revision history for this message
Phidias (phidias-chiang) wrote :

Utopic verified.
The bug can be reproduced after installing Trusty (3.16.0-23-generic),
and after update the bug is fixed (3.16.0-29-generic).

tags: added: verification-done-utopic
removed: verification-needed-utopic
Revision history for this message
Phidias (phidias-chiang) wrote :

Sorry, I mean "after installing Utopic" in #17.

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

This bug was fixed in the package linux - 3.18.0-8.9

---------------
linux (3.18.0-8.9) vivid; urgency=low

  [ Leann Ogasawara ]

  * Release Tracking Bug
    - LP: #1407692
  * rebase to v3.18.1
  * ubuntu: AUFS -- Resolve build failure union has no member named
    'd_child'

  [ Upstream Kernel Changes ]

  * arm64: optimized copy_to_user and copy_from_user assembly code
    - LP: #1400349
  * x86, kvm: Clear paravirt_enabled on KVM guests for espfix32's benefit
    - LP: #1400314
    - CVE-2014-8134
  * rebase to v3.18.1
 -- Leann Ogasawara <email address hidden> Mon, 05 Jan 2015 09:12:32 -0800

Changed in linux (Ubuntu Vivid):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (21.8 KiB)

This bug was fixed in the package linux - 3.16.0-29.39

---------------
linux (3.16.0-29.39) utopic; urgency=low

  [ Kamal Mostafa ]

  * Release Tracking Bug
    - LP: #1402822

  [ AceLan Kao ]

  * SAUCE: Add use_native_backlight quirk for HP ProBook 6570b
    - LP: #1359010

  [ Andy Whitcroft ]

  * Revert "SAUCE: (no-up) arm64: optimized copy_to_user and copy_from_user
    assembly code"
    - LP: #1398596
  * [Config] updateconfigs to balance CONFIG_SCOM_DEBUGFS

  [ Paolo Pisati ]

  * [Config] armhf: VIRTIO_[BALLOON|MMIO]=y

  [ Upstream Kernel Changes ]

  * Revert "arm64: Make default dma_ops to be noncoherent"
    - LP: #1386490
  * Revert "percpu: free percpu allocation info for uniprocessor system"
    - LP: #1401079
  * ath3k: Add support of MCI 13d3:3408 bt device
    - LP: #1395465
  * x86: kvm: use alternatives for VMCALL vs. VMMCALL if kernel text is
    read-only
    - LP: #1379340
  * cpufreq: Allow stop CPU callback to be used by all cpufreq drivers
    - LP: #1397928
  * cpufreq: powernv: Set the pstate of the last hotplugged out cpu in
    policy->cpus to minimum
    - LP: #1397928
  * cpufreq: powernv: Set the cpus to nominal frequency during reboot/kexec
    - LP: #1397928
  * xen-netfront: Remove BUGs on paged skb data which crosses a page
    boundary
    - LP: #1275879
  * ACPI / blacklist: blacklist Win8 OSI for Dell Vostro 3546
    - LP: #1383589
  * iwlwifi: add device / firmware to fw-error-dump file
    - LP: #1399440
  * iwlwifi: rename iwl_mvm_fw_error_next_data
    - LP: #1399440
  * iwlwifi: pcie: add firmware monitor capabilities
    - LP: #1399440
  * iwlwifi: remove wrong comment about alignment in iwl-fw-error-dump.h
    - LP: #1399440
  * iwlwifi: mvm: don't collect logs in the interrupt thread
    - LP: #1399440
  * iwlwifi: mvm: kill iwl_mvm_fw_error_rxf_dump
    - LP: #1399440
  * iwlwifi: mvm: update layout of firmware error dump
    - LP: #1399440
  * powerpc/pseries: Fix endiannes issue in RTAS call from xmon
    - LP: #1396235
  * mmc: sdhci-pci-o2micro: Fix Dell E5440 issue
    - LP: #1346067
  * mfd: rtsx: Fix PM suspend for 5227 & 5249
    - LP: #1359052
  * samsung-laptop: Add broken-acpi-video quirk for NC210/NC110
    - LP: #1401079
  * acer-wmi: Add acpi_backlight=video quirk for the Acer KAV80
    - LP: #1401079
  * pinctrl: baytrail: show output gpio state correctly on Intel Baytrail
    - LP: #1401079
  * ALSA: hda - Add dock support for Thinkpad T440 (17aa:2212)
    - LP: #1401079
  * ALSA: hda - Add ultra dock support for Thinkpad X240.
    - LP: #1401079
  * rbd: Fix error recovery in rbd_obj_read_sync()
    - LP: #1401079
  * ds3000: fix LNB supply voltage on Tevii S480 on initialization
    - LP: #1401079
  * powerpc: do_notify_resume can be called with bad thread_info flags
    argument
    - LP: #1401079
  * powerpc/powernv: Properly fix LPC debugfs endianness
    - LP: #1401079
  * irqchip: armada-370-xp: Fix MSI interrupt handling
    - LP: #1401079
  * irqchip: armada-370-xp: Fix MPIC interrupt handling
    - LP: #1401079
  * USB: kobil_sct: fix non-atomic allocation in write path
    - LP: #1401079
  * USB: opticon: fix non-atomic allocation in write path
    - LP: #14010...

Changed in linux (Ubuntu Utopic):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (10.8 KiB)

This bug was fixed in the package linux - 3.13.0-44.73

---------------
linux (3.13.0-44.73) trusty; urgency=low

  [ Kamal Mostafa ]

  * Release Tracking Bug
    - LP: #1402872

  [ AceLan Kao ]

  * SAUCE: Add use_native_backlight quirk for HP ProBook 6570b
    - LP: #1359010

  [ Andy Whitcroft ]

  * Revert "SAUCE: (no-up) arm64: optimized copy_to_user and copy_from_user
    assembly code"
    - LP: #1398596
  * [Config] updateconfigs to balance CONFIG_SCOM_DEBUGFS

  [ Upstream Kernel Changes ]

  * iwlwifi: mvm: fix merge damage
    - LP: #1393317
  * iwlwifi: remove IWL_UCODE_TLV_FLAGS_SCHED_SCAN flag
    - LP: #1393317
  * iwlwifi: mvm: disable scheduled scan to prevent firmware crash
    - LP: #1393317
  * iwlwifi: mvm: enable scheduled scan on newest firmware
    - LP: #1393317
  * x86: kvm: use alternatives for VMCALL vs. VMMCALL if kernel text is
    read-only
    - LP: #1379340
  * phylib: introduce PHY_INTERFACE_MODE_XGMII for 10G PHY
    - LP: #1381084
  * of: make of_get_phy_mode parse 'phy-connection-type'
    - LP: #1381084
  * xen-netfront: Remove BUGs on paged skb data which crosses a page
    boundary
    - LP: #1275879
  * ACPI / blacklist: blacklist Win8 OSI for Dell Vostro 3546
    - LP: #1383589
  * powerpc/pseries: Fix endiannes issue in RTAS call from xmon
    - LP: #1396235
  * mmc: sdhci-pci-o2micro: Fix Dell E5440 issue
    - LP: #1346067
  * mfd: rtsx: Fix PM suspend for 5227 & 5249
    - LP: #1359052
  * drivers:scsi:storvsc: Fix a bug in handling ring buffer failures that
    may result in I/O freeze
    - LP: #1400289
  * arm64: optimized copy_to_user and copy_from_user assembly code
    - LP: #1400349
  * net:socket: set msg_namelen to 0 if msg_name is passed as NULL in
    msghdr struct from userland.
    - LP: #1335478
  * drm/radeon: initialize sadb to NULL in the audio code
    - LP: #1402714
  * powerpc/vphn: NUMA node code expects big-endian
    - LP: #1401150
  * ALSA: usb-audio: Fix device_del() sysfs warnings at disconnect
    - LP: #1402853
  * ALSA: hda - Add mute LED pin quirk for HP 15 touchsmart
    - LP: #1334950, #1402853
  * rcu: Make callers awaken grace-period kthread
    - LP: #1402853
  * rcu: Use rcu_gp_kthread_wake() to wake up grace period kthreads
    - LP: #1402853
  * net: sctp: fix NULL pointer dereference in af->from_addr_param on
    malformed packet
    - LP: #1402853
  * KVM: x86: Don't report guest userspace emulation error to userspace
    - LP: #1402853
  * [media] ttusb-dec: buffer overflow in ioctl
    - LP: #1402853
  * arm64: __clear_user: handle exceptions on strb
    - LP: #1402853
  * ARM: pxa: fix hang on startup with DEBUG_LL
    - LP: #1402853
  * samsung-laptop: Add broken-acpi-video quirk for NC210/NC110
    - LP: #1402853
  * acer-wmi: Add Aspire 5741 to video_vendor_dmi_table
    - LP: #1402853
  * acer-wmi: Add acpi_backlight=video quirk for the Acer KAV80
    - LP: #1402853
  * rbd: Fix error recovery in rbd_obj_read_sync()
    - LP: #1402853
  * [media] ds3000: fix LNB supply voltage on Tevii S480 on initialization
    - LP: #1402853
  * powerpc: do_notify_resume can be called with bad thread_info flags
    argument
    - LP: #1402853
  * USB: kobil_sct: f...

Changed in linux (Ubuntu Trusty):
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.