On 07/18/2012 03:25 PM, levin li wrote: >>>> >>> Should this be explained in the source code where vdi_exist() is >>>> >>> called? >> > >> > How do you think about this? >> > > As I said in the commit log, before the notify message arrives at every node, > a new cluster request adding the VDI just deleted may be issued, > when the notify message arrives, if we try to delete the object list entries > belong to this VDI regardless of whether the VDI exists, it may causes fatal > error when next recovery happens. > I guess Kazutaka means documenting this rationale in the source. Thanks, Yuan |