On Thu, 11 May 2006 00:22:03 +0100
Al Viro <[EMAIL PROTECTED]> wrote:

> On Wed, May 10, 2006 at 04:14:05PM -0700, Stephen Hemminger wrote:
> > Fix warning from sparse in bonding code about "incorrect type in assignment"
> 
> *snerk*
> 
> Only if you are building without -Wcast-to-as.  It _is_ incorrect type in
> assignment.  And the real fix is to expand the call, killing set_fs()
> in there.

More like this (in br_if.c)?

        struct ethtool_cmd ecmd = { ETHTOOL_GSET };
        struct ifreq ifr;
        mm_segment_t old_fs;
        int err;

        strncpy(ifr.ifr_name, dev->name, IFNAMSIZ);
        ifr.ifr_data = (void __user *) &ecmd;

        old_fs = get_fs();
        set_fs(KERNEL_DS);
        err = dev_ethtool(&ifr);
        set_fs(old_fs);
        
        if (!err)
                ...
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to