Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 3 Nov 2008 13:40:08 -0500
From:      Jung-uk Kim <jkim@FreeBSD.org>
To:        src-committers@FreeBSD.org
Cc:        svn-src-head@FreeBSD.org, Alexander Motin <mav@FreeBSD.org>, svn-src-all@FreeBSD.org
Subject:   Re: svn commit: r184596 - head/sys/dev/acpica/Osd
Message-ID:  <200811031340.12072.jkim@FreeBSD.org>
In-Reply-To: <200811031828.mA3ISC84073057@svn.freebsd.org>
References:  <200811031828.mA3ISC84073057@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Monday 03 November 2008 01:28 pm, Alexander Motin wrote:
> Author: mav
> Date: Mon Nov  3 18:28:12 2008
> New Revision: 184596
> URL: http://svn.freebsd.org/changeset/base/184596
>
> Log:
>   Remove " + 1".
>   Thread ID can't be zero anyway while increment may give owerflow.
>
> Modified:
>   head/sys/dev/acpica/Osd/OsdSchedule.c
>
> Modified: head/sys/dev/acpica/Osd/OsdSchedule.c
> ===================================================================
>=========== --- head/sys/dev/acpica/Osd/OsdSchedule.c	Mon Nov  3
> 16:36:23 2008	(r184595) +++
> head/sys/dev/acpica/Osd/OsdSchedule.c	Mon Nov  3 18:28:12
> 2008	(r184596) @@ -191,5 +191,5 @@ AcpiOsGetThreadId(void)
>      /* XXX do not add ACPI_FUNCTION_TRACE here, results in
> recursive call. */
>
>      /* Returning 0 is not allowed. */
> -    return (curthread->td_tid + 1);
> +    return (curthread->td_tid);
>  }

Thanks!

Jung-uk Kim



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200811031340.12072.jkim>