From: Dmitry Lenev
Date: October 22 2010 8:21am
Subject: bzr commit into mysql-5.5-runtime branch (Dmitry.Lenev:3176)
List-Archive: http://lists.mysql.com/commits/121646
Message-Id: <20101022082158.794CFE5AA8@mockturtle>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0769894700=="

--===============0769894700==
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

#At file:///home/dlenev/src/bzr/mysql-5.5-rt-grl/ based on revid:kostja@stripped

 3176 Dmitry Lenev	2010-10-22
      More changes to draft patch refactoring global read
      lock implementation. Makes GRL yet another type of
      metadata lock and thus exposes it to deadlock detector
      in MDL subsystem.
      
      Solves bugs #54673 "It takes too long to get readlock for
      'FLUSH TABLES WITH READ LOCK'" and #57006 "Deadlock between
      HANDLER and FLUSH TABLES WITH READ LOCK".
      
      Work-in-progress. Extended test coverage. Fixed problem
      exposed in the process.

    modified:
      mysql-test/r/flush_read_lock.result
      mysql-test/t/flush_read_lock.test
      sql/sql_update.cc
=== modified file 'mysql-test/r/flush_read_lock.result'
--- a/mysql-test/r/flush_read_lock.result	2010-10-22 07:16:34 +0000
+++ b/mysql-test/r/flush_read_lock.result	2010-10-22 08:21:39 +0000
@@ -144,8 +144,9 @@ Success: Was not able to run 'alter even
 Success: 'alter event e1 comment 'test'' is blocked by FTWRL active in another connection.
 Success: FTWRL is blocked when 'alter event e1 comment 'test'' is active in another connection.
 #
-# 1.x) The rest of ALTER statements are too special to
-#      be tested here.
+# 1.x) The rest of ALTER statements (ALTER TABLESPACE,
+#      ALTER LOGFILE GROUP and ALTER SERVER) are too
+#      special to be tested here.
 #
 #
 # 2) ANALYZE TABLE statement is compatible with FTWRL.
@@ -385,18 +386,39 @@ Success: Was not able to run 'create use
 Success: 'create user mysqltest_u1' is blocked by FTWRL active in another connection.
 Success: FTWRL is blocked when 'create user mysqltest_u1' is active in another connection.
 #
-# 8.x) The rest of CREATE variants are too special to test here.
+# 8.x) The rest of CREATE variants (CREATE LOGFILE GROUP,
+#      CREATE TABLESPACE and CREATE SERVER) are too special
+#      to test here.
 #
 #
 # 9) PREPARE, EXECUTE and DEALLOCATE PREPARE statements.
 #
 # 9.1) PREPARE statement is compatible with FTWRL as it
 #      doesn't change any data.
+# 
+# 9.1.a) Prepare of simple INSERT statement.
+#
 # Skip last part of compatibility testing as this statement
 # releases metadata locks in non-standard place.
 Success: Was able to run 'prepare stmt1 from 'insert into t1_base values (1)'' under FTWRL.
 Success: Was able to run 'prepare stmt1 from 'insert into t1_base values (1)'' with FTWRL active in another connection.
 #
+# 9.1.b) Prepare of multi-UPDATE. At some point such statements
+#        tried to acquire thr_lock.c locks during prepare phase.
+#        This no longer happens and thus it is compatible with
+#        FTWRL.
+# Skip last part of compatibility testing as this statement
+# releases metadata locks in non-standard place.
+Success: Was able to run 'prepare stmt1 from 'update t1_base, t2_base set t1_base.i= 1 where t1_base.i = t2_base.j'' under FTWRL.
+Success: Was able to run 'prepare stmt1 from 'update t1_base, t2_base set t1_base.i= 1 where t1_base.i = t2_base.j'' with FTWRL active in another connection.
+#
+# 9.1.c) Prepare of multi-DELETE. Again PREPARE of such
+#        statement should be compatible with FTWRL.
+# Skip last part of compatibility testing as this statement
+# releases metadata locks in non-standard place.
+Success: Was able to run 'prepare stmt1 from 'delete t1_base from t1_base, t2_base where t1_base.i = t2_base.j'' under FTWRL.
+Success: Was able to run 'prepare stmt1 from 'delete t1_base from t1_base, t2_base where t1_base.i = t2_base.j'' with FTWRL active in another connection.
+#
 # 9.2) Compatibility of EXECUTE statement depends on statement
 #     to be executed.
 #
@@ -592,7 +614,8 @@ Success: 'drop trigger t1_bi' is blocked
 Success: FTWRL is blocked when 'drop trigger t1_bi' is active in another connection.
 drop trigger t1_bi;
 #
-# 13.x) The rest of DROP variants are too special to test here.
+# 13.x) The rest of DROP variants (DROP TABLESPACE, DROP LOGFILE
+#       GROUP and DROP SERVER) are too special to test here.
 #
 #
 # 14) FLUSH variants.

=== modified file 'mysql-test/t/flush_read_lock.test'
--- a/mysql-test/t/flush_read_lock.test	2010-10-22 07:16:34 +0000
+++ b/mysql-test/t/flush_read_lock.test	2010-10-22 08:21:39 +0000
@@ -183,8 +183,9 @@ let $cleanup_stmt1= alter event e1 comme
 --source include/check_ftwrl_incompatible.inc
 
 --echo #
---echo # 1.x) The rest of ALTER statements are too special to
---echo #      be tested here.
+--echo # 1.x) The rest of ALTER statements (ALTER TABLESPACE,
+--echo #      ALTER LOGFILE GROUP and ALTER SERVER) are too
+--echo #      special to be tested here.
 --echo #
 
 
@@ -493,7 +494,9 @@ let $cleanup_stmt1= drop user mysqltest_
 --source include/check_ftwrl_incompatible.inc
 
 --echo #
---echo # 8.x) The rest of CREATE variants are too special to test here.
+--echo # 8.x) The rest of CREATE variants (CREATE LOGFILE GROUP,
+--echo #      CREATE TABLESPACE and CREATE SERVER) are too special
+--echo #      to test here.
 --echo #
 
 
@@ -502,6 +505,9 @@ let $cleanup_stmt1= drop user mysqltest_
 --echo #
 --echo # 9.1) PREPARE statement is compatible with FTWRL as it
 --echo #      doesn't change any data.
+--echo # 
+--echo # 9.1.a) Prepare of simple INSERT statement.
+--echo #
 let $statement= prepare stmt1 from 'insert into t1_base values (1)';
 let $cleanup_stmt= deallocate prepare stmt1;
 --echo # Skip last part of compatibility testing as this statement
@@ -511,6 +517,30 @@ let $skip_3rd_check= 1;
 let $skip_3rd_check= ;
 
 --echo #
+--echo # 9.1.b) Prepare of multi-UPDATE. At some point such statements
+--echo #        tried to acquire thr_lock.c locks during prepare phase.
+--echo #        This no longer happens and thus it is compatible with
+--echo #        FTWRL.
+let $statement= prepare stmt1 from 'update t1_base, t2_base set t1_base.i= 1 where t1_base.i = t2_base.j';
+let $cleanup_stmt= deallocate prepare stmt1;
+--echo # Skip last part of compatibility testing as this statement
+--echo # releases metadata locks in non-standard place.
+let $skip_3rd_check= 1;
+--source include/check_ftwrl_compatible.inc
+let $skip_3rd_check= ;
+
+--echo #
+--echo # 9.1.c) Prepare of multi-DELETE. Again PREPARE of such
+--echo #        statement should be compatible with FTWRL.
+let $statement= prepare stmt1 from 'delete t1_base from t1_base, t2_base where t1_base.i = t2_base.j';
+let $cleanup_stmt= deallocate prepare stmt1;
+--echo # Skip last part of compatibility testing as this statement
+--echo # releases metadata locks in non-standard place.
+let $skip_3rd_check= 1;
+--source include/check_ftwrl_compatible.inc
+let $skip_3rd_check= ;
+
+--echo #
 --echo # 9.2) Compatibility of EXECUTE statement depends on statement
 --echo #     to be executed.
 --echo #
@@ -755,7 +785,8 @@ let $cleanup_stmt1= create trigger t1_bi
 drop trigger t1_bi;
 
 --echo #
---echo # 13.x) The rest of DROP variants are too special to test here.
+--echo # 13.x) The rest of DROP variants (DROP TABLESPACE, DROP LOGFILE
+--echo #       GROUP and DROP SERVER) are too special to test here.
 --echo #
 
 

=== modified file 'sql/sql_update.cc'
--- a/sql/sql_update.cc	2010-10-07 10:01:51 +0000
+++ b/sql/sql_update.cc	2010-10-22 08:21:39 +0000
@@ -1026,9 +1026,17 @@ int mysql_multi_update_prepare(THD *thd)
   /* following need for prepared statements, to run next time multi-update */
   thd->lex->sql_command= SQLCOM_UPDATE_MULTI;
 
-  /* open tables and create derived ones, but do not lock and fill them */
+  /*
+    Open tables and create derived ones, but do not lock and fill them yet.
+
+    During prepare phase acquire only S metadata locks instead of SW locks to
+    keep prepare of multi-UPDATE compatible with concurrent LOCK TABLES WRITE
+    and global read lock.
+  */
   if ((original_multiupdate &&
-       open_tables(thd, &table_list, &table_count, 0)) ||
+       open_tables(thd, &table_list, &table_count,
+                   (thd->stmt_arena->is_stmt_prepare() ?
+                    MYSQL_OPEN_FORCE_SHARED_MDL : 0))) ||
       mysql_handle_derived(lex, &mysql_derived_prepare))
     DBUG_RETURN(TRUE);
   /*


--===============0769894700==
MIME-Version: 1.0
Content-Type: text/bzr-bundle; charset="us-ascii";
	name="bzr/dmitry.lenev@stripped"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: dmitry.lenev@stripped
# target_branch: file:///home/dlenev/src/bzr/mysql-5.5-rt-grl/
# testament_sha1: 6a43a9e5948a9790e3050d0579cd441edf43124e
# timestamp: 2010-10-22 12:21:58 +0400
# base_revision_id: kostja@stripped
# 
# Begin bundle
IyBCYXphYXIgcmV2aXNpb24gYnVuZGxlIHY0CiMKQlpoOTFBWSZTWcTCcoUABSD/gFwQwABd9///
/+//oL////RgC6vmqBe2gHSJLUitNaDIkKAACimEkVPQKek9TNEk/0YJNNTanqbU8qe1R6aJmoep
tQ9TR+qDRoCUhG1JmmU8iGqeSeiB6gANADQAADQZDjJk0aA0aYjI0MQwJo0xBiNBhAAYRU/9VQAA
AAAAAAAAAAAAOMmTRoDRpiMjQxDAmjTEGI0GEABgkiAgTFPIAiMlPanoNKA9EzJMhppk0MamEkSR
TeMVWV6av+eNYxmGcb9vC0abTPTXtlhbrnmZQgV/Rfi9n6Xos07sFOM42zFiRjGKsisWR6kdZIYa
UNdN2b79MQYQvvvK2bqziS4zuH9UmOEZ7C85jFv2/ds1QLUYEKQOIZgGC3LkkjfZuA5m1bgZSFuv
Naxr1JmZMzMkbpeIZsMc3YshXBNAZnpgOVzOeadSJVq2ulWrMcpvuYg1BrKs9YOVK3EGF77j/1RX
VFZYDjmcccwPoV42CvoDiVe0/A8SI/22ZjmF9BGZMoY7Qmf1kTJlRfWu0Mgy7zifsqWjFtPxGMzX
AX4BwLylOOzUIDmHDiX229Cl0o2KKnI4dooqeydoxYFXmWA7jxGXwB/wl4Do/7cqBy6kcFA3BK+4
VzA6WpB8aqSIKIMbMloQIt+0nt7AiKwLwdSMRiAYh/sqcigHSpUgLxTCdcxYi1hlgEzmGIp55BIo
nYgVh1SCIxBg1/obi4hazMzEBlkpu9O1o5D1cxo1tHsdYZOvqzii3DiW8UoAAAAAAAAAAACCCCKF
DREWfAUYiLGS3e+36bj4HpP0F5xbzq91+7p0Zn8sDLMXUgXaGItlPmOkug7RGTDcjxSRAlPO3ZlO
VEptj7tPpuJ+JMPRQvz9W4WMaEPSQcdmYY4hxhLQWlz5ZhcMUCiGSKu6AkIkKR3Hge7sChWrroop
BgwrBmG4er1DRdoNJvmBDjoQzIaToCQ2DBkNAzoQKY1hCKLCTiUe+BFEFxq9YlYV9hr0MQng6vq9
dkOAoqpXD5BRH5ULpuwhfGySmmGUBvs8x5dO64oU3hZSrhFWvwpxq1mlTU2KAjzI2Q4zXtx2Q6kT
8mA/PWG5qLzbH4GpxIEU4g0HMloamsD2FS0EXlEXAwjblruEcR0lhFtxdIbEZy5XXI1GZGx2NQmI
JtLlCjkzl1dhyF10ESmSlGSI3uVpVUImjKxGZloUV5f19cyBfk9LhEVxgbguma7/4/mGMrDRtbvJ
JmijCe4xRQ4S35ZFtL2Ub0SeVG2wuHFoQxgRSrJRxVyRbZbK9ZuZv0NC0m2Qy0g286zfHCJvY8x1
do81PQalDeth3bNTlHJp3iSCvwXMuGm9lxforqVmZDnuNmPOg+QhddLGaoMBUi7CnBKNCGhfUc46
iidfnwkX550MsVcYlLMZGBStmaHRYciwR0EWWbM7y+1iZvoEGEY2sQvNvbslgXFtYNW2bc9U3pQc
tZrJzzNsbSI0QiPEdIqg97qoOgVuNBeYZzbatUxpkL2DfEJ7gwWRjFpnxo6T2KxBRiIuhqNEKinV
rSYe0f2B5V4B2hETT3pj0rUwDIa76Vv6CaTvSLke1BwF2HwJD1jHfENEL7WGIhibUyZm+5BJhfL8
FaULhiRcsj8T4SCgxbt9nHyyd4mF/A2MYcGBxGRspicgKNj5i/EDAkS0fITdPvdcWTNmEy4jE5yF
bfIUCj4nmoVMBmS/sT+5BxKoOwC0/KHzaP+LddLizggwArc4um4X5gwxqKo6heBGKShcROoB+htX
NhbQHFVIp5RhXGMRfpFHFGQ6DTBBMY0AgaASF0OoXSoyYWWNWAMGHp8gpyzYt25fUJkkwt4d5r7D
e5RMB9JMUD5xe9nFiQelicXeQmj6GWBMuMDNUeRT3fJDlyCIPuCUwStD2wigWCMCA8qYJ8zjP3n1
DIqaqNLhHiygG101AoOcD5uIZiCgm9DIPcDB7/YEgsySTDJK4NwTLRVjyICxEookh/h23klZ4WH6
ez1+BoNTYz8jMJKU+hEtgeKICOgQSYoxxJ/XU859h5jxSDeb/ikECA61/yaG0ZqNk+BDXiVQSgbh
Qbe+IONrCmrRrD4E0ETMhcwicBOBAF+ZU2IDjagtGTNNfch7CAzFNB3AcZQCA2YMQTHnPFc+fEkd
x9ppai1FUNYcznNKYYCmDC84unEp1Kf1JikDwxsRnmTXM7wZcbN7Z9HIG/TzHZFRXpPKZUSPQUOt
KiiBATCsGLAYUU6M4QEwmLA9lFMFFAwwto6kmsLjYVVTvGnmqNaxq/Cy5noCUTbCkiYU/DDgYjE+
w0DdySKhSi0HIqqs+zluNROZiz0YqGp+O8y2mc19evPJrvycsDRYbN6Q7BLgTh7ZCGidgSYZgZNx
TA6KMkFhYdgz+T0l9kgUOud27ZSSLYx16Yaoy2IgJkOwnUwPXZanmKKw89wKovX8VyVfBCLzZNqs
Mzj20XN1GyToNQ5GF5gzhb5LnXOEp17UzTmaJZWQde49AjSRRoH+4jLmFBSrVl8/Xez6Tdceb4mZ
kMHeAzAN67zmpL5RfIG2gniG4VBTIBaiaSLTOKO4R1gGAoJdbeaCqLgiKQ2/bzSaBGgcEHeDQNhm
Ych0rJYEDoKcQuCPM+3t40vqmRYB4HYrT1i9nEXliutL1IUAuM/o3GxS1FznbKihFtqU7AHA9lFZ
ym28XvDAHTYjQicCDiKhowZmIXkxS7YhkTXcd4cg3I/edhdI8VigyBMOmZM5eF8JchDncBJYm6wN
Gk5f/A0Myqm9QGSJ6E3RidYu/sfdfjJAzADNw1TmSiko55Yof2nUI44dpd6r3SLgsWEsgV6NUT5m
WoWCHa1mSgh3FAooFzEXEnEwRkaEiAKSB1Km8gqpRWuzsLCNjpMXOCdggDIYddDv0zvO5JDpPkbo
BAGChu5GLDMkhs0IbrPK6DogPIMCywBkoNDWtiaBelX7Gzh+oJhe9vfFYwTG1GIade4w6hvaWGgc
d00QDkaySDsdMhkTSIi6OHKovpx6hsrEqJiPgdhdKQe4Zhl3sd2MzdpYQEbskt6NMd67dDcK1smR
1nAvJgqScXdsLAtR5glBKxkignHScWpeJqMiN7IPrNjAwLGQvBbfCAXFQu2mzJSf4i51v/nsKt2o
LXkW3Byve3ALXGSGYvYUpA1RtTiWshiCICkEYByLmCs+n8o7KWIAw1ZwxXvNgpBhmpBgJ1mfOHjm
HY3qqZ2nyr5mifcMR8IuOapcWSNQbNkSCgf3GMhgpIqwbkRA0Pa20UiKG3IwdUwQAod6amDEYXFq
BgvjkPcMOvGXhJEyxWQox2gZiS+RyBi7pq+ImiL9SltgnkkFZhFMKkYMoCUOyaPQsBwTbEvcQNEy
D6JIsiyRnvGvDxIsOiTAqgvKMgjATZLuH8hDYCYDkZD05hwmmM8BD9wanDGQN+y/gshFJajEDowJ
ZH8F1qW+35xFD5czf/JaCz8eALIOCzV1DlMX7rD0HdARahVIHcKz1fGIuEYYGCD+/KrWgyFzRgcJ
E9Y+tNkDZqGhpIWkeHAnkIcs/+LuSKcKEhiYTlCg


--===============0769894700==--