> On Sep 21, 2015, at 7:44 AM, Tomas Henzl <the...@redhat.com> wrote: > On 16.9.2015 23:32, Matthew R. Ochs wrote: >> The adapter state machine is susceptible to missing and/or >> corrupting state updates at runtime. This can lead to a variety >> of unintended issues and is due to the lack of a serialization >> mechanism to protect the adapter state. >> >> Use an adapter-wide mutex to serialize state changes. > > I've just briefly looked into your code, but it seems to me that > an atomic variable would serve your needs also and might be > more effective resulting in a faster code execution?
Will keep this in mind. > If you keep the mutex way you don't need two mutexes > in cxlflash_afu_sync - you should remove the mutex &sync_active Agreed. Will remove as part of this patch in v3. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
