mirror of
https://github.com/MariaDB/server.git
synced 2025-01-16 20:12:31 +01:00
20acfbf30d
Added locking of lock mutex when updating status in external_unlock() for Aria and MyISAM tables. Fixed that 'source' command doesn't cause mysql command line tool to exit on error. DEBUG_EXECUTE() and DEBUG_EVALUATE_IF() should not execute things based on wildcards. (Allows one to run --debug with mysql-test-run scripts that uses @debug) Fixed several core dump, deadlock and crashed table bugs in handling of LOCK TABLE with MERGE tables: - Added priority of locks to avoid crashes with MERGE tables. - Added thr_lock_merge() to allow one to merge two results of thr_lock(). Fixed 'not found row' bug in REPLACE with Aria tables. Mark MyISAM tables that are part of MERGE with HA_OPEN_MERGE_TABLE and set the locks to have priority THR_LOCK_MERGE_PRIV. - By sorting MERGE tables last in thr_multi_unlock() it's safer to release and relock them many times (can happen when TRIGGERS are created) Avoid printing (null) in debug file (to easier find out wrong NULL pointer usage with %s). client/mysql.cc: Fixed that 'source' command doesn't cause mysql command line tool to exit on error. client/mysqltest.cc: Don't send NULL to fn_format(). (Can cause crash on Solaris when using --debug) dbug/dbug.c: DEBUG_EXECUTE() and DEBUG_EVALUATE_IF() should not execute things based on wildcards. include/my_base.h: Added flag to signal if one opens a MERGE table. Added extra() command to signal that one is not part of a MERGE table anymore. include/thr_lock.h: Added priority for locks (needed to fix bug in thr_lock when using MERGE tables) Added option to thr_unlock() if get_status() should be called. Added prototype for thr_merge_locks(). mysql-test/mysql-test-run.pl: Ignore crashed table warnings for tables named 'crashed'. mysql-test/r/merge.result: Renamed triggers to make debugging easier. Added some CHECK TABLES to catch errors earlier. Additional tests. mysql-test/r/merge_debug.result: Test of error handling when reopening MERGE tables. mysql-test/r/udf_query_cache.result: Added missing flush status mysql-test/suite/parts/r/partition_repair_myisam.result: Update results mysql-test/t/merge.test: Renamed triggers to make debugging easier. Added some CHECK TABLES to catch errors earlier. Additional tests. mysql-test/t/merge_debug.test: Test of error handling when reopening MERGE tables. mysql-test/t/udf_query_cache.test: Added missing flush status mysys/my_getopt.c: Removed not used variable mysys/my_symlink2.c: Changed (null) to (NULL) to make it easier to find NULL arguments to DBUG_PRINT() functions. (On linux, NULL to sprintf is printed 'null') mysys/thr_lock.c: Added priority of locks to avoid crashes with MERGE tables. Added thr_lock_merge() to allow one to merge two results of thr_lock(). - This is needed for MyISAM as all locked table must share the same status. If not, you will not see newly inserted rows in other instances of the table. If calling thr_unlock() with THR_UNLOCK_UPDATE_STATUS, call update_status() and restore_status() for the locks. This is needed in some rare cases where we call thr_unlock() followed by thr_lock() without calling external_unlock/external_lock in between. Simplify loop in thr_multi_lock(). Added 'start_trans', which is called at end of thr_multi_lock() when all locks are taken. - This was needed by Aria to ensure that transaction is started when we got all locks, not at get_status(). Without this, some rows could not be visible when we lock two tables at the same time, causing REPLACE using two tables to fail unexpectedly. sql/handler.cc: Add an assert() in handler::print_error() for "impossible errors" (like table is crashed) when --debug-assert-if-crashed-table is used. sql/lock.cc: Simplify mysql_lock_tables() code if get_lock_data() returns 0 locks. Added new parameter to thr_multi_unlock() In mysql_unlock_read_tables(), call first externa_unlock(), then thr_multi_unlock(); This is same order as we do in mysql_unlock_tables(). Don't abort locks in mysql_lock_abort() for merged tables when a MERGE table is deleted; Would cause a spin lock. Added call to thr_merge_locks() in mysql_lock_merge() to ensure consistency in thr_locks(). - New locks of same type and table is stored after the old lock to ensure that we get the status from the original lock. sql/mysql_priv.h: Added debug_assert_if_crashed_table sql/mysqld.cc: Added --debug-assert-if-crashed-table sql/parse_file.cc: Don't print '(null)' in DBUG_PRINT of no dir given sql/set_var.cc: Increase default size of buffer for @debug variable. sql/sql_base.cc: In case of error from reopen_table() in reopen_tables(), call unlock_open_table() and restart loop. - This fixed bug when we twice deleted same table from open_cache. Don't take name lock for already name locked table in open_unireg_entry(). - Fixed bug when doing repair in reopen_table(). - In detach_merge_children(), always detach if 'clear_refs' is given. We can't trust parent->children_attached as this function can be called twice, first time with clear_refs set to 0. sql/sql_class.cc: Changed printing of (null) to "" in set_thd_proc_info() sql/sql_parse.cc: Added DBUG sql/sql_trigger.cc: Don't call unlink_open_table() if reopen_table() fails as the table may already be freed. storage/maria/ma_bitmap.c: Fixed DBUG_ASSERT() in allocate_tail() storage/maria/ma_blockrec.c: Fixed wrong calculation of row length for very small rows in undo_row_update(). - Fixes ASSERT() when doing undo. storage/maria/ma_blockrec.h: Added _ma_block_start_trans() and _ma_block_start_trans_no_versioning() storage/maria/ma_locking.c: Call _ma_update_status_with_lock() when releasing write locks. - Fixes potential problem with updating status without the proper lock. storage/maria/ma_open.c: Changed to use start_trans() instead of get_status() to ensure that we see all rows in all locked tables when we got the locks. - Fixed 'not found row' bug in REPLACE with Aria tables. storage/maria/ma_state.c: Added _ma_update_status_with_lock() and _ma_block_start_trans(). This is to ensure that we see all rows in all locked tables when we got the locks. storage/maria/ma_state.h: Added _ma_update_status_with_lock() storage/maria/ma_write.c: More DBUG_PRINT storage/myisam/mi_check.c: Fixed error message storage/myisam/mi_extra.c: Added HA_EXTRA_DETACH_CHILD: - Detach MyISAM table to not be part of MERGE table (remove flag & lock priority). storage/myisam/mi_locking.c: Call mi_update_status_with_lock() when releasing write locks. - Fixes potential problem with updating status without the proper lock. Change to use new HA_OPEN_MERGE_TABLE flag to test if MERGE table. Added mi_fix_status(), called by thr_merge(). storage/myisam/mi_open.c: Added marker if part of MERGE table. Call mi_fix_status() in thr_lock() for transactional tables. storage/myisam/myisamdef.h: Change my_once_flag to uint, as it stored different values than just 0/1 Added 'open_flag' to store state given to mi_open() storage/myisammrg/ha_myisammrg.cc: Add THR_LOCK_MERGE_PRIV to THR_LOCK_DATA to get MERGE locks sorted after other types of locks. storage/myisammrg/myrg_locking.c: Remove windows specific code. storage/myisammrg/myrg_open.c: Use HA_OPEN_MERGE_TABLE to mi_open(). Set HA_OPEN_MERGE_TABLE for linked MyISAM tables. storage/xtradb/buf/buf0buf.c: Fixed compiler warning storage/xtradb/buf/buf0lru.c: Initialize variable that could be used not initialized.
182 lines
6.2 KiB
C
182 lines
6.2 KiB
C
/* Copyright 2000-2008 MySQL AB, 2008 Sun Microsystems, Inc.
|
|
|
|
This program is free software; you can redistribute it and/or modify
|
|
it under the terms of the GNU General Public License as published by
|
|
the Free Software Foundation; version 2 of the License.
|
|
|
|
This program is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
GNU General Public License for more details.
|
|
|
|
You should have received a copy of the GNU General Public License
|
|
along with this program; if not, write to the Free Software
|
|
Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */
|
|
|
|
/* For use with thr_lock:s */
|
|
|
|
#ifndef _thr_lock_h
|
|
#define _thr_lock_h
|
|
#ifdef __cplusplus
|
|
extern "C" {
|
|
#endif
|
|
|
|
#include <my_pthread.h>
|
|
#include <my_list.h>
|
|
|
|
struct st_thr_lock;
|
|
extern ulong locks_immediate,locks_waited ;
|
|
|
|
/*
|
|
Important: if a new lock type is added, a matching lock description
|
|
must be added to sql_test.cc's lock_descriptions array.
|
|
*/
|
|
enum thr_lock_type { TL_IGNORE=-1,
|
|
TL_UNLOCK, /* UNLOCK ANY LOCK */
|
|
/*
|
|
Parser only! At open_tables() becomes TL_READ or
|
|
TL_READ_NO_INSERT depending on the binary log format
|
|
(SBR/RBR) and on the table category (log table).
|
|
Used for tables that are read by statements which
|
|
modify tables.
|
|
*/
|
|
TL_READ_DEFAULT,
|
|
TL_READ, /* Read lock */
|
|
TL_READ_WITH_SHARED_LOCKS,
|
|
/* High prior. than TL_WRITE. Allow concurrent insert */
|
|
TL_READ_HIGH_PRIORITY,
|
|
/* READ, Don't allow concurrent insert */
|
|
TL_READ_NO_INSERT,
|
|
/*
|
|
Write lock, but allow other threads to read / write.
|
|
Used by BDB tables in MySQL to mark that someone is
|
|
reading/writing to the table.
|
|
*/
|
|
TL_WRITE_ALLOW_WRITE,
|
|
/*
|
|
Write lock, but allow other threads to read.
|
|
Used by ALTER TABLE in MySQL to allow readers
|
|
to use the table until ALTER TABLE is finished.
|
|
*/
|
|
TL_WRITE_ALLOW_READ,
|
|
/*
|
|
WRITE lock used by concurrent insert. Will allow
|
|
READ, if one could use concurrent insert on table.
|
|
*/
|
|
TL_WRITE_CONCURRENT_INSERT,
|
|
/* Write used by INSERT DELAYED. Allows READ locks */
|
|
TL_WRITE_DELAYED,
|
|
/*
|
|
parser only! Late bound low_priority flag.
|
|
At open_tables() becomes thd->update_lock_default.
|
|
*/
|
|
TL_WRITE_DEFAULT,
|
|
/* WRITE lock that has lower priority than TL_READ */
|
|
TL_WRITE_LOW_PRIORITY,
|
|
/* Normal WRITE lock */
|
|
TL_WRITE,
|
|
/* Abort new lock request with an error */
|
|
TL_WRITE_ONLY};
|
|
|
|
enum enum_thr_lock_result { THR_LOCK_SUCCESS= 0, THR_LOCK_ABORTED= 1,
|
|
THR_LOCK_WAIT_TIMEOUT= 2, THR_LOCK_DEADLOCK= 3 };
|
|
|
|
|
|
/* Priority for locks */
|
|
#define THR_LOCK_LATE_PRIV 1 /* For locks to be merged with org lock */
|
|
#define THR_LOCK_MERGE_PRIV 2 /* For merge tables */
|
|
|
|
#define THR_UNLOCK_UPDATE_STATUS 1
|
|
|
|
extern ulong max_write_lock_count;
|
|
extern ulong table_lock_wait_timeout;
|
|
extern my_bool thr_lock_inited;
|
|
extern enum thr_lock_type thr_upgraded_concurrent_insert_lock;
|
|
|
|
/*
|
|
A description of the thread which owns the lock. The address
|
|
of an instance of this structure is used to uniquely identify the thread.
|
|
*/
|
|
|
|
typedef struct st_thr_lock_info
|
|
{
|
|
pthread_t thread;
|
|
my_thread_id thread_id;
|
|
ulong n_cursors;
|
|
} THR_LOCK_INFO;
|
|
|
|
/*
|
|
Lock owner identifier. Globally identifies the lock owner within the
|
|
thread and among all the threads. The address of an instance of this
|
|
structure is used as id.
|
|
*/
|
|
|
|
typedef struct st_thr_lock_owner
|
|
{
|
|
THR_LOCK_INFO *info;
|
|
} THR_LOCK_OWNER;
|
|
|
|
|
|
typedef struct st_thr_lock_data {
|
|
THR_LOCK_OWNER *owner;
|
|
struct st_thr_lock_data *next,**prev;
|
|
struct st_thr_lock *lock;
|
|
pthread_cond_t *cond;
|
|
void *status_param; /* Param to status functions */
|
|
void *debug_print_param;
|
|
enum thr_lock_type type;
|
|
uint priority;
|
|
} THR_LOCK_DATA;
|
|
|
|
struct st_lock_list {
|
|
THR_LOCK_DATA *data,**last;
|
|
};
|
|
|
|
typedef struct st_thr_lock {
|
|
LIST list;
|
|
pthread_mutex_t mutex;
|
|
struct st_lock_list read_wait;
|
|
struct st_lock_list read;
|
|
struct st_lock_list write_wait;
|
|
struct st_lock_list write;
|
|
/* write_lock_count is incremented for write locks and reset on read locks */
|
|
ulong write_lock_count;
|
|
uint read_no_write_count;
|
|
void (*get_status)(void*, my_bool); /* When one gets a lock */
|
|
void (*copy_status)(void*,void*);
|
|
void (*update_status)(void*); /* Before release of write */
|
|
void (*restore_status)(void*); /* Before release of read */
|
|
my_bool (*start_trans)(void*); /* When all locks are taken */
|
|
my_bool (*check_status)(void *);
|
|
void (*fix_status)(void *, void *);/* For thr_merge_locks() */
|
|
my_bool allow_multiple_concurrent_insert;
|
|
} THR_LOCK;
|
|
|
|
|
|
extern LIST *thr_lock_thread_list;
|
|
extern pthread_mutex_t THR_LOCK_lock;
|
|
|
|
my_bool init_thr_lock(void); /* Must be called once/thread */
|
|
#define thr_lock_owner_init(owner, info_arg) (owner)->info= (info_arg)
|
|
void thr_lock_info_init(THR_LOCK_INFO *info);
|
|
void thr_lock_init(THR_LOCK *lock);
|
|
void thr_lock_delete(THR_LOCK *lock);
|
|
void thr_lock_data_init(THR_LOCK *lock,THR_LOCK_DATA *data,
|
|
void *status_param);
|
|
void thr_unlock(THR_LOCK_DATA *data, uint unlock_flags);
|
|
enum enum_thr_lock_result thr_multi_lock(THR_LOCK_DATA **data,
|
|
uint count, THR_LOCK_OWNER *owner);
|
|
void thr_multi_unlock(THR_LOCK_DATA **data,uint count, uint unlock_flags);
|
|
void thr_merge_locks(THR_LOCK_DATA **data, uint org_count, uint new_count);
|
|
void thr_abort_locks(THR_LOCK *lock, my_bool upgrade_lock);
|
|
my_bool thr_abort_locks_for_thread(THR_LOCK *lock, my_thread_id thread);
|
|
void thr_print_locks(void); /* For debugging */
|
|
my_bool thr_upgrade_write_delay_lock(THR_LOCK_DATA *data,
|
|
enum thr_lock_type new_lock_type);
|
|
void thr_downgrade_write_lock(THR_LOCK_DATA *data,
|
|
enum thr_lock_type new_lock_type);
|
|
my_bool thr_reschedule_write_lock(THR_LOCK_DATA *data);
|
|
#ifdef __cplusplus
|
|
}
|
|
#endif
|
|
#endif /* _thr_lock_h */
|