From owner-svn-src-all@FreeBSD.ORG Tue Oct 9 17:47:51 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 5B68A3AD for ; Tue, 9 Oct 2012 17:47:51 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: from mail-pa0-f54.google.com (mail-pa0-f54.google.com [209.85.220.54]) by mx1.freebsd.org (Postfix) with ESMTP id C576E8FC1B for ; Tue, 9 Oct 2012 17:47:50 +0000 (UTC) Received: by mail-pa0-f54.google.com with SMTP id bi1so6024486pad.13 for ; Tue, 09 Oct 2012 10:47:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eitanadler.com; s=0xdeadbeef; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type; bh=lIeh5LcE/O/HgiQOyV76aLryNW/EHoDku1QAycSLd9s=; b=YvhhW4Vk8wNkoh/VKOqkwCveAbVeaOoUMKvU6NBUnbDOqtPmkG6NJwP+j598N5jDox MtuIoufwWxwXw7HGoPZEpLT2H0TdpQoBdyRAC3AVrA2B8sPYQBY1pnmTK38XPbIO84xl Edu9rux7tb/WXd1npEWSyhTCa3RcOshig9nTI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :x-gm-message-state; bh=lIeh5LcE/O/HgiQOyV76aLryNW/EHoDku1QAycSLd9s=; b=M9aIrTTrz2AhUqARnhCi7Yb0Tos2ISoAs+ZwjBLP8/vcnb++nJvIKMH63czIr8oZ03 7XmLMpCiJDatVd/vxxjUfCxvyJk7pDkOVUUNdahRJSsC8g3HCG/PR32MBwKx13tFHxb0 azkBvGS7JYbAQVIc5Vc5x8he7FK/hJ/DvqXVLwuFHji+qKrOJy4+mSSFk+tRrfpbt/DB gmbOeUpqfZ8uJGs3VJ/+kGDBOGj/h5dHr3WqoGUmBOFqNIJO0z3DbsdVyhYJlLx1+KGt nI1w9Wy+/RhIz31FW5/zalPUBsLaykVlRXJhpaHlaUcBFbIrKOTmoxoa0I/8YKeCywRB nbmg== Received: by 10.66.77.40 with SMTP id p8mr54449126paw.78.1349804870329; Tue, 09 Oct 2012 10:47:50 -0700 (PDT) MIME-Version: 1.0 Sender: lists@eitanadler.com Received: by 10.66.161.163 with HTTP; Tue, 9 Oct 2012 10:47:20 -0700 (PDT) In-Reply-To: <507451DE.9060909@freebsd.org> References: <201210091425.q99EPFS6020787@svn.freebsd.org> <507451DE.9060909@freebsd.org> From: Eitan Adler Date: Tue, 9 Oct 2012 13:47:20 -0400 X-Google-Sender-Auth: luk8TWlNzV065ae0uL0Luu8jRX8 Message-ID: Subject: Re: svn commit: r241373 - head/lib/libc/stdlib To: Andrey Chernov Content-Type: text/plain; charset=UTF-8 X-Gm-Message-State: ALoCoQkpR3Ik6jkJeAkT81tRqvp5Kfh/MMz+YxCAld3RM7OhxgfbfKkKckfkMtFTg+RC4XBeq9Jx Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Oct 2012 17:47:51 -0000 On 9 October 2012 12:33, Andrey Chernov wrote: > Do you check assembler output for _both_ cases? Yes. > In my testing clang and gcc xor's 'junk' properly in case it have > 'volatile' keyword (as in srandomdev()) and elide it without 'volatile'. volatile is still undefined: see 5.1.2.2.3 and 6.7.2.4 of ISO9899 > IMHO this change should be backed out for srandomdev() and adding > 'volatile' for sranddev() instead. http://blog.eitanadler.com/2012/10/reduced-entropy-in-rand-and-random.html for additional details and actual assembler output. -- Eitan Adler Source & Ports committer X11, Bugbusting teams