Re: [PATCH 07/22] adfs: drop vmtruncate

[Christoph Hellwig]

> @@ -57,8 +57,9 @@ static int adfs_write_begin(struct file *file, struct 
> address_space *mapping,
>   _I(mapping->host)->mmu_private);
>   if (unlikely(ret)) {
>   loff_t isize = mapping->host->i_size;
> - if (pos + len > isize)
> - vmtruncate(mapping->host, isize);
> + if ((pos + len > isize) &&
> + inode_newsize_ok(mapping->host, isize) == 0)
> + truncate_setsize(mapping->host, isize);

No need for the inode_newsize_ok check here.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 07/22] adfs: drop vmtruncate

[Christoph Hellwig]

 @@ -57,8 +57,9 @@ static int adfs_write_begin(struct file *file, struct 
 address_space *mapping,
   ADFS_I(mapping-host)-mmu_private);
   if (unlikely(ret)) {
   loff_t isize = mapping-host-i_size;
 - if (pos + len  isize)
 - vmtruncate(mapping-host, isize);
 + if ((pos + len  isize) 
 + inode_newsize_ok(mapping-host, isize) == 0)
 + truncate_setsize(mapping-host, isize);

No need for the inode_newsize_ok check here.

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 07/22] adfs: drop vmtruncate

[Marco Stornelli]

Removed vmtruncate.

Signed-off-by: Marco Stornelli 
---
 fs/adfs/inode.c |5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/fs/adfs/inode.c b/fs/adfs/inode.c
index e9bad50..81de375 100644
--- a/fs/adfs/inode.c
+++ b/fs/adfs/inode.c
@@ -57,8 +57,9 @@ static int adfs_write_begin(struct file *file, struct 
address_space *mapping,
_I(mapping->host)->mmu_private);
if (unlikely(ret)) {
loff_t isize = mapping->host->i_size;
-   if (pos + len > isize)
-   vmtruncate(mapping->host, isize);
+   if ((pos + len > isize) &&
+   inode_newsize_ok(mapping->host, isize) == 0)
+   truncate_setsize(mapping->host, isize);
}
 
return ret;
-- 
1.7.3.4
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 07/22] adfs: drop vmtruncate

[Marco Stornelli]

Removed vmtruncate.

Signed-off-by: Marco Stornelli marco.storne...@gmail.com
---
 fs/adfs/inode.c |5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/fs/adfs/inode.c b/fs/adfs/inode.c
index e9bad50..81de375 100644
--- a/fs/adfs/inode.c
+++ b/fs/adfs/inode.c
@@ -57,8 +57,9 @@ static int adfs_write_begin(struct file *file, struct 
address_space *mapping,
ADFS_I(mapping-host)-mmu_private);
if (unlikely(ret)) {
loff_t isize = mapping-host-i_size;
-   if (pos + len  isize)
-   vmtruncate(mapping-host, isize);
+   if ((pos + len  isize) 
+   inode_newsize_ok(mapping-host, isize) == 0)
+   truncate_setsize(mapping-host, isize);
}
 
return ret;
-- 
1.7.3.4
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/