/*
- * $Id: DiskdFile.cc,v 1.5 2008/02/26 21:49:40 amosjeffries Exp $
+ * Copyright (C) 1996-2017 The Squid Software Foundation and contributors
*
- * DEBUG: section 79 Squid-side DISKD I/O functions.
- * AUTHOR: Duane Wessels
- *
- * SQUID Web Proxy Cache http://www.squid-cache.org/
- * ----------------------------------------------------------
- *
- * Squid is the result of efforts by numerous individuals from
- * the Internet community; see the CONTRIBUTORS file for full
- * details. Many organizations have provided support for Squid's
- * development; see the SPONSORS file for full details. Squid is
- * Copyrighted (C) 2001 by the Regents of the University of
- * California; see the COPYRIGHT file for full details. Squid
- * incorporates software developed and/or copyrighted by other
- * sources; see the CREDITS file for full details.
- *
- * 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; either version 2 of the License, or
- * (at your option) any later version.
- *
- * 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, USA.
- *
- * CopyRight (c) 2003, Robert Collins <robertc@squid-cache.org>
+ * Squid software is distributed under GPLv2+ license and includes
+ * contributions from numerous individuals and organizations.
+ * Please see the COPYING and CONTRIBUTORS files for details.
*/
-#include "squid.h"
+/* DEBUG: section 79 Squid-side DISKD I/O functions. */
-#include <sys/ipc.h>
-#include <sys/msg.h>
-#include <sys/shm.h>
-
-#include "DiskdFile.h"
+#include "squid.h"
#include "ConfigOption.h"
#include "diomsg.h"
-
+#include "DiskdFile.h"
#include "DiskdIOStrategy.h"
#include "DiskIO/IORequestor.h"
#include "DiskIO/ReadRequest.h"
#include "DiskIO/WriteRequest.h"
-CBDATA_CLASS_INIT(DiskdFile);
+#include "StatCounters.h"
-void *
-DiskdFile::operator new(size_t unused)
-{
- CBDATA_INIT_TYPE(DiskdFile);
- DiskdFile *result = cbdataAlloc(DiskdFile);
- /* Mark result as being owned - we want the refcounter to do the delete
- * call */
- debugs(79, 3, "diskdFile with base " << result << " allocating");
- return result;
-}
+#if HAVE_SYS_IPC_H
+#include <sys/ipc.h>
+#endif
+#if HAVE_SYS_MSG_H
+#include <sys/msg.h>
+#endif
+#if HAVE_SYS_SHM_H
+#include <sys/shm.h>
+#endif
-void
-DiskdFile::operator delete(void *address)
-{
- DiskdFile *t = static_cast<DiskdFile *>(address);
- debugs(79, 3, "diskdFile with base " << t << " deleting");
- cbdataFree(t);
-}
+CBDATA_CLASS_INIT(DiskdFile);
-DiskdFile::DiskdFile(char const *aPath, DiskdIOStrategy *anIO) : errorOccured (false), IO(anIO),
- inProgressIOs (0)
+DiskdFile::DiskdFile(char const *aPath, DiskdIOStrategy *anIO) :
+ errorOccured(false),
+ IO(anIO),
+ mode(0),
+ inProgressIOs(0)
{
- assert (aPath);
+ assert(aPath);
debugs(79, 3, "DiskdFile::DiskdFile: " << aPath);
- path_ = xstrdup (aPath);
- id = diskd_stats.sio_id++;
+ path_ = xstrdup(aPath);
+ id = diskd_stats.sio_id;
+ ++diskd_stats.sio_id;
}
DiskdFile::~DiskdFile()
{
- assert (inProgressIOs == 0);
+ assert(inProgressIOs == 0);
safe_free (path_);
}
void
-DiskdFile::open(int flags, mode_t aMode, RefCount< IORequestor > callback)
+DiskdFile::open(int flags, mode_t, RefCount<IORequestor> callback)
{
debugs(79, 3, "DiskdFile::open: " << this << " opening for " << callback.getRaw());
- assert (ioRequestor.getRaw() == NULL);
+ assert(ioRequestor.getRaw() == NULL);
ioRequestor = callback;
- assert (callback.getRaw());
+ assert(callback.getRaw());
mode = flags;
ssize_t shm_offset;
char *buf = (char *)IO->shm.get(&shm_offset);
ioRequestor = NULL;
}
- diskd_stats.open.ops++;
+ ++diskd_stats.open.ops;
}
void
-DiskdFile::create(int flags, mode_t aMode, RefCount< IORequestor > callback)
+DiskdFile::create(int flags, mode_t, RefCount<IORequestor> callback)
{
debugs(79, 3, "DiskdFile::create: " << this << " creating for " << callback.getRaw());
assert (ioRequestor.getRaw() == NULL);
NULL);
if (x < 0) {
+ int xerrno = errno;
ioCompleted();
errorOccured = true;
// IO->shm.put (shm_offset);
- debugs(79, 1, "storeDiskdSend CREATE: " << xstrerror());
+ debugs(79, DBG_IMPORTANT, "storeDiskdSend CREATE: " << xstrerr(xerrno));
notifyClient();
ioRequestor = NULL;
return;
}
- diskd_stats.create.ops++;
+ ++diskd_stats.create.ops;
}
void
aRead);
if (x < 0) {
+ int xerrno = errno;
ioCompleted();
errorOccured = true;
// IO->shm.put (shm_offset);
- debugs(79, 1, "storeDiskdSend READ: " << xstrerror());
+ debugs(79, DBG_IMPORTANT, "storeDiskdSend READ: " << xstrerr(xerrno));
notifyClient();
ioRequestor = NULL;
return;
}
- diskd_stats.read.ops++;
+ ++diskd_stats.read.ops;
}
void
NULL);
if (x < 0) {
+ int xerrno = errno;
ioCompleted();
errorOccured = true;
- debugs(79, 1, "storeDiskdSend CLOSE: " << xstrerror());
+ debugs(79, DBG_IMPORTANT, "storeDiskdSend CLOSE: " << xstrerr(xerrno));
notifyClient();
ioRequestor = NULL;
return;
}
- diskd_stats.close.ops++;
+ ++diskd_stats.close.ops;
}
bool
void
DiskdFile::openDone(diomsg *M)
{
- statCounter.syscalls.disk.opens++;
+ ++statCounter.syscalls.disk.opens;
debugs(79, 3, "storeDiskdOpenDone: status " << M->status);
if (M->status < 0) {
- diskd_stats.open.fail++;
+ ++diskd_stats.open.fail;
errorOccured = true;
} else {
- diskd_stats.open.success++;
+ ++diskd_stats.open.success;
}
ioCompleted();
void
DiskdFile::createDone(diomsg *M)
{
- statCounter.syscalls.disk.opens++;
+ ++statCounter.syscalls.disk.opens;
debugs(79, 3, "storeDiskdCreateDone: status " << M->status);
if (M->status < 0) {
- diskd_stats.create.fail++;
+ ++diskd_stats.create.fail;
errorOccured = true;
} else {
- diskd_stats.create.success++;
+ ++diskd_stats.create.success;
}
ioCompleted();
debugs(79, 3, "DiskdFile::write: this " << (void *)this << ", buf " << (void *)aRequest->buf << ", off " << aRequest->offset << ", len " << aRequest->len);
ssize_t shm_offset;
char *sbuf = (char *)IO->shm.get(&shm_offset);
- xmemcpy(sbuf, aRequest->buf, aRequest->len);
+ memcpy(sbuf, aRequest->buf, aRequest->len);
if (aRequest->free_func)
aRequest->free_func(const_cast<char *>(aRequest->buf));
aRequest);
if (x < 0) {
+ int xerrno = errno;
ioCompleted();
errorOccured = true;
- debugs(79, 1, "storeDiskdSend WRITE: " << xstrerror());
+ debugs(79, DBG_IMPORTANT, "storeDiskdSend WRITE: " << xstrerr(xerrno));
// IO->shm.put (shm_offset);
notifyClient();
ioRequestor = NULL;
return;
}
- diskd_stats.write.ops++;
+ ++diskd_stats.write.ops;
}
void
void
DiskdFile::closeDone(diomsg * M)
{
- statCounter.syscalls.disk.closes++;
+ ++statCounter.syscalls.disk.closes;
debugs(79, 3, "DiskdFile::closeDone: status " << M->status);
if (M->status < 0) {
- diskd_stats.close.fail++;
+ ++diskd_stats.close.fail;
errorOccured = true;
} else {
- diskd_stats.close.success++;
+ ++diskd_stats.close.success;
}
ioCompleted();
void
DiskdFile::readDone(diomsg * M)
{
- statCounter.syscalls.disk.reads++;
+ ++statCounter.syscalls.disk.reads;
debugs(79, 3, "DiskdFile::readDone: status " << M->status);
assert (M->requestor);
ReadRequest::Pointer readRequest = dynamic_cast<ReadRequest *>(M->requestor);
+
/* remove the free protection */
- readRequest->RefCountDereference();
+ if (readRequest != NULL) {
+ const uint32_t lcount = readRequest->unlock();
+ if (lcount == 0)
+ debugs(79, DBG_IMPORTANT, "invariant check failed: readRequest reference count is 0");
+ }
if (M->status < 0) {
- diskd_stats.read.fail++;
+ ++diskd_stats.read.fail;
ioCompleted();
errorOccured = true;
ioRequestor->readCompleted(NULL, -1, DISK_ERROR, readRequest);
return;
}
- diskd_stats.read.success++;
+ ++diskd_stats.read.success;
ioCompleted();
ioRequestor->readCompleted (IO->shm.buf + M->shm_offset, M->status, DISK_OK, readRequest);
void
DiskdFile::writeDone(diomsg *M)
{
- statCounter.syscalls.disk.writes++;
+ ++statCounter.syscalls.disk.writes;
debugs(79, 3, "storeDiskdWriteDone: status " << M->status);
assert (M->requestor);
WriteRequest::Pointer writeRequest = dynamic_cast<WriteRequest *>(M->requestor);
+
/* remove the free protection */
- writeRequest->RefCountDereference();
+ if (writeRequest != NULL) {
+ const uint32_t lcount = writeRequest->unlock();
+ if (lcount == 0)
+ debugs(79, DBG_IMPORTANT, "invariant check failed: writeRequest reference count is 0");
+ }
if (M->status < 0) {
errorOccured = true;
- diskd_stats.write.fail++;
+ ++diskd_stats.write.fail;
ioCompleted();
ioRequestor->writeCompleted (DISK_ERROR,0, writeRequest);
return;
}
- diskd_stats.write.success++;
+ ++diskd_stats.write.success;
ioCompleted();
ioRequestor->writeCompleted (DISK_OK,M->status, writeRequest);
}
{
return inProgressIOs != 0;
}
+