[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freebsd-net
Subject:    [Bug 221317] ifconfig down/up issue after ixgbe driver update in r320897
From:       bugzilla-noreply () freebsd ! org
Date:       2019-01-31 21:45:00
Message-ID: bug-221317-7501-0G5w6tEGxt () https ! bugs ! freebsd ! org/bugzilla/
[Download RAW message or body]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221317

--- Comment #41 from commit-hook@freebsd.org ---
A commit references this bug:

Author: erj
Date: Thu Jan 31 21:44:34 UTC 2019
New revision: 343621
URL: https://svnweb.freebsd.org/changeset/base/343621

Log:
  ix(4): Run {mod,msf,mbx,fdir,phy}_task in if_update_admin_status

  From Piotr:

  This patch introduces adapter->task_requests register responsible for
  recording requests for mod_task, msf_task, mbx_task, fdir_task and
  phy_task calls. Instead of enqueueing these tasks with
  GROUPTASK_ENQUEUE, handlers will be called directly from
  ixgbe_if_update_admin_status() while holding ctx lock.

  SIOCGIFXMEDIA ioctl() call reads adapter->media list. The list is
  deleted and rewritten in ixgbe_handle_msf() task without holding ctx
  lock. This change is needed to maintain data coherency when sharing
  adapter info via ioctl() calls.

  Patch co-authored by Krzysztof Galazka <krzysztof.galazka@intel.com>.

  PR:           221317
  Submitted by: Piotr Pietruszewski <piotr.pietruszewski@intel.com>
  Reviewed by:  sbruno@, IntelNetworking
  Sponsored by: Intel Corporation
  Differential Revision:        https://reviews.freebsd.org/D18468

Changes:
  head/sys/dev/ixgbe/if_ix.c
  head/sys/dev/ixgbe/ixgbe.h
  head/sys/dev/ixgbe/ixgbe_type.h

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
freebsd-net@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic