Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Masayoshi Mizuma
Hi Rik, I applied your patch to linux-next kernel, then divide error happened when I ran ltp stress test. The divide error occurred on the following div_u64(), so the following should be also fixed... static unsigned long bdi_position_ratio(struct backing_dev_info *bdi,

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Maxim Patlasov
On 04/30/2014 12:12 PM, Michal Hocko wrote: On Wed 30-04-14 12:04:04, Maxim Patlasov wrote: Hi Rik! On 04/29/2014 11:19 PM, Rik van Riel wrote: It is possible for "limit - setpoint + 1" to equal zero, leading to a divide by zero error. Blindly adding 1 to "limit - setpoint" is not working, so

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Michal Hocko
On Wed 30-04-14 12:04:04, Maxim Patlasov wrote: > Hi Rik! > > On 04/29/2014 11:19 PM, Rik van Riel wrote: > >It is possible for "limit - setpoint + 1" to equal zero, leading to a > >divide by zero error. Blindly adding 1 to "limit - setpoint" is not > >working, so we need to actually test the

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Maxim Patlasov
Hi Rik! On 04/29/2014 11:19 PM, Rik van Riel wrote: It is possible for "limit - setpoint + 1" to equal zero, leading to a divide by zero error. Blindly adding 1 to "limit - setpoint" is not working, so we need to actually test the divisor before calling div64. The patch looks correct, but I'm

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Maxim Patlasov
Hi Rik! On 04/29/2014 11:19 PM, Rik van Riel wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the divisor before calling div64. The patch looks correct, but I'm

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Michal Hocko
On Wed 30-04-14 12:04:04, Maxim Patlasov wrote: Hi Rik! On 04/29/2014 11:19 PM, Rik van Riel wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the divisor before

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Maxim Patlasov
On 04/30/2014 12:12 PM, Michal Hocko wrote: On Wed 30-04-14 12:04:04, Maxim Patlasov wrote: Hi Rik! On 04/29/2014 11:19 PM, Rik van Riel wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-30 Thread Masayoshi Mizuma
Hi Rik, I applied your patch to linux-next kernel, then divide error happened when I ran ltp stress test. The divide error occurred on the following div_u64(), so the following should be also fixed... static unsigned long bdi_position_ratio(struct backing_dev_info *bdi,

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Andrew Morton
On Tue, 29 Apr 2014 18:48:11 -0400 Rik van Riel wrote: > On 04/29/2014 06:39 PM, Andrew Morton wrote: > > On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel wrote: > > > >> It is possible for "limit - setpoint + 1" to equal zero, leading to a > >> divide by zero error. Blindly adding 1 to "limit

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Rik van Riel
On 04/29/2014 06:39 PM, Andrew Morton wrote: > On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel wrote: > >> It is possible for "limit - setpoint + 1" to equal zero, leading to a >> divide by zero error. Blindly adding 1 to "limit - setpoint" is not >> working, so we need to actually test the

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Andrew Morton
On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel wrote: > It is possible for "limit - setpoint + 1" to equal zero, leading to a > divide by zero error. Blindly adding 1 to "limit - setpoint" is not > working, so we need to actually test the divisor before calling div64. > > ... > > ---

RE: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Motohiro Kosaki
..@suse.cz; fengguang...@intel.com; mpatla...@parallels.com > Subject: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom > > It is possible for "limit - setpoint + 1" to equal zero, leading to a divide > by zero error. Blindly adding 1 to "limit - setpoin

[PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Rik van Riel
It is possible for "limit - setpoint + 1" to equal zero, leading to a divide by zero error. Blindly adding 1 to "limit - setpoint" is not working, so we need to actually test the divisor before calling div64. Signed-off-by: Rik van Riel Cc: sta...@vger.kernel.org --- mm/page-writeback.c | 7

[PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Rik van Riel
It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the divisor before calling div64. Signed-off-by: Rik van Riel r...@redhat.com Cc: sta...@vger.kernel.org ---

RE: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Motohiro Kosaki
...@intel.com; mpatla...@parallels.com Subject: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the divisor before

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Andrew Morton
On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel r...@redhat.com wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the divisor before calling div64. ... ---

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Rik van Riel
On 04/29/2014 06:39 PM, Andrew Morton wrote: On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel r...@redhat.com wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly adding 1 to limit - setpoint is not working, so we need to actually test the

Re: [PATCH] mm,writeback: fix divide by zero in pos_ratio_polynom

2014-04-29 Thread Andrew Morton
On Tue, 29 Apr 2014 18:48:11 -0400 Rik van Riel r...@redhat.com wrote: On 04/29/2014 06:39 PM, Andrew Morton wrote: On Tue, 29 Apr 2014 15:19:10 -0400 Rik van Riel r...@redhat.com wrote: It is possible for limit - setpoint + 1 to equal zero, leading to a divide by zero error. Blindly