Project

General

Profile

Actions

Bug #2155

closed

ceph: xattr: wrong value assumed for "no preferred PG"

Added by Alex Elder about 12 years ago. Updated about 12 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

In ceph_vxattrcb_file_layout() the layout for a file is printed.

The preferred placement group is extracted from the ceph inode layout,
but the code assumes that a 0 placement group is used to indicate
"no preference." That is incorrect. No preference is represented
by the reserved value (-1).

Actions #1

Updated by Alex Elder about 12 years ago

  • Status changed from New to 7

This has been fixed in this commit:
260ac0e65b ceph: fix three bugs, two in ceph_vxattrcb_file_layout()

The commit will go into the ceph-client/testing branch, and after
some nightly test coverage will be pushed to the master branch.

Actions #2

Updated by Alex Elder about 12 years ago

This got rebased: 3489b42a72a41d477665ab37f196ae9257180abb

This has been sent as part of a pull request to Linus for
Linux 3.4. Will close this for good when I see it's been
accepted into mainline.

Actions #3

Updated by Alex Elder about 12 years ago

  • Status changed from 7 to Resolved

Linus pulled in the changes without any immediate trouble, so
I'm marking this and a few others resolved.

Actions

Also available in: Atom PDF