1. 24 1月, 2013 2 次提交
  2. 23 1月, 2013 9 次提交
    • C
      netpoll: fix an uninitialized variable · e39363a9
      Cong Wang 提交于
      Fengguang reported:
      
         net/core/netpoll.c: In function 'netpoll_setup':
         net/core/netpoll.c:1049:6: warning: 'err' may be used uninitialized in this function [-Wmaybe-uninitialized]
      
      in !CONFIG_IPV6 case, we may error out without initializing
      'err'.
      Reported-by: NFengguang Wu <fengguang.wu@intel.com>
      Cc: David S. Miller <davem@davemloft.net>
      Signed-off-by: NCong Wang <amwang@redhat.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      e39363a9
    • C
      ipv6: remove duplicated declaration of ip6_fragment() · 9647bb80
      Cong Wang 提交于
      It is declared in:
      include/net/ip6_route.h:187:int ip6_fragment(struct sk_buff *skb, int (*output)(struct sk_buff *));
      
      and net/ip6_route.h is already included.
      
      Cc: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
      Cc: David S. Miller <davem@davemloft.net>
      Signed-off-by: NCong Wang <amwang@redhat.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      9647bb80
    • D
      Merge branch 'legacy-isa-delete' of git://git.kernel.org/pub/scm/linux/kernel/git/paulg/linux · 930d52c0
      David S. Miller 提交于
      Paul Gortmaker says:
      
      ====================
      The Ethernet-HowTo was maintained for roughly 10 years, from 1993 to 2003.
      Fortunately sane hardware probing and auto detection (via PCI and ISA/PnP)
      largely made the document a relic of the past, hence it being abandoned
      a decade ago.
      
      However, there is one last useful thing that we can extract from the
      effort made in maintaining that document.  We can use it to guide us
      with respect to what rare, experimental and/or super ancient 10Mbit
      ISA drivers don't make sense to maintain in-tree anymore.
      
      Nobody will argue that ISA is obsolete.  Availability went away at about
      the time Pentium3 motherboards moved from 500MHz Slot1/SECC processors
      to the green 500MHz Socket 370 Pentium3 chips, at the turn of the century.
      
      In theory, it is possible that someone could still be running one of these
      12+ year old P3 machines and want 3.9+ bleeding edge kernels (but unlikely).
      In light of the above (remote) possibility, we can defer the removal of some
      ISA network drivers that were highly popular and well tested.  Typically
      that means the stuff more from the mid to late '90s, some with ISA PnP
      support, like the 3c509, the wd/SMC 8390 based stuff, PCnet/lance etc.
      
      But a lot of other drivers, typically from the early 1990s were for rare
      hardware, and experimental (to the point of requiring a cron job that would
      do a test ping, and then ifconfig down/up and/or a rmmod/insmod!).  And
      some of these drivers (znet, and lp486e to name two) are physically tied
      to platforms with on motherboard ethernet -- of 486 machines that date
      from the early 1990s and can only have single digit amounts of memory.
      
      What I'd like to achieve here with this series, is to get rid of those old
      drivers that are no longer being used.  In an earlier discussion where
      I'd proposed deleting a single driver, Alan suggested we instead dump
      all the historical stuff in one go, to make it "...immediately obvious
      where the break point is..."[1] and that it was "perfectly reasonable it
      (and a pile of other ISA cards) ought to be shown the door"[2].  So that
      is the goal here - make a clear line in the sand where the really ancient
      stuff finally gets kicked to the curb.
      
      Two old parallel port drivers are considered for removal here as well,
      since in early 386/486 ISA machines, the parallel port was typically found
      with the UARTS on the multi-I/O ISA controller card.  These drivers also date
      from the early 1990's; parallel ports are no longer found on modern boards,
      and their performance was not even capable of 10% of 10Mbit bandwidth.
      
      Allow me a preemptive justification against the inevitable comments from
      well meaning bystanders who suggest "why not just leave all this alone?".
      Dead drivers cost us all if they are left in tree.  If you think that
      is false, then please first consider:
      
      -every time you type "git status", you are checking to see if modifications
       have been made by you to all that dead code.
      
      -every time you type "git grep <regex>" you are searching through files
       which contain that dead code that simply does not interest you.
      
      -every time you build a "allyesconfig" and an "allmodconfig" (don't tell
       me you skip this step before submitting your changes to a maintainer),
       you waste CPU cycles building this dead code.
      
      -every time there is a tree wide API change, or cleanup, or file relocation,
       we pay the cost of updating dead code, or moving dead code.
      
      -daily regression tests (take linux-next as the most transparent
       example) spend time building (and possibly running) this dead code.
      
      -hard working people who regularly run auditing tools looking for lurking
       bugs (sparse/coverity/smatch/coccinelle) are wasting time checking for,
       and fixing bugs in this dead code.
      
      This last one is key.  Please take a look at the git history for the
      files that are proposed for removal here.  Look at the git history for
      any one of them ("git whatchanged --follow drivers/net/.../driver.c")
      Mentally sort the changes into two bins -- (1) the robotic tree-wide
      changes, and (2) the "look I found a real run-time bug while using this"
      category.  You will see that category #2 is essentially empty.
      
      Further to that, realize that drivers don't simply disappear.  We are
      not operating in the binary-only distribution space like other OS.  All
      these drivers remain in the git history forever.  If a person is an
      enthusiast for extreme legacy hardware, they are probably already
      customizing their kernel source and building it themselves to support
      such systems.  Also keep in mind that they could still build the 3.8
      kernel exactly as-is, and run it (or a 3.8.x stable variant of it) for
      several more years if they were really determined to cling to these old
      experimental ISA drivers for some reason.
      
      In summary, I hope that folks can be pragmatic about this, and not
      get swept up in nostalgia.  Ask yourself whether it is realistic to
      expect a person would have a genuine use case where they would
      need to build a 3.9+ modern kernel and install it on some legacy hardware
      that has no option but to absolutely _require_ one of the drivers
      that are deleted here.
      
      The following series was created with --irreversible-delete for
      ease of review (it skips showing the content of files that are
      deleted); however the complete patches can be pulled as per below.
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      930d52c0
    • Y
    • Y
      3f0d2ba0
    • Y
      50c3a487
    • Y
      8fbcec24
    • Y
      neigh: Keep neighbour cache entries if number of them is small enough. · 2724680b
      YOSHIFUJI Hideaki / 吉藤英明 提交于
      Since we have removed NCE (Neighbour Cache Entry) reference from
      routing entries, the only refcnt holders of an NCE are its timer
      (if running) and its owner table, in usual cases.  As a result,
      neigh_periodic_work() purges NCEs over and over again even for
      gateways.
      
      It does not make sense to purge entries, if number of them is
      very small, so keep them.  The minimum number of entries to keep
      is specified by gc_thresh1.
      Signed-off-by: NYOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      2724680b
    • N
      ipmr: fix sparse warning when testing origin or group · 360eb5da
      Nicolas Dichtel 提交于
      mfc_mcastgrp and mfc_origin are __be32, thus we need to convert INADDR_ANY.
      Because INADDR_ANY is 0, this patch just fix sparse warnings.
      Reported-by: NFengguang Wu <fengguang.wu@intel.com>
      Signed-off-by: NNicolas Dichtel <nicolas.dichtel@6wind.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      360eb5da
  3. 22 1月, 2013 29 次提交
    • P
      drivers/net: delete old x86 variant of the seeq8005 driver · 463d413c
      Paul Gortmaker 提交于
      The last update to the Ethernet HowTo (over 10 years ago) listed this:
      
       ------------------------
         SEEQ 8005
      
         Status: Obsolete, Driver Name: seeq8005
      
         There is little information about the card included in the driver,
         and hence little information to be put here. If you have a question,
         you are probably best trying to e-mail the driver author as listed
         in the source.
      
         It was marked obsolete as of the 2.4 series kernels.
       ------------------------
      
      If it was obsolete over a decade ago, the situation can not have
      improved with the passage of time, so let us act on that.  Even with
      today's improved search engines, I was unable to locate any real
      meaningful information on the ISA implementation of this rare chip.
      
      There are ARM and SGI variants of the driver in tree, but they do
      not depend on the original x86 driver source or header file.  We
      leave those non-x86 drivers to be deleted by the arch maintainers
      when they decide to expire those legacy platforms as a whole.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      463d413c
    • P
      drivers/net: delete Digital EtherWorks-3 support. · 0ffd89e4
      Paul Gortmaker 提交于
      This is another one that makes sense to target for obsolescence, since
      it (a)appeared pre-1995, and (b)was rather rare, and (c)did not
      really have any statistically significant active linux user base.
      
      Removing this ISA 10Mbit driver support is unlikely to be even noticed
      by the user base of 3.9+ linux kernels, especially when the documentation
      clearly indicates the vintage with this text:
      
      	 "...designed to  work with all kernels > 1.1.33"
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      0ffd89e4
    • P
      drivers/net: delete old DEC depca ISA drivers support. · 1f1c7a5c
      Paul Gortmaker 提交于
      These are old ISA 10Mbit cards from the 1st 1/2 of the 1990s and
      required manual jumper settings in order to configure them.  Here
      we remove them on the premise that they are no longer used in any
      modern 3.9+ kernels.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      1f1c7a5c
    • P
      drivers/net: delete the really obsolete 8390 based 10Mbit ISA drivers · fce3cd45
      Paul Gortmaker 提交于
      This is an area I know all too well, after being author of several 8390
      drivers, and maintainer of all 8390 drivers during a large part of their
      active lifecycle.
      
      To that end, I can say this with a reasonable degree of confidence.
      The drivers deleted here represent the earliest (as in early 1990)
      hardware and/or rare hardware.  The remaining hardware not deleted
      here is the more modern/sane of the lot, with ISA-PnP and jumperless
      "soft configuration" like the wd and smc cards had.
      
      The original ne2000 driver (ne.c) gets a pass at this time since
      AT/LANTIC based cards that could be both ne2000 or wd-like (with
      shared memory) and with jumperless configuration were made in the
      mid to late 1990's, and performed reasonably well for their era.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      fce3cd45
    • P
      drivers/net: delete old fujitsu based eth16i driver · bb37f122
      Paul Gortmaker 提交于
      This is another driver for relatively rare 10Mbit hardware that
      originated in the early 1990's.  So we select it for removal at
      this point in time as well.
      
      Cc: Mika Kuoppala <miku@iki.fi>
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      bb37f122
    • P
      drivers/net: delete at1700 ISA 10Mbit driver · 13a80cb8
      Paul Gortmaker 提交于
      These Fujitsu MB86965 based ISA 10Mbit cards were another of the
      relatively rare cards dating from the early 1990s that for one reason
      or another didn't seem to get a lot of use in linux.  So we retire it
      now with a reasonable degree of confidence that it won't impact anyone.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      13a80cb8
    • P
      drivers/net: delete old 8 bit ISA Racal ni5010 support. · d2477de7
      Paul Gortmaker 提交于
      These cards were only available in 8bit format, and in addition
      they only had AUI and BNC(10-Base2) interfaces (i.e. no RJ-45).
      
      In fact, they are so rare, that an internet search on these old
      cards almost comes up empty, unless the "Micom interlan" name
      is used.
      
      This puts them in the equivalent domain as the 3c501, so there
      should be no strong opposition to the driver removal, as nobody
      is seriously using 3.9+ with 8 bit ISA hardware.
      
      In doing so, the whole "ethernet/racal" category becomes empty,
      so we clean up the Makefile/Kconfig and subdir appropriately.
      
      Cc: Andreas Mohr <andi@lisas.de>
      Cc: Jan-Pascal van Best <janpascal@vanbest.org>
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      d2477de7
    • P
      drivers/net: delete Racal Interlan ISA ni52 (i825xx) driver · 04861c53
      Paul Gortmaker 提交于
      Like the other drivers that were in the ISA i825xx family, the ni52
      was rather rare, not widely used, and hence perhaps not as reliable
      as the more mainstream ISA drivers that were heavily used.  Given
      that, it is chosen for retirement at this time as well.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      04861c53
    • P
      drivers/net: delete intel i825xx based znet notebook driver · 8a594170
      Paul Gortmaker 提交于
      This driver supported early to mid 1990's Zenith laptops, of the
      2" thick variety.  The driver was already dead 10+ years ago, but
      we see this in the source:
      
       ----------------
       /* 10/2002
      
       [...]
      
         Tested on a vintage Zenith Z-Note 433Lnp+. Probably broken on
         anything else. Testers (and detailed bug reports) are welcome :-).
       ----------------
      
      To clarify, a 433 translates into a 486 at 33MHz, and a system with
      a default of 4MB RAM.  I can't fault the noble effort to keep things
      working a decade ago, but at this point in time, there is no valid
      justification to continue carrying this driver along.
      
      Note that there is no associated Space.c cleanup here since this
      driver was using module_init to hook itself in.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      8a594170
    • P
      drivers/net: delete ISA intel eexpress and eepro i825xx drivers · f84932d8
      Paul Gortmaker 提交于
      These old drivers should not be confused with the very common PCI
      cards that are supported by e100.c -- these older 10Mbit ISA only
      drivers were not as commonly used as some of the other ISA drivers,
      simply due to hardware availability and pricing.
      
      Given the rarity of the hardware, and the subsequent less extensive
      use of the drivers, it makes sense to obsolete them at this point
      in time, along with other rare/experimental ISA drivers.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      f84932d8
    • P
      drivers/net: delete the 3Com 3c505/3c507 intel i825xx support · 0e245dba
      Paul Gortmaker 提交于
      For those of us who were around in the early to mid 1990's, we
      will remember that the i825xx ethernet support was not something
      that was considered sufficiently vetted for 24/7 use.
      
      Folks might be inclined to use *functional* ISA hardware on some
      near expired P3 ISA machines for dedicated workhorse applications,
      but the odds of using (and relying on) one of these old/experimental
      drivers is essentially nil.  So lets remove them.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      0e245dba
    • P
      drivers/net: delete old parallel port de600/de620 drivers · 168e06ae
      Paul Gortmaker 提交于
      The parallel port is largely replaced by USB, and even in the
      day where these drivers were current, the documented speed was
      less than 100kB/s.  Let us not pretend that anyone cares about
      these drivers anymore, or worse - pretend that anyone is using
      them on a modern kernel.
      
      As a side bonus, this is the end of legacy parallel port ethernet,
      so we get to drop the whole chunk relating to that in the legacy
      Space.c file containing the non-PCI unified probe dispatch.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      168e06ae
    • P
      drivers/net: delete old 8bit ISA 3c501 driver. · de8270ff
      Paul Gortmaker 提交于
      It was amusing that linux was able to make use of this 1980's
      technology on machines long past its intended lifespan, but
      it probably should go now.
      
      To set some context, the 3c501 was designed in the 1980's to be
      used on 8088 PC-XT 8bit ISA machines.  It was built using a large
      number of discrete TTL components and truly looks like a relic
      of the ancient past before large scale integration was common.
      
      But from a functional point of view, the real issue, as stated
      in the (also obsolete) Ethernet-HowTo, is that "...the 3c501 can
      only do one thing at a time -- while you are removing one packet
      from the single-packet buffer it cannot receive another packet,
      nor can it receive a packet while loading a transmit packet."
      
      You know things are not good when the Kconfig help text suggests
      you make a cron job doing a ping every minute.
      
      Hardware that old and crippled is simply not going to be used by
      anyone in a time where 10 year old 100Mbit PCI cards (that are
      still functional) are largely give-away items.
      
      Cc: Alan Cox <alan@linux.intel.com>
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      de8270ff
    • P
      drivers/net: delete intel 486 panther onboard ethernet support · 5205939d
      Paul Gortmaker 提交于
      This driver was specific to a "professional workstation" line
      of products from around 1993 that used the i82596 ethernet chip
      as an on-board ethernet solution.
      
      With a 486 processor, and the premium top of the line model maxing
      out at a clock speed of 50MHz, we can safely retire this support.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      5205939d
    • P
      drivers/net: delete 486 Apricot support · 6e07ba3e
      Paul Gortmaker 提交于
      The Apricot was a 486 PC with 4MB RAM, and an on-board ethernet
      via an intel i82596 hard-wired to i/o 0x300.
      
      Those who were using linux in the 1990's will recall that the
      i82596 driver was not one of the more stable or widely used
      drivers of its day.  Combine that with the extremely limited
      resources of the platform, and it is truly time to expire the
      support for this thing.
      
      There are some old m68k targets who were also using this chip,
      so rather than poll the m68k user base, we simply cut out the
      x86/Apricot support here in this commit.
      Signed-off-by: NPaul Gortmaker <paul.gortmaker@windriver.com>
      6e07ba3e
    • T
      isdn/gigaset: beautify ev-layer.c · 63b203b4
      Tilman Schmidt 提交于
      Cosmetic changes to drivers/isdn/gigaset/ev-layer.c and
      drivers/isdn/gigaset/gigaset.h to improve readability.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      63b203b4
    • T
      isdn/gigaset: beautify common.c · bc882b18
      Tilman Schmidt 提交于
      Rearrange the gigaset_freecs() function to make it more readable,
      and adapt gigaset_initcs() accordingly.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      bc882b18
    • T
      isdn/gigaset: beautify interface.c · cdc48270
      Tilman Schmidt 提交于
      Avoid forward declarations and remove a needless initialization.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      cdc48270
    • T
      isdn/gigaset: leave DLE mode before hanging up · d2ca8481
      Tilman Schmidt 提交于
      Some firmware releases of Gigaset M105 do not accept AT+VLS=0 command
      in DLE mode, so always leave DLE mode before sending the command.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      d2ca8481
    • T
      isdn/divert: fix readability damage · 03f18285
      Tilman Schmidt 提交于
      Fix up some of the readibility deterioration caused by last year's
      ISDN whitespace coding style cleanup.
      Note that the checkpatch complaints all apply to the state of the
      source before this patch as well, and in many cases even more so.
      Signed-off-by: NTilman Schmidt <tilman@imap.cc>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      03f18285
    • J
      drivers/net/ethernet/sfc/ptp.c: adjust duplicate test · 56567c6f
      Julia Lawall 提交于
      Delete successive tests to the same location.  rc was previously tested and
      not subsequently updated.  efx_phc_adjtime can return an error code, so the
      call is updated so that is tested instead.
      
      A simplified version of the semantic match that finds this problem is as
      follows: (http://coccinelle.lip6.fr/)
      
      // <smpl>
      @s exists@
      local idexpression y;
      expression x,e;
      @@
      
      *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) )
       { ... when forall
         return ...; }
      ... when != \(y = e\|y += e\|y -= e\|y |= e\|y &= e\|y++\|y--\|&y\)
          when != \(XT_GETPAGE(...,y)\|WMI_CMD_BUF(...)\)
      *if ( \(x == NULL\|IS_ERR(x)\|y != 0\) )
       { ... when forall
         return ...; }
      // </smpl>
      Signed-off-by: NJulia Lawall <Julia.Lawall@lip6.fr>
      Acked-by: NBen Hutchings <bhutchings@solarflare.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      56567c6f
    • Y
      ndisc: Do not try to update "updated" time if neighbour has already gone. · b820bb6b
      YOSHIFUJI Hideaki / 吉藤英明 提交于
      Commit 2152caea ("ipv6: Do not depend on rt->n in rt6_probe().")
      introduce a bug to try to update "updated" time in neighbour
      structure.
      Update the "updated" time only if neighbour is available.
      
      Bug was found by Dan Carpenter <dan.carpenter@oracle.com>
      Signed-off-by: NYOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      b820bb6b
    • D
      Merge branch 'dsa' · 10020414
      David S. Miller 提交于
      Florian Fainelli says:
      
      ====================
      These two patches are non-critical bugfixes based on net-next which I
      stumbled upon while working on Device Tree bindings for DSA (will comme
      as a separate patch later).
      
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      10020414
    • F
      dsa: make dsa_switch_setup check for valid port names · f9bf5a2c
      Florian Fainelli 提交于
      This patch changes dsa_switch_setup() to ensure that at least one valid
      valid port name is specified and will bail out with an error in case we
      walked the maximum number of port with a valid port name found.
      Signed-off-by: NFlorian Fainelli <florian@openwrt.org>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      f9bf5a2c
    • F
      dsa: use an unique and non conflicting bus name for the slave MII bus · f490be04
      Florian Fainelli 提交于
      The slave MII bus registered by the DSA code is using the parent MII bus
      as part of its name (ds->master_mii_bus_id), in case the parent MII bus
      name is already 16 characters long (such as d0072004.mdio-mi) we will
      get the following WARN_ON in dsa_switch_setup() when calling
      mdiobus_register():
      
      [   79.088782] ------------[ cut here ]------------
      [   79.093448] WARNING: at fs/sysfs/dir.c:536 sysfs_add_one+0x80/0xa0()
      [   79.099831] sysfs: cannot create duplicate filename
      '/class/mdio_bus/d0072004.mdio-mi'
      
      This is a genuine warning, because the DSA slave MII bus will also be
      named d0072004.mdio-mi, and since MII_BUS_ID_SIZE is 17 characters long
      (with null-terminator) the following will truncate the slave MII bus id:
      
      snprintf(ds->slave_mii_bus->id, MII_BUS_ID_SIZE, "%s-%d:%.2x",
                              ds->master_mii_bus->id, ds->pd->sw_addr);
      
      Fix this by using dsa-<switch index->:<sw_add> which is guaranteed to be
      unique.
      Signed-off-by: NFlorian Fainelli <florian@openwrt.org>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      f490be04
    • C
      net: move rx and tx hash functions to net/core/flow_dissector.c · 441d9d32
      Cong Wang 提交于
      __skb_tx_hash() and __skb_get_rxhash() are all for calculating hash
      value based by some fields in skb, mostly used for selecting queues
      by device drivers.
      
      Meanwhile, net/core/dev.c is bloating.
      
      Cc: "David S. Miller" <davem@davemloft.net>
      Cc: Eric Dumazet <edumazet@google.com>
      Cc: Ben Hutchings <bhutchings@solarflare.com>
      Signed-off-by: NCong Wang <xiyou.wangcong@gmail.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      441d9d32
    • D
      Merge branch 'virtio_mac' · cd5f2046
      David S. Miller 提交于
      Amos Kong says:
      
      ====================
      Currenly mac is programmed byte by byte. This means that we
      have an intermediate step where mac is wrong.
      
      Third patch introduced a new vq control command to set mac
      address, it's atomic.
      
      V2: check return of sending command, delay eth_mac_addr()
      V3: restore software address when fail to set hardware address
      V4: split eth_mac_addr, fix error handle
      V5: rebase patches to net-next tree
      ====================
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      cd5f2046
    • A
      virtio-net: introduce a new control to set macaddr · 7e58d5ae
      Amos Kong 提交于
      Currently we write MAC address to pci config space byte by byte,
      this means that we have an intermediate step where mac is wrong.
      This patch introduced a new control command to set MAC address,
      it's atomic.
      
      VIRTIO_NET_F_CTRL_MAC_ADDR is a new feature bit for compatibility.
      Signed-off-by: NAmos Kong <akong@redhat.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      7e58d5ae
    • S
      net: split eth_mac_addr for better error handling · fa0879e3
      Stefan Hajnoczi 提交于
      When we set mac address, software mac address in system and hardware mac
      address all need to be updated. Current eth_mac_addr() doesn't allow
      callers to implement error handling nicely.
      
      This patch split eth_mac_addr() to prepare part and real commit part,
      then we can prepare first, and try to change hardware address, then do
      the real commit if hardware address is set successfully.
      Signed-off-by: NStefan Hajnoczi <stefanha@gmail.com>
      Signed-off-by: NAmos Kong <akong@redhat.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      fa0879e3