summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* rgw: fix hex string translationwip-4755Yehuda Sadeh2013-04-231-7/+7
| | | | | | | | | | | Fixes: #4755 Original issue was that we implicitly assumed that char is signed, which is not the case on all architectures and was breaking arm. This fix avoids the implicit signed declaration and makes the actual test clearer. Reported-by: Adam Borowski <kilobyte@angband.pl> Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* rgw: don't send tail to gc if copying object to itselfYehuda Sadeh2013-04-222-2/+6
| | | | | | | | | Fixes: #4776 Backport: bobtail Need to make sure that when copying an object into itself we don't send the tail to the garbage collection. Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
* Merge pull request #232 from ceph/wip-4710Josh Durgin2013-04-221-4/+8
|\ | | | | Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
| * rbd: only set STRIPINGV2 feature when neededSage Weil2013-04-221-1/+3
| | | | | | | | | | | | | | | | Only set the STRIPINGV2 feature if the striping parameters are non-default. Specifically, fix the case where the passed-in size and count are == 0. Fixes: #4710 Signed-off-by: Sage Weil <sage@inktank.com>
| * rbd: fix feature display for --infoSage Weil2013-04-221-3/+5
| | | | | | | | | | | | | | Only include the feature if it is set! Backport: bobtail Signed-off-by: Sage Weil <sage@inktank.com>
* | Merge pull request #233 from ceph/wip-mon-idempotentSage Weil2013-04-221-3/+3
|\ \ | |/ |/| | | | | Wip mon idempotent Reviewed-by: Dan Mick <dan.mick@inktank.com>
| * mon: make 'osd pool rmsnap ...' idempotentSage Weil2013-04-221-1/+1
| | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * mon: make 'osd pool mksnap ...' idempotentSage Weil2013-04-221-1/+1
| | | | | | | | Signed-off-by: Sage Weil <sage@inktank.com>
| * mon: make 'osd blacklist rm ...' idempotentSage Weil2013-04-221-1/+1
|/ | | | Signed-off-by: Sage Weil <sage@inktank.com>
* rbd: avoid clobbering return value with udevadm settleSage Weil2013-04-221-3/+3
| | | | | Fixes: #4707 Signed-off-by: Sage Weil <sage@inktank.com>
* FileJournal: a valid entry after invalid entry =/=> corruptSamuel Just2013-04-221-33/+0
| | | | | | | | | | Out of order journal entry writes using aio may cause entry n+2 to be written prior to n. This does not indicate corruption. Fixes: #4736 Signed-off-by: Samuel Just <sam.just@inktank.com> Reviewed-by: Greg Farnum <greg@inktank.com>
* radosgw: Fix duplicate 'Content-Type' when using 'response-content-type'Sylvain Munaut2013-04-221-2/+3
| | | | | Signed-off-by: Sylvain Munaut <s.munaut@whatever-company.com> Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
* ceph-create-keys: Don't wait if permission deniedGary Lowell2013-04-211-6/+14
| | | | | | | | If get or create keys returns permssion denied, exit gracefully instead of retrying. Signed-off-by: Gary Lowell <gary.lowell@inktank.com> Reviewed-by: Sage Weil <sage@inktank.com>
* doc: Aesthetic improvements. Removed unnecessary graphic and overrode margin ↵John Wilkins2013-04-201-4/+1
| | | | | | for h3 tag. Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* doc: Added a scenario to PG troubleshooting.John Wilkins2013-04-201-0/+12
| | | | Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* doc: Changed usage to "bucket-name". Description was okay.John Wilkins2013-04-201-1/+1
| | | | | | fixes: #4102 Signed-off-by: John Wilkins <john.wilkins@inktank.com>
* Merge branch 'wip-4201' into nextDavid Zafman2013-04-196-80/+1128
|\ | | | | | | Reviewed-by: Samuel Just <sam.just@inktank.com>
| * tools/ceph-filestore-dump: Implement remove, export and importDavid Zafman2013-04-191-61/+1073
| | | | | | | | | | | | | | | | | | | | Change local names to be clearer Break real_log() into common function get_log() Move infos_oid, biginfo_oid and log_oid to globals for general use Feature: #4201 (osd: data loss: pg export/import/remove) Signed-off-by: David Zafman <david.zafman@inktank.com>
| * tools/ceph-filestore-dump: Error messages lost because stderr is closedDavid Zafman2013-04-191-10/+11
| | | | | | | | | | | | | | | | | | Use cout instead of cerr for command errors Use cerr for debug mode because stderr is avail Output map_epoch in debug mode Fix a message and only for debug mode Signed-off-by: David Zafman <david.zafman@inktank.com>
| * osd: Make clear_temp() public for use by removeDavid Zafman2013-04-191-2/+1
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
| * osd: Add flag to force version write in _write_info()David Zafman2013-04-192-3/+3
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
| * osd: Create static PG::_write_log() functionDavid Zafman2013-04-192-4/+11
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
| * osd: Create new static function PG::_write_info() for use by PG importDavid Zafman2013-04-192-5/+27
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
| * osd: Add OSD::make_infos_oid() as common function to create oidDavid Zafman2013-04-192-1/+5
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
| * filestore, osd: Fixes to comform to programming guidelinesDavid Zafman2013-04-192-3/+6
| | | | | | | | Signed-off-by: David Zafman <david.zafman@inktank.com>
* | Merge branch 'wip_4662_clean' into nextSamuel Just2013-04-1912-109/+343
|\ \ | | | | | | | | | Reviewed-by: Greg Farnum <greg@inktank.com>
| * | ReplicatedPG::_finish_mark_all_unfound_lost: only requeue if !deletingSamuel Just2013-04-191-1/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG::_applied_recovered_object*: don't queue scrub if deletingSamuel Just2013-04-191-2/+3
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: check for pg change in ~FlushStateSamuel Just2013-04-191-1/+2
| | | | | | | | | | | | | | | Fixes: #4662 Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: bail if deleting in _finish_recoverySamuel Just2013-04-191-0/+4
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | AsyncReserver: delete context in cancel_reservationSamuel Just2013-04-191-0/+1
| | | | | | | | | | | | | | | | | | Fixes: #4662 Backport: bobtail Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | OpRequest: don't maintain history if the OSD is shutting downSamuel Just2013-04-191-1/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | osd/: optionally track every pg refSamuel Just2013-04-197-49/+220
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This involves three pieces: For intrusive_ptr type references, we use TrackedIntPtr instead. This uses get_with_id and put_with_id to associate an id and backtrace with each particular ref instance. For refs taken via direct calls to get() and put(), get and put now require a tag string. The PG tracks individual ref counts for each tag as well as the total. Finally, PGs register/unregister themselves on construction/destruction with OSDService. As a result, on shutdown, we can check for live pgs and determine where the references are held. This behavior is compiled out by default, but can be included with the --enable-pgrefdebugging flag. Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | common/: add tracked_int_ptr.hppSamuel Just2013-04-192-0/+68
| | | | | | | | | | | | | | | | | | | | | TrackedIntPtr acts like intrusive_ptr, but is able to track a ref id. Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG: use the ReplicatedPGRef typedefSamuel Just2013-04-191-4/+4
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG: use ReplicatedPGRef for C_PG_MarkUnfoundLostSamuel Just2013-04-191-5/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG: use ReplicatedPGRef for C_OSD_OpCommitSamuel Just2013-04-191-3/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG: use ReplicatedPGRef for C_OSD_OpAppliedSamuel Just2013-04-191-3/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | OSD: use PGRef in handle_pg_removeSamuel Just2013-04-191-2/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | OSD: use PGRef in handle_pg_stats_ackSamuel Just2013-04-191-2/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: use PGRef in QueuePeeringEvtSamuel Just2013-04-191-1/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | OSD: use PGRef in consume_mapSamuel Just2013-04-191-7/+5
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: use PGRef for FlushStateSamuel Just2013-04-191-5/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: use PGRef for C_PG_FinishRecoverySamuel Just2013-04-191-5/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: use PGRef in C_PG_ActivateCommittedSamuel Just2013-04-191-3/+1
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG: do not put() in scrub() if pg is deletingSamuel Just2013-04-191-1/+0
| | | | | | | | | | | | | | | | | | | | | scrub() no longer handles the put, this call must have been missed. Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | PG,ReplicatedPG: move intrusive_ptr declarations to topSamuel Just2013-04-192-10/+10
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | ReplicatedPG: add ReplicatedPGRefSamuel Just2013-04-191-0/+2
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | FileStore::_do_copy_range: read(2) might return EINTRSamuel Just2013-04-191-4/+8
| | | | | | | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>
| * | FileStore::_do_clone_range: _do_copy_range encodes error in return, not errnoSamuel Just2013-04-191-2/+2
| |/ | | | | | | Signed-off-by: Samuel Just <sam.just@inktank.com>