Project

General

Profile

Actions
Copy link

Bug #4237

closed

krbd: ignore zero-length requests

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

Status:
Resolved
Priority:
Normal
Assignee:
Alex Elder
Target version:
v0.58
% 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

The old request code simply ignored zero-length requests
as soon as they were seen. The new request code currently
sends zero-length request out to the osd's. In order to
minimize behavioral changes in the new code, make zero-length
requests get ignored.

I've created http://tracker.ceph.com/issues/4236 to suggest
taking another look at this, specifically as it relates
to FLUSH operations.

Actions
Copy link
 #1

Updated by Alex Elder about 11 years ago

  • Status changed from In Progress to Fix Under Review
  • Target version set to v0.58

Posted for review.

commit 98a4b33438b09a70145fe53531c265418bc83a5f
Author: Alex Elder <elder@inktank.com>
Date:   Wed Feb 20 21:59:33 2013 -0600

    rbd: ignore zero-length requests

    The old request code simply ignored zero-length requests.  We should
    still operate that same way to avoid any changes in behavior.  We
    can implement handling for special zero-length requests separately
    (see http://tracker.ceph.com/issues/4236).

    Add some assertions based on this new constraint.

    This resolves:
        http://tracker.ceph.com/issues/4237

    Signed-off-by: Alex Elder <elder@inktank.com>
Actions
Copy link
 #2

Updated by Alex Elder about 11 years ago

  • Status changed from Fix Under Review to Resolved

This has been committed.

commit 4dda41d3d76747414586a4bad5615b550e0986b1
Author: Alex Elder <>
Date: Wed Feb 20 21:59:33 2013 -0600

rbd: ignore zero-length requests
Actions
Copy link

Also available in: Atom PDF