blockdev-util: add new helper blockdev_get_device_size()
This function is just a wrapper around the BLKGETSIZE64. Which is a pretty simple ioctl. The only reason to wrap it, is that the headers we need to call it are a bit messy (as "linux/fs.h" is incompatible with certain glibc headers). Hence add the simple helper that wraps it and allows us to do the header mess needed in one file only. It's also nicely symmetric to blockdev_get_sector_size().
This commit is contained in:
@@ -777,6 +777,21 @@ int blockdev_get_sector_size(int fd, uint32_t *ret) {
|
||||
return 0;
|
||||
}
|
||||
|
||||
int blockdev_get_device_size(int fd, uint64_t *ret) {
|
||||
uint64_t sz = 0;
|
||||
|
||||
assert(fd >= 0);
|
||||
assert(ret);
|
||||
|
||||
/* This is just a type-safe wrapper around BLKGETSIZE64 that gets us around having to include messy linux/fs.h in various clients */
|
||||
|
||||
if (ioctl(fd, BLKGETSIZE64, &sz) < 0)
|
||||
return -errno;
|
||||
|
||||
*ret = sz;
|
||||
return 0;
|
||||
}
|
||||
|
||||
int blockdev_get_root(int level, dev_t *ret) {
|
||||
_cleanup_free_ char *p = NULL;
|
||||
dev_t devno;
|
||||
|
||||
@@ -57,5 +57,6 @@ int block_device_has_partitions(sd_device *dev);
|
||||
int blockdev_reread_partition_table(sd_device *dev);
|
||||
|
||||
int blockdev_get_sector_size(int fd, uint32_t *ret);
|
||||
int blockdev_get_device_size(int fd, uint64_t *ret);
|
||||
|
||||
int blockdev_get_root(int level, dev_t *ret);
|
||||
|
||||
@@ -13,6 +13,7 @@
|
||||
#include <unistd.h>
|
||||
|
||||
#include "alloc-util.h"
|
||||
#include "blockdev-util.h"
|
||||
#include "btrfs-util.h"
|
||||
#include "chase.h"
|
||||
#include "chattr-util.h"
|
||||
@@ -446,8 +447,9 @@ static int image_make(
|
||||
read_only = true;
|
||||
}
|
||||
|
||||
if (ioctl(block_fd, BLKGETSIZE64, &size) < 0)
|
||||
log_debug_errno(errno, "Failed to issue BLKGETSIZE64 on device %s/%s, ignoring: %m", path ?: strnull(parent), filename);
|
||||
r = blockdev_get_device_size(block_fd, &size);
|
||||
if (r < 0)
|
||||
log_debug_errno(r, "Failed to issue BLKGETSIZE64 on device %s/%s, ignoring: %m", path ?: strnull(parent), filename);
|
||||
|
||||
block_fd = safe_close(block_fd);
|
||||
}
|
||||
|
||||
@@ -1759,8 +1759,9 @@ static int fs_grow(const char *node_path, int mount_fd, const char *mount_path)
|
||||
if (node_fd < 0)
|
||||
return log_debug_errno(errno, "Failed to open node device %s: %m", node_path);
|
||||
|
||||
if (ioctl(node_fd, BLKGETSIZE64, &size) != 0)
|
||||
return log_debug_errno(errno, "Failed to get block device size of %s: %m", node_path);
|
||||
r = blockdev_get_device_size(node_fd, &size);
|
||||
if (r < 0)
|
||||
return log_debug_errno(r, "Failed to get block device size of %s: %m", node_path);
|
||||
|
||||
if (mount_fd < 0) {
|
||||
assert(mount_path);
|
||||
|
||||
Reference in New Issue
Block a user