Skip to content

Commit

Permalink
sealing: Fix restartSectors race
Browse files Browse the repository at this point in the history
  • Loading branch information
magik6k authored and arajasek committed Jun 15, 2021
1 parent 520a009 commit 81b4123
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 0 deletions.
3 changes: 3 additions & 0 deletions extern/storage-sealing/fsm.go
Original file line number Diff line number Diff line change
Expand Up @@ -514,6 +514,8 @@ func planCommitting(events []statemachine.Event, state *SectorInfo) (uint64, err
}

func (m *Sealing) restartSectors(ctx context.Context) error {
defer m.startupWait.Done()

trackedSectors, err := m.ListSectors()
if err != nil {
log.Errorf("loading sector list: %+v", err)
Expand All @@ -531,6 +533,7 @@ func (m *Sealing) restartSectors(ctx context.Context) error {
}

func (m *Sealing) ForceSectorState(ctx context.Context, id abi.SectorNumber, state SectorState) error {
m.startupWait.Wait()
return m.sectors.Send(id, SectorForceState{state})
}

Expand Down
2 changes: 2 additions & 0 deletions extern/storage-sealing/garbage.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ import (
)

func (m *Sealing) PledgeSector(ctx context.Context) (storage.SectorRef, error) {
m.startupWait.Wait()

m.inputLk.Lock()
defer m.inputLk.Unlock()

Expand Down
4 changes: 4 additions & 0 deletions extern/storage-sealing/input.go
Original file line number Diff line number Diff line change
Expand Up @@ -376,6 +376,8 @@ func (m *Sealing) updateInput(ctx context.Context, sp abi.RegisteredSealProof) e
}

func (m *Sealing) tryCreateDealSector(ctx context.Context, sp abi.RegisteredSealProof) error {
m.startupWait.Wait()

cfg, err := m.getConfig()
if err != nil {
return xerrors.Errorf("getting storage config: %w", err)
Expand Down Expand Up @@ -422,6 +424,8 @@ func (m *Sealing) createSector(ctx context.Context, cfg sealiface.Config, sp abi
}

func (m *Sealing) StartPacking(sid abi.SectorNumber) error {
m.startupWait.Wait()

return m.sectors.Send(uint64(sid), SectorStartPacking{})
}

Expand Down
7 changes: 7 additions & 0 deletions extern/storage-sealing/sealing.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,8 @@ type Sealing struct {
feeCfg config.MinerFeeConfig
events Events

startupWait sync.WaitGroup

maddr address.Address

sealer sectorstorage.SectorManager
Expand Down Expand Up @@ -161,6 +163,7 @@ func New(api SealingAPI, fc config.MinerFeeConfig, events Events, maddr address.
bySector: map[abi.SectorID]statSectorState{},
},
}
s.startupWait.Add(1)

s.sectors = statemachine.New(namespace.Wrap(ds, datastore.NewKey(SectorStorePrefix)), s, SectorInfo{})

Expand Down Expand Up @@ -188,10 +191,14 @@ func (m *Sealing) Stop(ctx context.Context) error {
}

func (m *Sealing) Remove(ctx context.Context, sid abi.SectorNumber) error {
m.startupWait.Wait()

return m.sectors.Send(uint64(sid), SectorRemove{})
}

func (m *Sealing) Terminate(ctx context.Context, sid abi.SectorNumber) error {
m.startupWait.Wait()

return m.sectors.Send(uint64(sid), SectorTerminate{})
}

Expand Down

0 comments on commit 81b4123

Please sign in to comment.