From owner-freebsd-bugs@FreeBSD.ORG Wed Mar 8 20:00:27 2006 Return-Path: X-Original-To: freebsd-bugs@hub.freebsd.org Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C139F16A420 for ; Wed, 8 Mar 2006 20:00:27 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8480443D5F for ; Wed, 8 Mar 2006 20:00:25 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.4/8.13.4) with ESMTP id k28K0PoP057133 for ; Wed, 8 Mar 2006 20:00:25 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.4/8.13.4/Submit) id k28K0PHt057132; Wed, 8 Mar 2006 20:00:25 GMT (envelope-from gnats) Date: Wed, 8 Mar 2006 20:00:25 GMT Message-Id: <200603082000.k28K0PHt057132@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: "Poul-Henning Kamp" Cc: Subject: Re: kern/94238: 32 bit overflow prevents proper setting of base in tc_cpu_ticks() X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Poul-Henning Kamp List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Mar 2006 20:00:27 -0000 The following reply was made to PR kern/94238; it has been noted by GNATS. From: "Poul-Henning Kamp" To: Tim Kuik Cc: freebsd-gnats-submit@FreeBSD.org Subject: Re: kern/94238: 32 bit overflow prevents proper setting of base in tc_cpu_ticks() Date: Wed, 08 Mar 2006 20:56:30 +0100 Already done. In message <200603081944.k28JiQGr006673@www.freebsd.org>, Tim Kuik writes: > >>Number: 94238 >>Category: kern >>Synopsis: 32 bit overflow prevents proper setting of base in tc_cpu_ticks() >>Confidential: no >>Severity: serious >>Priority: low >>Responsible: freebsd-bugs >>State: open >>Quarter: >>Keywords: >>Date-Required: >>Class: sw-bug >>Submitter-Id: current-users >>Arrival-Date: Wed Mar 08 19:50:09 GMT 2006 >>Closed-Date: >>Last-Modified: >>Originator: Tim Kuik >>Release: current >>Organization: >Cisco >>Environment: >FreeBSD tkuik-freebsd.cisco 7.0-CURRENT FreeBSD 7.0-CURRENT #0: Mon Mar 6 12:49:57 CST 2006 root@tkuik-freebsd.cisco:/usr/obj/usr/src/sys/GENERIC i386 > >>Description: >in kern_tc.c: tc_cpu_ticks(), the line >base += tc->tc_counter_mask + 1; >will cause a 32 bit overflow if the tc_counter_mask is 0xffffffff, resulting in base not being incremented. >>How-To-Repeat: >As long as the mask is set to 0xffffffff, it will always happen. >>Fix: >Change the line to: >base += (uint64_t)(tc->tc_counter_mask) + 1; >>Release-Note: >>Audit-Trail: >>Unformatted: >_______________________________________________ >freebsd-bugs@freebsd.org mailing list >http://lists.freebsd.org/mailman/listinfo/freebsd-bugs >To unsubscribe, send any mail to "freebsd-bugs-unsubscribe@freebsd.org" > -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.