Commit dd7f3d54 authored by Alexey Khoroshilov's avatar Alexey Khoroshilov Committed by Christoph Hellwig

hfsplus: Add error propagation for hfsplus_ext_write_extent_locked

Implement error propagation through the callers of
hfsplus_ext_write_extent_locked().
Signed-off-by: default avatarAlexey Khoroshilov <khoroshilov@ispras.ru>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent 5bd9d99d
...@@ -119,23 +119,31 @@ static void __hfsplus_ext_write_extent(struct inode *inode, ...@@ -119,23 +119,31 @@ static void __hfsplus_ext_write_extent(struct inode *inode,
set_bit(HFSPLUS_I_EXT_DIRTY, &hip->flags); set_bit(HFSPLUS_I_EXT_DIRTY, &hip->flags);
} }
static void hfsplus_ext_write_extent_locked(struct inode *inode) static int hfsplus_ext_write_extent_locked(struct inode *inode)
{ {
int res;
if (HFSPLUS_I(inode)->extent_state & HFSPLUS_EXT_DIRTY) { if (HFSPLUS_I(inode)->extent_state & HFSPLUS_EXT_DIRTY) {
struct hfs_find_data fd; struct hfs_find_data fd;
if (!hfs_find_init(HFSPLUS_SB(inode->i_sb)->ext_tree, &fd)) { res = hfs_find_init(HFSPLUS_SB(inode->i_sb)->ext_tree, &fd);
if (res)
return res;
__hfsplus_ext_write_extent(inode, &fd); __hfsplus_ext_write_extent(inode, &fd);
hfs_find_exit(&fd); hfs_find_exit(&fd);
} }
} return 0;
} }
void hfsplus_ext_write_extent(struct inode *inode) int hfsplus_ext_write_extent(struct inode *inode)
{ {
int res;
mutex_lock(&HFSPLUS_I(inode)->extents_lock); mutex_lock(&HFSPLUS_I(inode)->extents_lock);
hfsplus_ext_write_extent_locked(inode); res = hfsplus_ext_write_extent_locked(inode);
mutex_unlock(&HFSPLUS_I(inode)->extents_lock); mutex_unlock(&HFSPLUS_I(inode)->extents_lock);
return res;
} }
static inline int __hfsplus_ext_read_extent(struct hfs_find_data *fd, static inline int __hfsplus_ext_read_extent(struct hfs_find_data *fd,
...@@ -477,7 +485,9 @@ int hfsplus_file_extend(struct inode *inode) ...@@ -477,7 +485,9 @@ int hfsplus_file_extend(struct inode *inode)
insert_extent: insert_extent:
dprint(DBG_EXTENT, "insert new extent\n"); dprint(DBG_EXTENT, "insert new extent\n");
hfsplus_ext_write_extent_locked(inode); res = hfsplus_ext_write_extent_locked(inode);
if (res)
goto out;
memset(hip->cached_extents, 0, sizeof(hfsplus_extent_rec)); memset(hip->cached_extents, 0, sizeof(hfsplus_extent_rec));
hip->cached_extents[0].start_block = cpu_to_be32(start); hip->cached_extents[0].start_block = cpu_to_be32(start);
......
...@@ -374,7 +374,7 @@ extern const struct file_operations hfsplus_dir_operations; ...@@ -374,7 +374,7 @@ extern const struct file_operations hfsplus_dir_operations;
/* extents.c */ /* extents.c */
int hfsplus_ext_cmp_key(const hfsplus_btree_key *, const hfsplus_btree_key *); int hfsplus_ext_cmp_key(const hfsplus_btree_key *, const hfsplus_btree_key *);
void hfsplus_ext_write_extent(struct inode *); int hfsplus_ext_write_extent(struct inode *);
int hfsplus_get_block(struct inode *, sector_t, struct buffer_head *, int); int hfsplus_get_block(struct inode *, sector_t, struct buffer_head *, int);
int hfsplus_free_fork(struct super_block *, u32, int hfsplus_free_fork(struct super_block *, u32,
struct hfsplus_fork_raw *, int); struct hfsplus_fork_raw *, int);
......
...@@ -135,9 +135,13 @@ static int hfsplus_system_write_inode(struct inode *inode) ...@@ -135,9 +135,13 @@ static int hfsplus_system_write_inode(struct inode *inode)
static int hfsplus_write_inode(struct inode *inode, static int hfsplus_write_inode(struct inode *inode,
struct writeback_control *wbc) struct writeback_control *wbc)
{ {
int err;
dprint(DBG_INODE, "hfsplus_write_inode: %lu\n", inode->i_ino); dprint(DBG_INODE, "hfsplus_write_inode: %lu\n", inode->i_ino);
hfsplus_ext_write_extent(inode); err = hfsplus_ext_write_extent(inode);
if (err)
return err;
if (inode->i_ino >= HFSPLUS_FIRSTUSER_CNID || if (inode->i_ino >= HFSPLUS_FIRSTUSER_CNID ||
inode->i_ino == HFSPLUS_ROOT_CNID) inode->i_ino == HFSPLUS_ROOT_CNID)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment