delorie.com/archives/browse.cgi   search  
Mail Archives: cygwin/2004/06/23/12:21:10

Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Date: Wed, 23 Jun 2004 18:21:00 +0200
From: Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To: cygwin AT cygwin DOT com
Subject: Re: Bug fix to /proc/cpuinfo implementation
Message-ID: <20040623162100.GE19325@cygbert.vinschen.de>
Reply-To: cygwin AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
References: <1087996722 DOT 18400 DOT 69 DOT camel AT whiteice DOT nop DOT iki DOT fi>
Mime-Version: 1.0
In-Reply-To: <1087996722.18400.69.camel@whiteice.nop.iki.fi>
User-Agent: Mutt/1.4.2i

Tomas,

On Jun 23 16:18, Tomas Ukkonen wrote:
> Hi
> 
> I hope I email to correct mailing list...

not really.  It's cygwin-patches AT cygwin DOT com usually.

> It seems that '/proc/cpuinfo' doesn't report 3dnow and 3dnowext support
> correctly.
> 
> Because I had been using it for recognizing processor features I looked
> briefly into fhandler_proc.cc (taken freshly from cvs) and I think I
> fixed the problem. The changed file compiles and *should* correctly
> detect presence of 3dnow instruction support (by using AMD's 0x8000..1
> extended function). I haven't have time to test it (= figure out how
> build+install process actually works) because I decided to write my own
> cpuid detection routines from scratch instead.

Thanks for the patch.  I've applied it.  However...

> Patch is under whatever license needed to make it possible to use it.

...it's a border case in terms of licensing.  In theory it's a bit too
big to fit into the "trivial patch" rule but since its functionality
is pretty simple, I turned a blind eye to the size.

For the future, if you want to submit more patches, please have a start
by reading http://cygwin.com/contrib.html.  It contains all information
you need as a contributor.  Signing the assignment form is (unfortunately)
most important.


Thanks again for the patch,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Co-Project Leader          mailto:cygwin AT cygwin DOT com
Red Hat, Inc.

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

- Raw text -


  webmaster     delorie software   privacy  
  Copyright © 2019   by DJ Delorie     Updated Jul 2019