New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 779308 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocked on:
issue 773474



Sign in to add a comment

Block component updater for certain kernel version in M62

Project Member Reported by xiaochu@chromium.org, Oct 28 2017

Issue description

Due to imageloader issue with certain kernel version (v3.8&3.10 kernel has a refcounting issue preventing the /dev/mapper device from being removed), we are disabling component updater for kernel 3.8&3.10 in M62.
 
Cc: adlr@chromium.org
Labels: Merge-Request-62
Description: Show this description
Labels: -Merge-Request-62 Merge-Approved-62
Consider this merge approved, but I don't see a CL here to review.
Blockedon: 773474
Here is the CL: https://chromium-review.googlesource.com/c/chromium/src/+/745303
 
This Fix is only on m62.
Description: Show this description
Cc: bhthompson@chromium.org
 bhthompson@ can you please review the CL https://chromium-review.googlesource.com/c/chromium/src/+/745303?
Project Member

Comment 8 by sheriffbot@chromium.org, Nov 3 2017

Cc: bhthompson@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 9 by sheriffbot@chromium.org, Nov 6 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 10 by bugdroid1@chromium.org, Nov 7 2017

Labels: -merge-approved-62 merge-merged-3202
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce1d259a434311632300857ffa1c1c655140b6f3

commit ce1d259a434311632300857ffa1c1c655140b6f3
Author: Xiaochu Liu <xiaochu@chromium.org>
Date: Tue Nov 07 22:01:20 2017

Block component updater in M62 for kernel 3.8 and 3.10

This is a temporary fix.

BUG= chromium:779308 
TEST=tested on kefka

Change-Id: I177b2b3887b307b63dfa0afe585b4471ddfc42d5
Reviewed-on: https://chromium-review.googlesource.com/745303
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Greg Kerr <kerrnel@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/branch-heads/3202@{#780}
Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098}
[modify] https://crrev.com/ce1d259a434311632300857ffa1c1c655140b6f3/chrome/browser/component_updater/cros_component_installer.cc

Status: Fixed (was: Assigned)
thanks everyone!

Comment 12 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 13 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment