Lag due to threads swapping

Question

I have come across some recent lag spikes in a game I have been developing. It is consistent, happens around the same time. Using the java profiler jvisualvm I have found it occurs at the same time a particular thread seems to restart or something (AWT-EventQueue-0):

Other than that, there is no visible cause, not in heap usage, processor use, memory space, or method uses. It will sometimes cause a ConcurrentModificationException when drawing my array of objects, but this should only happen with substantial lag, and my game is hardly intensive.

I don't recall performing any recent changes to the project, however I have carried out the following recently:

• Updated java to the latest version
• Downloaded latest version of JDK7 (though it is not being used in this project)
• Fixed bug with eclipse that occurred as a result of installing JDK7 (removed 256m limit in eclipse.ini)

I am running Eclipse Indigo-service-1 on 32 bit XP. My processors are barely used.

Answer 1

It seems you are doing too much on the Event Dispatch Thread (EDT). AWT-Event-Queue-0 looks to be the EDT. Additionally, your last comment says

...it seems the lag spike only occurs when I draw my game board to an image first rather than directly to the component.

You'll need to push some of your computations to other threads, and it sounds like drawing the game board is a good choice for this. Also, any AI you may have.

Your keyboard & mouse handlers run on the EDT, and the graphics updates need to too. But you can pre-render to an image (like you are currently doing) outside the EDT. And you can send the keyboard & mouse events to another thread via a BlockingQueue.

One other thing you could do is decouple your game-update rate from your frame-update rate.

But without any details, I can't give much more advice.

Update: (just read your bit about ConcurrentModificationException)

This can be caused by two different things:

1. you are updating a collection (like your ArrayList) in a different thread from the one you are reading it in; or
2. you are iterating through said collection and updating it in the loop.

Point 2 is easy to fix; but I'm afraid I can't teach thread safety in such a short amount of space.