Skip to content

crush/mapper: fix divide-by-0 in straw2 (hammer)#4321

Closed
liewegas wants to merge 1 commit intoceph:hammerfrom
liewegas:wip-crush-straw2-div0
Closed

crush/mapper: fix divide-by-0 in straw2 (hammer)#4321
liewegas wants to merge 1 commit intoceph:hammerfrom
liewegas:wip-crush-straw2-div0

Conversation

@liewegas
Copy link
Member

@liewegas liewegas commented Apr 9, 2015

If the item weight is 0 we don't want to divide; instead draw a minimal
value.

Fixes: #11357 Reported-by: Yann Dupont yd@objoo.org Tested-by: Yann Dupont
yd@objoo.org Signed-off-by: Sage Weil sage@redhat.com

@liewegas liewegas added this to the hammer milestone Apr 9, 2015
If the item weight is 0 we don't want to divide; instead draw a minimal
value.

Fixes: ceph#11357
Reported-by: Yann Dupont <yd@objoo.org>
Tested-by: Yann Dupont <yd@objoo.org>
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-crush-straw2-div0 branch from 7d236a6 to 64d1e90 Compare April 9, 2015 21:43
@ghost
Copy link

ghost commented Apr 9, 2015

could you cherry pick 64d1e90 instead to help with cross referencing ?

@liewegas
Copy link
Member Author

yeah i'll cherry-pick and push manually

@liewegas liewegas closed this Apr 10, 2015
@liewegas liewegas deleted the wip-crush-straw2-div0 branch April 10, 2015 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant