Project

General

Profile

Actions

Bug #3875

closed

osd_client: don't use r_num_pages for bio requests

Added by Alex Elder over 11 years ago. Updated about 11 years ago.

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

0%

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

Description

There is an osd request field "r_num_pages" that's used
to record the number of pages supplied with the request.
For page-oriented requests this makes perfect sense.

But for bio requests it doesn't. The length of space
available in the complete bio chain is instead what
should be recorded.

I hit this because the r_num_pages field is checked
when data is coming back on a read request. The osd
client fails saying, in effect, we don't have enough
pages ready to receive the data.

For now I'll just assign a value that works for the
bio case.

Actions #1

Updated by Alex Elder about 11 years ago

  • Status changed from New to Resolved

commit 06224afd90f261256b1e0a0db2334f39c21872a9
Author: Alex Elder <>
Date: Thu Jan 31 16:02:00 2013 -0600

libceph: don't require r_num_pages for bio requests
Actions #2

Updated by Alex Elder about 11 years ago

Sorry, I made a mistake and had to rebase.

commit 012d5bda1c0f229494c67098d00edfa24c531ea5
Author: Alex Elder <>
Date: Thu Jan 31 16:02:00 2013 -0600

libceph: don't require r_num_pages for bio requests
Actions

Also available in: Atom PDF