maria-developers team mailing list archive
-
maria-developers team
-
Mailing list archive
-
Message #12484
Re: 889f90365ef: MDEV-23536 : Race condition between KILL and transaction commit
Hi, Sergei, Jan,
I have already approved the InnoDB changes.
On Sat, Dec 19, 2020 at 12:43 PM Sergei Golubchik <serg@xxxxxxxxxxx> wrote:
> Hi, Jan!
[snip]
> > + mysql_mutex_lock(&LOCK_thd_kill);
> > + mysql_mutex_unlock(&LOCK_thd_kill);
>
> Note that THD destructor starts from `assert_not_linked()` -
> it's important, because if the THD is linked in the list of threads,
> then someone might start using it after you released LOCK_thd_kill.
>
> With the new logic you need to move assert_not_linked to
> free_connection().
>
> otherwise ok to push
Do you mean that also the empty mutex lock/unlock pair should be
removed from THD::~THD()? It was copied (not moved) from there in this
patch.
Marko
--
Marko Mäkelä, Lead Developer InnoDB
MariaDB Corporation
Follow ups
References