Hi Sergey, Thanks for the review. I would push the fix into 5.5 if there are no buildbot failures. There seems to be something wrong with solaris though. Need to #ifdef the __sync_synchronize() there. The patch generally applies to 5.5 as well. Since we're having a 5.5 week next, I'll test it for 5.5 as well, then push. Vicentiu On Tue, 2 Aug 2016 at 13:47 Sergey Vojtovich <svoj@mariadb.org> wrote:

Hi Vicentiu,

Ok to push. This patch is against 10.2, do you plan to fix previous versions separately?

On Mon, Aug 01, 2016 at 02:46:03PM +0300, vicentiu wrote:

...

revision-id: 5486f3458859fa4bd161150b9ac0e9c0f633a0e3 (mariadb-10.2.1-9-g5486f34) parent(s): 08683a726773f8cdf16a4a3dfb3920e5f7842481 author: Vicențiu Ciorbaru committer: Vicențiu Ciorbaru timestamp: 2016-08-01 14:43:41 +0300 message:

MDEV-10341: InnoDB: Failing assertion: mutex_own(mutex) - mutex_exit_func

Fix memory barrier issues on releasing mutexes. We must have Looks like a cutted comment.

...

--- storage/innobase/include/os0sync.h | 17 ++++++++--------- storage/innobase/include/sync0sync.ic | 2 ++ storage/xtradb/include/os0sync.h | 16 ++++++++-------- storage/xtradb/include/sync0sync.ic | 2 ++ 4 files changed, 20 insertions(+), 17 deletions(-)

Thanks, Sergey

_______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to : maria-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~maria-developers More help : https://help.launchpad.net/ListHelp