Re: CVS commit: src/sys/arch/x86/x86

[Jean-Yves Migeon]

Le 09/10/2015 06:49, Masanobu SAITOH a écrit :
> On 2015/10/06 6:10, Jean-Yves Migeon wrote:
>>> Log Message:
>>> kmem_free() the address returned by kmem_alloc().  found by Brainy.
>>> use the newly aligned location if we needed it.  found by kre.
>>>
>>>
>>> To generate a diff of this commit:
>>> cvs rdiff -u -r1.8 -r1.9 src/sys/arch/x86/x86/cpu_ucode_intel.c
>>
>> IMHO this should be pulled-up to -6 and -7.
>>
>> Any argument against? If the old code worked, it's pure luck.
> 
>  netbsd-6 doesn't support the microcode update function for Intel
> CPU. That bug should be pulled up to netbsd-7 (and netbsd-7-1) branch.

Makes sense. I'll check and ask for this pullup.

-- 
Jean-Yves Migeon

Re: CVS commit: src/sys/arch/arm/allwinner

[Manuel Bouyer]

On Thu, Oct 08, 2015 at 07:19:15PM -0300, Jared McNeill wrote:
> Unless you plan on implementing a better fix, I'm going to commit something
> like this (not compile tested) to restore HDMI audio functionality for A20
> and A31.

By experiments, I found that it's better to turn off the signals in
HDMI than in debe (the monitor goes to sleep immediately).
But I've not implemented anything for the audio issue yet (I got distracted
by the stability issues on my lime2, which I ended up fixing by implementing
voltage changes in axp20x, and cpu speed change).

So I think the best things to do it to revert my previous commit until
I get with something better in the hdmi driver.

-- 
Manuel Bouyer 
 NetBSD: 26 ans d'experience feront toujours la difference
--

Re: CVS commit: src/usr.bin/tail

[Joerg Sonnenberger]

On Fri, Oct 09, 2015 at 01:51:27PM -0400, Christos Zoulas wrote:
> Module Name:  src
> Committed By: christos
> Date: Fri Oct  9 17:51:27 UTC 2015
> 
> Modified Files:
>   src/usr.bin/tail: forward.c
> 
> Log Message:
> PR/50322: Timo Buhrmester: tail -F  misbehaves with stdin closed
> Compare fp with stdin not fileno(fp) with STDIN_FILENO, because if tail
> is called with 0 closed, then we are not going to be setting event filters
> for the file because we'll erroneously think it is stdin.

Well, you are not supposed to do that. We are just only enforcing it for
suid/sgid binaries.

Joerg