Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Valeriy E. Ushakov
On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: > struct device * -> device_t, no functional changes intended. Why don't you cmp(1) the objects before and after to verify that? "Same object code generated" is, unlike intentions, something that can be verified. SY, Uwe -- u...@std

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Robert Swindells
Valeriy E. Ushakov wrote: >On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: >> struct device * -> device_t, no functional changes intended. >Why don't you cmp(1) the objects before and after to verify that? >"Same object code generated" is, unlike intentions, something that can >be

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Quentin Garnier
On Tue, May 12, 2009 at 04:24:07PM +0100, Robert Swindells wrote: > > Valeriy E. Ushakov wrote: > >On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: > > >> struct device * -> device_t, no functional changes intended. > > >Why don't you cmp(1) the objects before and after to verify t

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Christoph Egger
> > Valeriy E. Ushakov wrote: > >On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: > > >> struct device * -> device_t, no functional changes intended. > > >Why don't you cmp(1) the objects before and after to verify that? > >"Same object code generated" is, unlike intentions, somet

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Izumi Tsutsui
> Do a search in sys/dev/ic for 'adapt_dev', any driver that casts this > to a softc instead of calling device_private() will crash. struct device * and device_t are identical, so they won't crash unless softc will actuall be split from device_t. But I don't see benefits by mechanical changes aga

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Robert Swindells
Christoph Egger wrote: >> Valeriy E. Ushakov wrote: >> >On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: >> >> >> struct device * -> device_t, no functional changes intended. >> >> >Why don't you cmp(1) the objects before and after to verify that? >> >"Same object code generated" i

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Christoph Egger
Robert Swindells wrote: > Christoph Egger wrote: >>> Valeriy E. Ushakov wrote: On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: > struct device * -> device_t, no functional changes intended. Why don't you cmp(1) the objects before and after to verify that? "Same obj

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Quentin Garnier
On Tue, May 12, 2009 at 05:15:05PM +0100, Robert Swindells wrote: > > Christoph Egger wrote: > >> Valeriy E. Ushakov wrote: > >> >On Tue, May 12, 2009 at 14:18:16 +, Christoph Egger wrote: > >> > >> >> struct device * -> device_t, no functional changes intended. > >> > >> >Why don't you cmp(

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Izumi Tsutsui
> Maybe ahc(4) was the only one that was broken. The point is that you > could have found this by comparing the object files. Although ahc(4) has many quirks (to share sources among *BSD/Linux), only aic7xxx_osm.c belongs to ahc(4) and it doesn't have any bad cast, per output of grep adapt_dev sys

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Izumi Tsutsui
> Now, of course, that doesn't come as a surprise. Some people just never > learn or listen. Or apologise. Honestly, I'd like to see proper fixes for output of "grep -r 'memcpy( ' sys/arch" , before another set of mechanical changes. --- Izumi Tsutsui

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Christoph Egger
Izumi Tsutsui wrote: >> Now, of course, that doesn't come as a surprise. Some people just never >> learn or listen. Or apologise. > > Honestly, I'd like to see proper fixes for output of > "grep -r 'memcpy( ' sys/arch" , > before another set of mechanical changes. I broke it, so I will fix it.

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Izumi Tsutsui
> > Honestly, I'd like to see proper fixes for output of > > "grep -r 'memcpy( ' sys/arch" , > > before another set of mechanical changes. > > I broke it, so I will fix it. > Do you want to review the patch ? Well, it isn't functional changes at all. Can you check it yourself before calling for r

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Izumi Tsutsui
I wrote: > > Maybe ahc(4) was the only one that was broken. The point is that you > > could have found this by comparing the object files. > > Although ahc(4) has many quirks (to share sources among *BSD/Linux), > only aic7xxx_osm.c belongs to ahc(4) and it doesn't have any bad cast, > per output

Re: CVS commit: src/sys/netinet

2009-05-12 Thread Quentin Garnier
On Tue, May 12, 2009 at 09:48:42PM +, Elad Efrat wrote: > Module Name: src > Committed By: elad > Date: Tue May 12 21:48:42 UTC 2009 > > Modified Files: > src/sys/netinet: ip_carp.c > > Log Message: > Fix inverted permissions check. > - if ((l == NULL) || (error = k

Re: CVS commit: src/sys/netinet

2009-05-12 Thread Elad Efrat
Quentin Garnier wrote: On Tue, May 12, 2009 at 09:48:42PM +, Elad Efrat wrote: Module Name:src Committed By: elad Date: Tue May 12 21:48:42 UTC 2009 Modified Files: src/sys/netinet: ip_carp.c Log Message: Fix inverted permissions check. - if ((l == NUL

re: CVS commit: src/sys/netinet

2009-05-12 Thread matthew green
Quentin Garnier wrote: > On Tue, May 12, 2009 at 09:48:42PM +, Elad Efrat wrote: >> Module Name: src >> Committed By: elad >> Date: Tue May 12 21:48:42 UTC 2009 >> >> Modified Files: >> src/sys/netinet: ip_carp.c >> >> Log Message: >> Fix

Re: CVS commit: src/crypto/external/bsd/netpgp/dist

2009-05-12 Thread Perry E. Metzger
"Alistair G. Crooks" writes: > + allow a choice of hash algorithms for the signature digest (rather > than hardcoding SHA1 - it is looking as though collisions are easier > to manufacture based on recent findings) > + move default signature RSA hash algorithm to SHA256 (from SHA1). This is >

Re: CVS commit: src/sys/dev/dec

2009-05-12 Thread Perry E. Metzger
Christoph Egger writes: > Izumi Tsutsui wrote: >>> Now, of course, that doesn't come as a surprise. Some people just never >>> learn or listen. Or apologise. >> >> Honestly, I'd like to see proper fixes for output of >> "grep -r 'memcpy( ' sys/arch" , >> before another set of mechanical change

Re: CVS commit: src/sys/dev/scsipi

2009-05-12 Thread Christoph Egger
Christos Zoulas wrote: > Module Name: src > Committed By: christos > Date: Wed May 13 02:35:25 UTC 2009 > > Modified Files: > src/sys/dev/scsipi: scsipiconf.h > > Log Message: > sprinkle #ifdef _KERNEL to make scsictl compile. > Ah, the device_t change exposed something that shou