Program Execution gets stuck












0















I'm trying to run the following worker threads in a multi-threaded environment but it seems like all the threads get stuck and none of them are able to proceed.



func executeTask(currentTask task, serial bool, imagePixels map[string]*imagePixelContainer, completedTracker map[string]*partsCompleted,
bufferPixels map[string]*imagePixelContainer, nextEffects map[string]*nextEffect, pl *sync.Mutex, threadid int) {

newPixels := applyEffect(imagePixels[currentTask.image].pixels, currentTask.effect, currentTask.starty, currentTask.endy)
mergeComputation(newPixels, bufferPixels[currentTask.image], currentTask.starty, currentTask.endy)
completedTracker[currentTask.image].mux.Lock()
completedTracker[currentTask.image].completed += 1
if completedTracker[currentTask.image].completed == 3 {
imagePixels[currentTask.image].mux.Lock()
imagePixels[currentTask.image].pixels = duplicate(bufferPixels[currentTask.image].pixels)
imagePixels[currentTask.image].mux.Unlock()
completedTracker[currentTask.image].completed = 0
nextEffects[currentTask.image].mux.Lock()
nextEffects[currentTask.image].index += 1
nextEffects[currentTask.image].mux.Unlock()
}
completedTracker[currentTask.image].mux.Unlock()
}

func mergeComputation(computedPixels Pixel, storing *imagePixelContainer, starty int, endy int) {
storing.mux.Lock()
for y := 0; y < len(computedPixels); y++ {
for x := 0; x < len(computedPixels[y]); x++ {
storing.pixels[starty][x] = computedPixels[y][x]
}
starty += 1
}
storing.mux.Unlock()
}

// Worker threads that get spawned
func worker(que *Queue, nextEffects map[string]*nextEffect, effectorder map[string]string, imagePixels map[string]*imagePixelContainer,
completedTracker map[string]*partsCompleted, bufferPixels map[string]*imagePixelContainer, printlock *sync.Mutex, counter *int, threadid int) {
for !(que.Empty()) {
ctask := que.Pop()
if ctask.effect != "" { // Indicates that the task queue is empty at this moment
nextEffects[ctask.image].mux.Lock()
cond := ctask.effect != effectorder[ctask.image][nextEffects[ctask.image].index]
nextEffects[ctask.image].mux.Unlock()
if cond {
que.Push(ctask)
} else {
executeTask(ctask, false, imagePixels, completedTracker, bufferPixels, nextEffects, printlock, threadid)
}
}
}
*counter -= 1
}


The queue's Push and Pop are thread safe. Each map object has a string key to a pointer struct that holds the data variable and a mutex variable to protect that data. Thanks for the help :)










share|improve this question




















  • 1





    dude , go fmt pleeeeasseee

    – Seaskyways
    Nov 24 '18 at 16:33






  • 3





    I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

    – Gavin
    Nov 24 '18 at 18:45











  • Start by getting a stack trace and seeing where each goroutine is blocked.

    – JimB
    Nov 24 '18 at 20:54
















0















I'm trying to run the following worker threads in a multi-threaded environment but it seems like all the threads get stuck and none of them are able to proceed.



func executeTask(currentTask task, serial bool, imagePixels map[string]*imagePixelContainer, completedTracker map[string]*partsCompleted,
bufferPixels map[string]*imagePixelContainer, nextEffects map[string]*nextEffect, pl *sync.Mutex, threadid int) {

newPixels := applyEffect(imagePixels[currentTask.image].pixels, currentTask.effect, currentTask.starty, currentTask.endy)
mergeComputation(newPixels, bufferPixels[currentTask.image], currentTask.starty, currentTask.endy)
completedTracker[currentTask.image].mux.Lock()
completedTracker[currentTask.image].completed += 1
if completedTracker[currentTask.image].completed == 3 {
imagePixels[currentTask.image].mux.Lock()
imagePixels[currentTask.image].pixels = duplicate(bufferPixels[currentTask.image].pixels)
imagePixels[currentTask.image].mux.Unlock()
completedTracker[currentTask.image].completed = 0
nextEffects[currentTask.image].mux.Lock()
nextEffects[currentTask.image].index += 1
nextEffects[currentTask.image].mux.Unlock()
}
completedTracker[currentTask.image].mux.Unlock()
}

func mergeComputation(computedPixels Pixel, storing *imagePixelContainer, starty int, endy int) {
storing.mux.Lock()
for y := 0; y < len(computedPixels); y++ {
for x := 0; x < len(computedPixels[y]); x++ {
storing.pixels[starty][x] = computedPixels[y][x]
}
starty += 1
}
storing.mux.Unlock()
}

// Worker threads that get spawned
func worker(que *Queue, nextEffects map[string]*nextEffect, effectorder map[string]string, imagePixels map[string]*imagePixelContainer,
completedTracker map[string]*partsCompleted, bufferPixels map[string]*imagePixelContainer, printlock *sync.Mutex, counter *int, threadid int) {
for !(que.Empty()) {
ctask := que.Pop()
if ctask.effect != "" { // Indicates that the task queue is empty at this moment
nextEffects[ctask.image].mux.Lock()
cond := ctask.effect != effectorder[ctask.image][nextEffects[ctask.image].index]
nextEffects[ctask.image].mux.Unlock()
if cond {
que.Push(ctask)
} else {
executeTask(ctask, false, imagePixels, completedTracker, bufferPixels, nextEffects, printlock, threadid)
}
}
}
*counter -= 1
}


The queue's Push and Pop are thread safe. Each map object has a string key to a pointer struct that holds the data variable and a mutex variable to protect that data. Thanks for the help :)










share|improve this question




















  • 1





    dude , go fmt pleeeeasseee

    – Seaskyways
    Nov 24 '18 at 16:33






  • 3





    I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

    – Gavin
    Nov 24 '18 at 18:45











  • Start by getting a stack trace and seeing where each goroutine is blocked.

    – JimB
    Nov 24 '18 at 20:54














0












0








0








I'm trying to run the following worker threads in a multi-threaded environment but it seems like all the threads get stuck and none of them are able to proceed.



func executeTask(currentTask task, serial bool, imagePixels map[string]*imagePixelContainer, completedTracker map[string]*partsCompleted,
bufferPixels map[string]*imagePixelContainer, nextEffects map[string]*nextEffect, pl *sync.Mutex, threadid int) {

newPixels := applyEffect(imagePixels[currentTask.image].pixels, currentTask.effect, currentTask.starty, currentTask.endy)
mergeComputation(newPixels, bufferPixels[currentTask.image], currentTask.starty, currentTask.endy)
completedTracker[currentTask.image].mux.Lock()
completedTracker[currentTask.image].completed += 1
if completedTracker[currentTask.image].completed == 3 {
imagePixels[currentTask.image].mux.Lock()
imagePixels[currentTask.image].pixels = duplicate(bufferPixels[currentTask.image].pixels)
imagePixels[currentTask.image].mux.Unlock()
completedTracker[currentTask.image].completed = 0
nextEffects[currentTask.image].mux.Lock()
nextEffects[currentTask.image].index += 1
nextEffects[currentTask.image].mux.Unlock()
}
completedTracker[currentTask.image].mux.Unlock()
}

func mergeComputation(computedPixels Pixel, storing *imagePixelContainer, starty int, endy int) {
storing.mux.Lock()
for y := 0; y < len(computedPixels); y++ {
for x := 0; x < len(computedPixels[y]); x++ {
storing.pixels[starty][x] = computedPixels[y][x]
}
starty += 1
}
storing.mux.Unlock()
}

// Worker threads that get spawned
func worker(que *Queue, nextEffects map[string]*nextEffect, effectorder map[string]string, imagePixels map[string]*imagePixelContainer,
completedTracker map[string]*partsCompleted, bufferPixels map[string]*imagePixelContainer, printlock *sync.Mutex, counter *int, threadid int) {
for !(que.Empty()) {
ctask := que.Pop()
if ctask.effect != "" { // Indicates that the task queue is empty at this moment
nextEffects[ctask.image].mux.Lock()
cond := ctask.effect != effectorder[ctask.image][nextEffects[ctask.image].index]
nextEffects[ctask.image].mux.Unlock()
if cond {
que.Push(ctask)
} else {
executeTask(ctask, false, imagePixels, completedTracker, bufferPixels, nextEffects, printlock, threadid)
}
}
}
*counter -= 1
}


The queue's Push and Pop are thread safe. Each map object has a string key to a pointer struct that holds the data variable and a mutex variable to protect that data. Thanks for the help :)










share|improve this question
















I'm trying to run the following worker threads in a multi-threaded environment but it seems like all the threads get stuck and none of them are able to proceed.



func executeTask(currentTask task, serial bool, imagePixels map[string]*imagePixelContainer, completedTracker map[string]*partsCompleted,
bufferPixels map[string]*imagePixelContainer, nextEffects map[string]*nextEffect, pl *sync.Mutex, threadid int) {

newPixels := applyEffect(imagePixels[currentTask.image].pixels, currentTask.effect, currentTask.starty, currentTask.endy)
mergeComputation(newPixels, bufferPixels[currentTask.image], currentTask.starty, currentTask.endy)
completedTracker[currentTask.image].mux.Lock()
completedTracker[currentTask.image].completed += 1
if completedTracker[currentTask.image].completed == 3 {
imagePixels[currentTask.image].mux.Lock()
imagePixels[currentTask.image].pixels = duplicate(bufferPixels[currentTask.image].pixels)
imagePixels[currentTask.image].mux.Unlock()
completedTracker[currentTask.image].completed = 0
nextEffects[currentTask.image].mux.Lock()
nextEffects[currentTask.image].index += 1
nextEffects[currentTask.image].mux.Unlock()
}
completedTracker[currentTask.image].mux.Unlock()
}

func mergeComputation(computedPixels Pixel, storing *imagePixelContainer, starty int, endy int) {
storing.mux.Lock()
for y := 0; y < len(computedPixels); y++ {
for x := 0; x < len(computedPixels[y]); x++ {
storing.pixels[starty][x] = computedPixels[y][x]
}
starty += 1
}
storing.mux.Unlock()
}

// Worker threads that get spawned
func worker(que *Queue, nextEffects map[string]*nextEffect, effectorder map[string]string, imagePixels map[string]*imagePixelContainer,
completedTracker map[string]*partsCompleted, bufferPixels map[string]*imagePixelContainer, printlock *sync.Mutex, counter *int, threadid int) {
for !(que.Empty()) {
ctask := que.Pop()
if ctask.effect != "" { // Indicates that the task queue is empty at this moment
nextEffects[ctask.image].mux.Lock()
cond := ctask.effect != effectorder[ctask.image][nextEffects[ctask.image].index]
nextEffects[ctask.image].mux.Unlock()
if cond {
que.Push(ctask)
} else {
executeTask(ctask, false, imagePixels, completedTracker, bufferPixels, nextEffects, printlock, threadid)
}
}
}
*counter -= 1
}


The queue's Push and Pop are thread safe. Each map object has a string key to a pointer struct that holds the data variable and a mutex variable to protect that data. Thanks for the help :)







multithreading go






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 24 '18 at 16:35









peterSO

92.9k14160172




92.9k14160172










asked Nov 24 '18 at 16:06









kir bakkir bak

1614




1614








  • 1





    dude , go fmt pleeeeasseee

    – Seaskyways
    Nov 24 '18 at 16:33






  • 3





    I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

    – Gavin
    Nov 24 '18 at 18:45











  • Start by getting a stack trace and seeing where each goroutine is blocked.

    – JimB
    Nov 24 '18 at 20:54














  • 1





    dude , go fmt pleeeeasseee

    – Seaskyways
    Nov 24 '18 at 16:33






  • 3





    I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

    – Gavin
    Nov 24 '18 at 18:45











  • Start by getting a stack trace and seeing where each goroutine is blocked.

    – JimB
    Nov 24 '18 at 20:54








1




1





dude , go fmt pleeeeasseee

– Seaskyways
Nov 24 '18 at 16:33





dude , go fmt pleeeeasseee

– Seaskyways
Nov 24 '18 at 16:33




3




3





I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

– Gavin
Nov 24 '18 at 18:45





I'm failing to see where exactly you are spawning any goroutines, and please note that a goroutine != a thread. As an aside, mux is usually shorthand for multiplexer, not mutex.

– Gavin
Nov 24 '18 at 18:45













Start by getting a stack trace and seeing where each goroutine is blocked.

– JimB
Nov 24 '18 at 20:54





Start by getting a stack trace and seeing where each goroutine is blocked.

– JimB
Nov 24 '18 at 20:54












0






active

oldest

votes











Your Answer






StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});














draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53459963%2fprogram-execution-gets-stuck%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown

























0






active

oldest

votes








0






active

oldest

votes









active

oldest

votes






active

oldest

votes
















draft saved

draft discarded




















































Thanks for contributing an answer to Stack Overflow!


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53459963%2fprogram-execution-gets-stuck%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

A CLEAN and SIMPLE way to add appendices to Table of Contents and bookmarks

Calculate evaluation metrics using cross_val_predict sklearn

Insert data from modal to MySQL (multiple modal on website)