mirror of
git://sourceware.org/git/lvm2.git
synced 2025-01-02 01:18:26 +03:00
Remove O_DIRECT as it causes problems with some systems.
Harumph.
This commit is contained in:
parent
f55f988643
commit
da253e3aac
@ -1,5 +1,5 @@
|
||||
/*
|
||||
* Copyright (C) 2002 Sistina Software
|
||||
* Copyright (C) 2001 Sistina Software
|
||||
*
|
||||
* This file is released under the LGPL.
|
||||
*/
|
||||
@ -10,22 +10,10 @@
|
||||
|
||||
#include <sys/types.h>
|
||||
#include <sys/stat.h>
|
||||
#include <unistd.h>
|
||||
#include <stdlib.h>
|
||||
#include <fcntl.h>
|
||||
#include <unistd.h>
|
||||
#include <sys/ioctl.h>
|
||||
#include <linux/fs.h> /* UGH!!! for BLKSSZGET */
|
||||
|
||||
|
||||
/* Buffer for O_DIRECT, allocated as
|
||||
twice _SC_PAGE_SIZE so we can make sure it is
|
||||
properly aligned.
|
||||
It is assumed that _SC_PAGE_SIZE is a power of 2
|
||||
but I think that's fairly safe :)
|
||||
*/
|
||||
static char *bigbuf;
|
||||
static char *aligned_buf;
|
||||
static long page_size;
|
||||
#include <linux/fs.h> // UGH!!! for BLKSSZGET
|
||||
|
||||
int dev_get_size(struct device *dev, uint64_t * size)
|
||||
{
|
||||
@ -53,25 +41,31 @@ int dev_get_size(struct device *dev, uint64_t * size)
|
||||
|
||||
int dev_get_sectsize(struct device *dev, uint32_t * size)
|
||||
{
|
||||
int fd;
|
||||
int s;
|
||||
const char *name = dev_name(dev);
|
||||
int fd;
|
||||
int s;
|
||||
const char *name = dev_name(dev);
|
||||
|
||||
log_very_verbose("Getting size of %s", name);
|
||||
if ((fd = open(name, O_RDONLY)) < 0) {
|
||||
log_sys_error("open", name);
|
||||
return 0;
|
||||
}
|
||||
log_very_verbose("Getting size of %s", name);
|
||||
if ((fd = open(name, O_RDONLY)) < 0) {
|
||||
log_sys_error("open", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
if (ioctl(fd, BLKSSZGET, &s) < 0) {
|
||||
log_sys_error("ioctl BLKSSZGET", name);
|
||||
close(fd);
|
||||
return 0;
|
||||
}
|
||||
if (ioctl(fd, BLKSSZGET, &s) < 0) {
|
||||
log_sys_error("ioctl BLKSSZGET", name);
|
||||
close(fd);
|
||||
return 0;
|
||||
}
|
||||
|
||||
close(fd);
|
||||
*size = (uint32_t) s;
|
||||
return 1;
|
||||
close(fd);
|
||||
*size = (uint32_t) s;
|
||||
return 1;
|
||||
}
|
||||
|
||||
|
||||
static void _flush(int fd)
|
||||
{
|
||||
ioctl(fd, BLKFLSBUF, 0);
|
||||
}
|
||||
|
||||
int dev_open(struct device *dev, int flags)
|
||||
@ -79,17 +73,6 @@ int dev_open(struct device *dev, int flags)
|
||||
struct stat buf;
|
||||
const char *name = dev_name_confirmed(dev);
|
||||
|
||||
/* First time through - allocate & align the buffer */
|
||||
if (!page_size) {
|
||||
page_size = sysconf(_SC_PAGESIZE);
|
||||
bigbuf = malloc(page_size*2);
|
||||
if (!bigbuf) {
|
||||
stack;
|
||||
return 0;
|
||||
}
|
||||
aligned_buf = (char *)(((long)bigbuf + page_size) & ~(page_size-1) );
|
||||
}
|
||||
|
||||
if (!name) {
|
||||
stack;
|
||||
return 0;
|
||||
@ -105,12 +88,7 @@ int dev_open(struct device *dev, int flags)
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* If write was requested then add read so we can pre-fill
|
||||
buffers for aligned writes */
|
||||
if (flags & O_WRONLY)
|
||||
flags = (flags & ~O_WRONLY) | O_RDWR;
|
||||
|
||||
if ((dev->fd = open(name, flags|O_DIRECT)) < 0) {
|
||||
if ((dev->fd = open(name, flags)) < 0) {
|
||||
log_sys_error("open", name);
|
||||
return 0;
|
||||
}
|
||||
@ -120,7 +98,7 @@ int dev_open(struct device *dev, int flags)
|
||||
dev_close(dev);
|
||||
return 0;
|
||||
}
|
||||
|
||||
_flush(dev->fd);
|
||||
dev->flags = 0;
|
||||
|
||||
return 1;
|
||||
@ -134,6 +112,9 @@ int dev_close(struct device *dev)
|
||||
return 0;
|
||||
}
|
||||
|
||||
if (dev->flags & DEV_ACCESSED_W)
|
||||
_flush(dev->fd);
|
||||
|
||||
if (close(dev->fd))
|
||||
log_sys_error("close", dev_name(dev));
|
||||
|
||||
@ -170,37 +151,18 @@ int64_t dev_read(struct device * dev, uint64_t offset,
|
||||
{
|
||||
const char *name = dev_name(dev);
|
||||
int fd = dev->fd;
|
||||
int64_t newlen;
|
||||
int64_t newoffset;
|
||||
int64_t offsetdiff;
|
||||
int ret;
|
||||
|
||||
if (fd < 0) {
|
||||
log_err("Attempt to read an unopened device (%s).", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* Adjust offset to page size */
|
||||
newoffset = offset & ~(page_size-1);
|
||||
offsetdiff = offset - newoffset;
|
||||
|
||||
if (lseek(fd, newoffset, SEEK_SET) < 0) {
|
||||
if (lseek(fd, offset, SEEK_SET) < 0) {
|
||||
log_sys_error("lseek", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* Copy to aligned buffer & pad to page size */
|
||||
newlen = (len+offsetdiff) + page_size - ((len+offsetdiff) % page_size);
|
||||
|
||||
ret = _read(fd, aligned_buf, newlen);
|
||||
if (ret < len+offsetdiff)
|
||||
{
|
||||
return ret;
|
||||
}
|
||||
|
||||
/* Copy back to users buffer */
|
||||
memcpy(buffer, aligned_buf+offsetdiff, len);
|
||||
return len;
|
||||
return _read(fd, buffer, len);
|
||||
}
|
||||
|
||||
int _write(int fd, const void *buf, size_t count)
|
||||
@ -232,53 +194,26 @@ int64_t dev_write(struct device * dev, uint64_t offset,
|
||||
{
|
||||
const char *name = dev_name(dev);
|
||||
int fd = dev->fd;
|
||||
int64_t newlen;
|
||||
int64_t newoffset;
|
||||
int64_t offsetdiff;
|
||||
int ret;
|
||||
|
||||
if (fd < 0) {
|
||||
log_error("Attempt to write to unopened device %s", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
/* Adjust offset to page size */
|
||||
newoffset = offset & ~(page_size-1);
|
||||
offsetdiff = offset - newoffset;
|
||||
|
||||
if (lseek(fd, newoffset, SEEK_SET) < 0) {
|
||||
if (lseek(fd, offset, SEEK_SET) < 0) {
|
||||
log_sys_error("lseek", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
dev->flags |= DEV_ACCESSED_W;
|
||||
|
||||
/* Pad to page size */
|
||||
newlen = (len+offsetdiff) + page_size - ((len+offsetdiff) % page_size);
|
||||
|
||||
/* We read the page in and just overwrite the bits
|
||||
requested. Needs to fudge O_RDWR in dev_open for this to work. */
|
||||
if (_read(fd, aligned_buf, newlen) <= 0) {
|
||||
log_sys_error("pre-read", name);
|
||||
return 0;
|
||||
}
|
||||
if (lseek(fd, newoffset, SEEK_SET) < 0) {
|
||||
log_sys_error("re-lseek", name);
|
||||
return 0;
|
||||
}
|
||||
|
||||
memcpy(aligned_buf+offsetdiff, buffer, len);
|
||||
ret = _write(fd, aligned_buf, newlen);
|
||||
|
||||
if (ret == newlen)
|
||||
return len;
|
||||
else
|
||||
return ret;
|
||||
return _write(fd, buffer, len);
|
||||
}
|
||||
|
||||
int dev_zero(struct device *dev, uint64_t offset, int64_t len)
|
||||
{
|
||||
int64_t r, s;
|
||||
char buffer[4096];
|
||||
const char *name = dev_name(dev);
|
||||
int fd = dev->fd;
|
||||
|
||||
@ -293,10 +228,10 @@ int dev_zero(struct device *dev, uint64_t offset, int64_t len)
|
||||
return 0;
|
||||
}
|
||||
|
||||
memset(aligned_buf, 0, page_size);
|
||||
memset(buffer, 0, sizeof(buffer));
|
||||
while (1) {
|
||||
s = len > page_size ? page_size : len;
|
||||
r = _write(fd, aligned_buf, s);
|
||||
s = len > sizeof(buffer) ? sizeof(buffer) : len;
|
||||
r = _write(fd, buffer, s);
|
||||
|
||||
if (r <= 0)
|
||||
break;
|
||||
|
Loading…
Reference in New Issue
Block a user