diff options
author | Jeremy Roman <jbroman@chromium.org> | 2015-09-23 19:26:26 -0400 |
---|---|---|
committer | Jeremy Roman <jbroman@chromium.org> | 2015-09-23 23:28:49 +0000 |
commit | 55c3216b15cec4ae024d16960f01ca7d3874dd55 (patch) | |
tree | 257b2469e8f127efbd56cf4b0054d0f4d72ec059 /sql/statement.cc | |
parent | b388c75b293fb4b0ae394ff75683416aa0a8c5ab (diff) | |
download | chromium_src-55c3216b15cec4ae024d16960f01ca7d3874dd55.zip chromium_src-55c3216b15cec4ae024d16960f01ca7d3874dd55.tar.gz chromium_src-55c3216b15cec4ae024d16960f01ca7d3874dd55.tar.bz2 |
Revert of [sql] Use memory-mapped I/O for sql::Connection. (patchset #8 id:140001 of https://codereview.chromium.org/1349863003/ )
Reason for revert:
mmap_enabled_ isn't initialized, causing MSAN failures:
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_MSAN/5882/layout-test-results/virtual/sharedarraybuffer/fast/workers/constructor-proto-crash-log.txt
STDERR: ==3138==WARNING: MemorySanitizer: use-of-uninitialized-value
STDERR: #0 0x7fc8068d3a65 in ReleaseCacheMemoryIfNeeded sql/connection.cc:513:7
STDERR: #1 0x7fc8068d3a65 in sql::Connection::ExecuteAndReturnErrorCode(char const*) sql/connection.cc:943:0
STDERR: #2 0x7fc8068ca454 in sql::Connection::OpenInternal(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, sql::Connection::Retry) sql/connection.cc:1275:9
STDERR: #3 0x7fc8068c845f in sql::Connection::Open(base::FilePath const&) sql/connection.cc:367:10
STDERR: #4 0x7fc806b1b868 in storage::QuotaDatabase::LazyOpen(bool) storage/browser/quota/quota_database.cc:488:14
Original issue's description:
> [sql] Use memory-mapped I/O for sql::Connection.
>
> sql::Connection::Open*() uses PRAGMA mmap_size to enable SQLite's
> memory-mapped I/O. Additionally instrument to flush dirty pages from
> the page cache after writes.
>
> BUG=489784,533682
>
> Committed: https://crrev.com/9a1948a4d6d445d5c8e209bdcd1cd050af72060b
> Cr-Commit-Position: refs/heads/master@{#350362}
R=shess@chromium.org
TBR=pavely@chromium.org, pvalenzuela@chromium.org, rmcilroy@chromium.org, shess@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=489784,533682
Review URL: https://codereview.chromium.org/1365783002 .
Cr-Commit-Position: refs/heads/master@{#350386}
Diffstat (limited to 'sql/statement.cc')
-rw-r--r-- | sql/statement.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/sql/statement.cc b/sql/statement.cc index 2a0faa7..09c95da 100644 --- a/sql/statement.cc +++ b/sql/statement.cc @@ -110,11 +110,6 @@ void Statement::Reset(bool clear_bound_vars) { ref_->connection()->RecordOneEvent(Connection::EVENT_STATEMENT_SUCCESS); } - // Potentially release dirty cache pages if an autocommit statement made - // changes. - if (ref_->connection()) - ref_->connection()->ReleaseCacheMemoryIfNeeded(false); - succeeded_ = false; stepped_ = false; } |