Project

General

Profile

Actions

Bug #3964

closed

krbd: 32-bit, kernel 3.2.0 system can't do O_DIRECT writes to mapped rbd image with snapshot

Added by Dan Mick about 11 years ago. Updated almost 11 years ago.

Status:
Won't Fix
Priority:
Low
Assignee:
-
Target version:
-
% Done:

0%

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

Description

fghaas reported, I reproduced on a precise 32-bit system:

create an image, map, writes work fine, even with dd oflag=direct
make a snapshot, dd oflag=direct fails

Actions #1

Updated by Dan Mick about 11 years ago

  • Subject changed from 32-bit, kernel 3.2.0 system can't do O_DIRECT writes to mapped rbd image with snapshot to krbd: 32-bit, kernel 3.2.0 system can't do O_DIRECT writes to mapped rbd image with snapshot
Actions #2

Updated by Dan Mick about 11 years ago

rbd create image -s 100
rbd map image
dd if=/dev/zero of=/dev/rbd0 oflag=direct count=10
rbd snap create image@s1
dd if=/dev/zero of=/dev/rbd0 oflag=direct count=10  [<< fails]
rbd snap rm image@s1
dd if=/dev/zero of=/dev/rbd0 oflag=direct count=10
Actions #3

Updated by Sage Weil about 11 years ago

This works fine on current testing 3.6.0-00210-g8cc17ca

Actions #4

Updated by Sage Weil about 11 years ago

added test to suite, commit:c782d2ac531cbb7650968e62f0b24e6136a64359

Actions #5

Updated by Alex Elder about 11 years ago

So is this then a request to port whatever it was that
fixed the problem back to 3.2?

If so, how do we prioritize this?

And Sage, your (Dan's) test script is nice; does it ever
get run (i.e., nightly)?

Actions #6

Updated by Dan Mick about 11 years ago

I don't know if Sage tested 32-bit, or if it matters, and no, that script was just a reproduction scenario; as far as I know we just found out about this last night. I'll check with Josh if this doesn't sound familiar to you, Alex, but, I don't know what our path forward is; I suspect we'd prefer people to upgrade kernels, but, I dunno.

Actions #7

Updated by Dan Mick about 11 years ago

Josh thinks 32-bitness probably doesn't matter, and remembers problems with snapshots that were fixed long ago; I guess it comes down to "do we backport anything to 3.2", and I suspect we'd really rather not...

Actions #8

Updated by Dan Mick about 11 years ago

...and to answer your other question Alex, there's now a workunit test Sage just added
in c782d2ac531cbb7650968e62f0b24e6136a64359

Actions #9

Updated by Ian Colle about 11 years ago

  • Priority changed from Normal to Low
Actions #10

Updated by Sage Weil almost 11 years ago

  • Status changed from New to Won't Fix
Actions

Also available in: Atom PDF