linux-kernel - Re: divide error trying to set rt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080626133924.GA20188@elte.hu>
Date:	Thu, 26 Jun 2008 15:39:24 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Dario Faggioli <raistlin@...ux.it>
Cc:	Michael Trimarchi <trimarchimichael@...oo.it>,
	PeterZijlstra <peterz@...radead.org>, "Daniel K." <dk@...no>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: divide error trying to set rt_period to zero


* Dario Faggioli <raistlin@...ux.it> wrote:

> Hi all again,
> 
> Here it is another little Oops we found:
> 
> echo 0 > /dev/cgroups/0/cpu.rt_period_us
> or
> echo 4294967296 > /dev/cgroups/0/cpu.rt_period_us
> 
> [  205.509825] divide error: 0000 [#1] 
> [  205.510151] Modules linked in:
> [  205.510151] 
> [  205.510151] Pid: 2339, comm: bash Not tainted (2.6.26-rc8 #33)
> [  205.510151] EIP: 0060:[<c030c6ef>] EFLAGS: 00000293 CPU: 0
> [  205.510151] EIP is at div64_u64+0x5f/0x70
> [  205.510151] EAX: 0000389f EBX: 00000000 ECX: 00000000 EDX: 00000000
> [  205.510151] ESI: d9800000 EDI: 00000000 EBP: c6cede60 ESP: c6cede50
> [  205.510151]  DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068
> [  205.510151] Process bash (pid: 2339, ti=c6cec000 task=c79be370 task.ti=c6cec000)
> [  205.510151] Stack: d9800000 0000389f c05971a0 d9800000 c6cedeb4 c0214dbd 00000000 00000000 
> [  205.510151]        c6cede88 c0242bd8 c05377c0 c7a41b40 00000000 00000000 00000000 c05971a0 
> [  205.510151]        c780ed20 c7508494 c7a41b40 00000000 00000002 c6cedebc c05971a0 ffffffea 
> [  205.510151] Call Trace:
> [  205.510151]  [<c0214dbd>] ? __rt_schedulable+0x1cd/0x240
> [  205.510151]  [<c0242bd8>] ? cgroup_file_open+0x18/0xe0
> [  205.510151]  [<c0214fe4>] ? tg_set_bandwidth+0xa4/0xf0
> [  205.510151]  [<c0215066>] ? sched_group_set_rt_period+0x36/0x50
> [  205.510151]  [<c021508e>] ? cpu_rt_period_write_uint+0xe/0x10
> [  205.510151]  [<c0242dc5>] ? cgroup_file_write+0x125/0x160
> [  205.510151]  [<c0232c15>] ? hrtimer_interrupt+0x155/0x190
> [  205.510151]  [<c02f047f>] ? security_file_permission+0xf/0x20
> [  205.510151]  [<c0277ad8>] ? rw_verify_area+0x48/0xc0
> [  205.510151]  [<c0283744>] ? dupfd+0x104/0x130
> [  205.510151]  [<c027838c>] ? vfs_write+0x9c/0x160
> [  205.510151]  [<c0242ca0>] ? cgroup_file_write+0x0/0x160
> [  205.510151]  [<c027850d>] ? sys_write+0x3d/0x70
> [  205.510151]  [<c0203019>] ? sysenter_past_esp+0x6a/0x91
> [  205.510151]  =======================
> [  205.510151] Code: 0f 45 de 31 f6 0f ad d0 d3 ea f6 c1 20 0f 45 c2 0f 45 d6 89 45 f0 89 55 f4 8b 55 f4 31 c9 8b 45 f0 39 d3 89 c6 77 08 89 d0 31 d2 <f7> f3 89 c1 83 c4 08 89 f0 f7 f3 89 ca 5b 5e 5d c3 55 89 e5 56 
> [  205.510151] EIP: [<c030c6ef>] div64_u64+0x5f/0x70 SS:ESP 0068:c6cede50
> 
> The attached patch solves the issue for me.

> @@ -8501,6 +8501,9 @@ int sched_group_set_rt_period(struct task_group
> *tg, long rt_period_us)
>  	rt_period = (u64)rt_period_us * NSEC_PER_USEC;
>  	rt_runtime = tg->rt_bandwidth.rt_runtime;
>  
> +	if (period == 0)
> +		return -EINVAL;

this patch doesnt even build, because there's no "period" variable.

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