Halide icon indicating copy to clipboard operation
Halide copied to clipboard

[Hexagon] Host API for changing stack size

Open aankit-ca opened this issue 4 years ago • 12 comments

@dsharletg @pranavb-ca Users will have to call halide_hexagon_set_thread_params with thread priority and stack size before any RPC call.

aankit-ca avatar Feb 05 '21 01:02 aankit-ca

Looks like this PR still needs some cleanup before it can land?

steven-johnson avatar Feb 12 '21 17:02 steven-johnson

Where does this PR stand?

steven-johnson avatar Feb 17 '21 18:02 steven-johnson

@steven-johnson The PR needs some changes. Unfortunately my dev machine is down due to power outage since last week. I'll update the PR soon.

aankit-ca avatar Feb 17 '21 18:02 aankit-ca

@steven-johnson & @dsharletg - Due to the winter storm that Texas saw last week, our office where all our hardware for building and testing is saw a massive power outage. As if that wasn't enough, a sprinkler pipe burst caused flooding in our office. As such we are scrambling to bring up some testing infrastructure for our group. @aankit-ca will post an update on this PR once we have some capability restored.

pranavb-ca avatar Feb 23 '21 16:02 pranavb-ca

Thanks for the update!

steven-johnson avatar Feb 23 '21 17:02 steven-johnson

This PR has been sitting around for a few months now; is it still active?

steven-johnson avatar Jun 08 '21 16:06 steven-johnson

@steven-johnson Sorry this PR fell through the cracks. But I think we need this PR. I'll resume working on this.

aankit-ca avatar Sep 15 '21 18:09 aankit-ca

@steven-johnson Sorry this PR fell through the cracks. But I think we need this PR. I'll resume working on this.

Thanks for the update!

steven-johnson avatar Sep 15 '21 18:09 steven-johnson

We now have another stack usage regression causing crashes. Is this PR something that is ready to merge early this week?

dsharletg avatar Sep 20 '21 03:09 dsharletg

We now have another stack usage regression causing crashes. Is this PR something that is ready to merge early this week?

Saw this late. I've most of the changes ready. Will put them up for review soon.

aankit-ca avatar Sep 28 '21 15:09 aankit-ca

@dsharletg @pranavb-ca PTAL

aankit-ca avatar Oct 06 '21 19:10 aankit-ca

@dsharletg @pranavb-ca Ping

aankit-ca avatar Oct 14 '21 20:10 aankit-ca