Ticket #5127 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

Crash running activities while indexing usb

Reported by: marco Owned by: tomeu
Priority: blocker Milestone: Update.1
Component: sugar-datastore Version:
Keywords: relnote, review+ Cc: marco, tomeu
Action Needed: Verified: no
Deployments affected: Blocked By:
Blocking:

Description

To reproduce:

1 Take an usb with a lot of files on it and no .olpc.store. 2 Start sugar, insert the usb stick, launch an activity.

The sugar-datastore process is gone and activities does not start anymore.

From gdb it looks like it's crashing in FlintTable_base::block_free_at_start.

Attachments

index_on_idle.patch (8.0 kB) - added by tomeu 5 years ago.

Change History

  Changed 5 years ago by jg

  • milestone changed from Never Assigned to Update.1

  Changed 5 years ago by kimquirk

  • keywords relnote added

Changed 5 years ago by tomeu

  Changed 5 years ago by tomeu

  • cc marco added
  • keywords relnote, review? added; relnote removed

Marco, can you review, please?

follow-up: ↓ 5   Changed 5 years ago by marco

  • keywords review+ added; review? removed

Looks good, do we actually need to flush for every file?

in reply to: ↑ 4   Changed 5 years ago by tomeu

Replying to marco:

Looks good, do we actually need to flush for every file?

We should do a flush for every change we do, so we do a real flush every X changes or X seconds after last change.

But in that case no flush should be done because an implicit flush happens inside create(). Will take that flush out and push to HEAD for testing in joyride.

  Changed 5 years ago by tomeu

  • owner changed from tomeu to marco

Pushed, can you do an rpm?

  Changed 5 years ago by tomeu

  • cc tomeu added

  Changed 5 years ago by marco

  • owner changed from marco to tomeu

It will be in the next joyride.

  Changed 5 years ago by marco

  • owner changed from tomeu to ApprovalForUpdate

Seem to work well in joyride and I could not reproduce the crash with it.

  Changed 5 years ago by jg

  • owner changed from ApprovalForUpdate to dgilmore

Approved for Update.1

  Changed 5 years ago by marco

  • owner changed from dgilmore to tomeu

The ticket to get this in the build is now #5350. Reassigning to Tomeu for testing when it lands in Update.1.

  Changed 5 years ago by marco

  • status changed from new to closed
  • resolution set to fixed

Verified fixed in 657

Note: See TracTickets for help on using tickets.