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
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
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
>
> 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
> 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
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
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
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(
> 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
> 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
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.
> > 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
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
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
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
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
"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
>
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
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
19 matches
Mail list logo