specs icon indicating copy to clipboard operation
specs copied to clipboard

What is a FaultSet, why does a submitPoSt message allow for multiple FaultSet, and how do they "stack"?

Open laser opened this issue 6 years ago • 6 comments

The submitPoSt message includes a parameter faults of type []FaultSet.

The GeneratePost operation can return a slice of faults.

Couple of questions:

  1. What is a FaultSet? Is it just a set of the faulty sector ids returned from an invocation of GeneratePost?
  2. The type of faults is []FaultSet. Under what circumstances would someone send a submitPoSt message containing more than one FaultSet?
  3. The spec says that the miner actor needs to "ensure the fault sets properly stack." What does this mean?

laser avatar Jan 25 '19 22:01 laser

cc @ZenGround0 @whyrusleeping

laser avatar Jan 25 '19 22:01 laser

A faultset is a bitfield and a timestamp. If a miner does not have some of their data when they need to prove it for their post, they mark it in the faultset as missing, and denote the timestamp (blockheight, or challenge index, not 100% sure yet) that the failed challenge occurred at. Miners must pay a fee for every failed sector. If the failure was only temporary (say they had to replace a disk or something) the miner can also mark the sectors in the 'recovered' set so that they don't get removed from the proving set.

Ensuring the fault sets properly stack means that no fault set overlaps with any other fault set.

The reason you need several fault sets and not just one, is because in order to properly validate the PoSt, you need to know when the failed challenges happened.

whyrusleeping avatar Jan 25 '19 23:01 whyrusleeping

Needs documentation in the spec

pooja avatar Apr 05 '19 19:04 pooja

further work here relies on the completion of #133

whyrusleeping avatar Apr 09 '19 00:04 whyrusleeping

Moving forward with usage of rle+. Just needs to be written into the spec. @dignifiedquire Could you help with the spec'ing part?

pooja avatar Apr 17 '19 22:04 pooja

Separate to the bitfield representation, the spec needs content similar to @whyrusleeping's comment above. This was also quite opaque to me.

Go-filecoin needs this to be specified so that the implementation can compute the appropriate fee to attach for temporary sector failures.

anorth avatar Jun 05 '19 03:06 anorth