Re: [Devel] [PATCH RH7] ploop: Fix crash in purge_lru_warn()
On 11/10/20 5:47 PM, Kirill Tkhai wrote: > do_div() works wrong in case of the second argument is long. > We don't need remainder, so we don't need do_div() at all. > > https://jira.sw.ru/browse/PSBM-122035 > > Reported-by: Evgenii Shatokhin > Signed-off-by: Kirill Tkhai Reviewed-by: Andrey Ryabinin ___ Devel mailing list Devel@openvz.org https://lists.openvz.org/mailman/listinfo/devel
Re: [Devel] [PATCH RH7] ploop: Fix crash in purge_lru_warn()
On 10.11.2020 17:47, Kirill Tkhai wrote: do_div() works wrong in case of the second argument is long. We don't need remainder, so we don't need do_div() at all. https://jira.sw.ru/browse/PSBM-122035 Reported-by: Evgenii Shatokhin Signed-off-by: Kirill Tkhai --- drivers/block/ploop/io_direct_map.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/block/ploop/io_direct_map.c b/drivers/block/ploop/io_direct_map.c index 5528e86aab43..8f09ab083315 100644 --- a/drivers/block/ploop/io_direct_map.c +++ b/drivers/block/ploop/io_direct_map.c @@ -377,7 +377,7 @@ static inline void purge_lru_warn(struct extent_map_tree *tree) loff_t ratio = i_size_read(tree->mapping->host) * 100; long images_size = atomic_long_read(_io_images_size) ? : 1; - do_div(ratio, images_size); + ratio /= images_size; printk(KERN_WARNING "Purging lru entry from extent tree for inode %ld " "(map_size=%d ratio=%lld%%)\n", . Looks good to me. The simpler the better. Reviewed-by: Evgenii Shatokhin ___ Devel mailing list Devel@openvz.org https://lists.openvz.org/mailman/listinfo/devel
[Devel] [PATCH RH7] ploop: Fix crash in purge_lru_warn()
do_div() works wrong in case of the second argument is long. We don't need remainder, so we don't need do_div() at all. https://jira.sw.ru/browse/PSBM-122035 Reported-by: Evgenii Shatokhin Signed-off-by: Kirill Tkhai --- drivers/block/ploop/io_direct_map.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/block/ploop/io_direct_map.c b/drivers/block/ploop/io_direct_map.c index 5528e86aab43..8f09ab083315 100644 --- a/drivers/block/ploop/io_direct_map.c +++ b/drivers/block/ploop/io_direct_map.c @@ -377,7 +377,7 @@ static inline void purge_lru_warn(struct extent_map_tree *tree) loff_t ratio = i_size_read(tree->mapping->host) * 100; long images_size = atomic_long_read(_io_images_size) ? : 1; - do_div(ratio, images_size); + ratio /= images_size; printk(KERN_WARNING "Purging lru entry from extent tree for inode %ld " "(map_size=%d ratio=%lld%%)\n", ___ Devel mailing list Devel@openvz.org https://lists.openvz.org/mailman/listinfo/devel