On 09/03/2013 03:47 PM, Manfred Spraul wrote: > Hi Vineet, > > On 09/03/2013 11:51 AM, Vineet Gupta wrote: >> On 09/03/2013 02:53 PM, Manfred Spraul wrote: >>> The access to msq->q_cbytes is not protected. >>> >>> Vineet, could you try to move the test for free space after ipc_lock? >>> I.e. the lock must not get dropped between testing for free space and >>> enqueueing the messages. >> Hmm, the code movement is not trivial. I broke even the simplest of cases (patch >> attached). This includes the additional change which Linus/Davidlohr had asked for. > The attached patch should work. Could you try it? > Yes this did the trick, now the default config of 100k iterations + 16 processes runs to completion. Thx, -Vineet -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
