]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/log
FreeBSD/FreeBSD.git
8 years agoAllow silencing of 'promiscuous mode enabled/disabled' messages.
Nick Hibma [Thu, 12 May 2016 19:42:13 +0000 (19:42 +0000)]
Allow silencing of 'promiscuous mode enabled/disabled' messages.

PR: 166255
Submitted by: eugen.grosbein.net
Obtained from: eugen.grosbein.net
MFC after: 1 week

8 years agoDocument r296633, OpenSSH updated to 7.2p2.
Steven Kreuzer [Thu, 12 May 2016 19:32:40 +0000 (19:32 +0000)]
Document r296633, OpenSSH updated to 7.2p2.

Approved by: gjb@ (implicit with re@ hat on)

8 years agoixl: Re-add a change to TC0 setup made in D5203.
Eric Joyner [Thu, 12 May 2016 18:22:34 +0000 (18:22 +0000)]
ixl: Re-add a change to TC0 setup made in D5203.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.27-k.
Eric Joyner [Thu, 12 May 2016 18:22:12 +0000 (18:22 +0000)]
ixl: Update to 1.4.27-k.

Changes:

Kevin Scott i40e-shared: Save off VSI resource count when updating VSI
Anjali Singhai Jain i40e-shared: Expose some registers to program parser, FD and RSS logic
Eric Joyner ixl: Fix errors in queue interrupt setup in MSIX mode.
Shannon Nelson i40e-shared: implement and use rx ctl helper functions
Shannon Nelson i40e-shared: don't use AQ calls from clear_hw
Eric Joyner ixl: Use rx ctl read/write functions instead of register accesses in certain places.
Shannon Nelson i40e-shared: add adminq commands for rx ctl registers
Shannon Nelson i40e-shared: implement and use rx ctl helper functions
Jeremiah Kyle i40e-shared: Corrected function name in comment block
Deepthi Kavalur i40e-shared: correcting a HW capability display info
Shannon Nelson i40e-shared: fixups for (Linux) upstream consistency
Eric Joyner ixl: Only stop firmware's LLDP agent on older firmware versions.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.24-k.
Eric Joyner [Thu, 12 May 2016 18:21:52 +0000 (18:21 +0000)]
ixl: Update to 1.4.24-k.

Changes by author:

Eric Joyner ixl: Fix compile error when IXL_DEBUG is defined.
Eric Joyner ixl: Fix taskqueues created in init() not being freed in stop().
Eric Joyner ixl: Add additional debug sysctls, for Tx and Rx queue stats.
Eric Joyner ixl: Enable dynamic itr by default.
Eric Joyner ixl: Edit spacing, comments,  function signatures (to conform to style(9)).
Eric Joyner ixl: Check for errors when tearing down msix interrupts.
Eric Joyner ixl: Remove unnecessary register reads/writes.
Eric Joyner ixl: Remove admin queue interrupt enable from general interrupt enable.
Eric Joyner ixl: Update switch config after teardown/reset flow in init().
Eric Joyner ixl: Add additional admin queue error code output to admin queue call errors.
Eric Joyner ixl: Don't destroy i40e spinlock if it's already uninitialized.
Shannon Nelson i40e-shared: clean event descriptor before use
Anjali Singhai Jain i40e-shared: When in promisc mode apply promisc mode to Tx Traffic as well
Kevin Scott i40e_shared: Increase timeout when checking GLGEN_RSTAT_DEVSTATE bit
Eric Joyner ixlv: Fix IXL_DEBUG compile issue.
Eric Joyner ixlv: Attempt to fix panic/other issues when rapidly unloading/loading driver.
Eric Joyner ixl/ixlv: Revert m_collapse() in ixl_xmit() to m_defrag().
Deepthi Kavalur i40e_shared: Trace logging HW capabilities
Eric Joyner ixlv: Correctly unlock/relock around init() call in vc_completion().
Eric Joyner ixl: Stop preventing changing flow control mode for CR4 media.
Eric Joyner ixl: Set IPv6 TCP offload flag when doing TSO.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.20-k.
Eric Joyner [Thu, 12 May 2016 18:21:34 +0000 (18:21 +0000)]
ixl: Update to 1.4.20-k.

Changes by author:

Eric Joyner ixl: Add more error messages/checks to ixl_vsi_assign_msix().
Eric Joyner ixl/ixlv: Clarify a comment about descriptors.
Eric Joyner ixl/ixlv: Improve i40e_debug() implementation.
Eric Joyner ixl/ixlv: Remove unused ASSERT() macro; move struct around.
Eric Joyner ixl: Set initial advertised speed value in init_locked().
Eric Joyner ixl: Fix flow control sysctl value being stored when new value is invalid.
Eric Joyner Edit comments and spacing.
Carolyn Wyborny i40e-shared: Add functions to blink led on Coppervale PHY
Eric Joyner ixl: Re-do interrupt setup.
Eric Joyner ixl: Remove VFLR task setup from legacy flow.
Eric Joyner ixl: Shutdown/setup HMC when handling an EMPR reset.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.17-k.
Eric Joyner [Thu, 12 May 2016 18:21:17 +0000 (18:21 +0000)]
ixl: Update to 1.4.17-k.

Changes:

Kiran Patil i40e-shared: APIs to Add/remove port mirroring rules
Shannon Nelson i40e-shared: add VEB stat control and remove L2 cloud filter
Eric Joyner ixl: Update NVM version information shown.
Eric Joyner ixl: Remove empty else block.
Eric Joyner ixl: Slightly re-work ixl_init_msix().
Eric Joyner ixl: Remove duplicate queue enablement.
Shannon Nelson i40e-shared: implement the API function for aq_set_switch_config
Eric Joyner ixl: Update nvm version string shown in sysctl.
Eric Joyner ixl/ixlv: Changes to PF/VF minor version checking/handling.
Eric Joyner ixlv: Reduce maximum wait time for responses to VF AQ messages.
Eric Joyner ixl/ixlv: Edit comments, comment out code, and edit spacing.
Eric Joyner ixl: Print log message when SR-IOV init is successful.
Eric Joyner ixl: Add Tx Flow Control filter from main PF VSI.
Eric Joyner ixlv: Add extra error message when ixlv_get_vf_config times out.
Eric Joyner ixl: Assign current MOCS optics the XLPPI media type.
Eric Joyner ixl: Remove conditional wait after link status event.
Eric Joyner ixl: Add line break and remove extraneous return statement.
Eric Joyner ixl: Allow 40G speeds in the advertise_speed sysctl.
Eric Joyner ixl: Add "CRC enable" field to link_status sysctl output.
Eric Joyner ixl: Move sbuf.h include out of IXL_DEBUG* defines.
Eric Joyner ixl: Move remaining debug sysctl funcs to IXL_DEBUG_SYSCTL define.
Eric Joyner ixl: Add cases for all remaining media types in shared code to media_status().

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.13-k.
Eric Joyner [Thu, 12 May 2016 18:20:59 +0000 (18:20 +0000)]
ixl: Update to 1.4.13-k.

Changes:

Shannon Nelson i40e-shared: set shared bit for multicast filters
Piotr Raczynski i40e-shared: Bump AQ minor version for FVL/FPK
Eric Joyner ixl: Remove call to i40e_aq_set_mac_config() in init path.
Eric Joyner ixl: Always add interface mac address to driver+hw filter list.
Eric Joyner ixl: Edit comments.
Eric Joyner ixl: Add more descriptive sysctl help messages.
Eric Joyner ixl: Report more info in link_status sysctl.
Eric Joyner ixl: Flip set_aq_phy_int_mask mask bits.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.12-k.
Eric Joyner [Thu, 12 May 2016 18:20:36 +0000 (18:20 +0000)]
ixl: Update to 1.4.12-k.

Changes by author:

Eric Joyner ixl: Remove substitution of EACCES for EPERM when perrno is set on an nvmupdate command return.
Eric Joyner ixl: Print message when hardware sends GRST interrupt.
Eric Joyner ixl: Fix kernel panic when driver fails to initialize admin queue.
Eric Joyner ixl: Print out messages when a non-handled other interrupt occurs.
Eric Joyner ixl: Fix spaces in a couple messages.
Eric Joyner ixl: Add lock around nvmupd command entry point and reconvert EPERM errors to EACCES.
Anjali Singhai Jain i40e-shared: Make some changes in the nvm read code
Shannon Nelson i40e-shared: AQ Add Run PHY Activity struct
Shannon Nelson i40e-shared: AQ Add Geneve cloud tunnel type
Shannon Nelson i40e-shared: AQ Add external power class to get link status response struct
Shannon Nelson i40e-shared: AQ Add shared resource flags for macvlan filters
Shannon Nelson i40e-shared: AQ Add set_switch_config
Shannon Nelson i40e-shared: AQ Add VXLAN-GPE tunnel type for cloud filter and tunnel commands
Shannon Nelson i40e-shared: AQ thermal sensor control struct
Shannon Nelson i40e-shared: Bump AQ minor version to 1.5 for FVL5 features
Shannon Nelson i40e-shared: add a little more to an NVM update debug message
Carolyn Wyborny i40e-shared: Fix for PHY NVM interaction problem
Eric Joyner i40e-shared: Add prototypes for private NVM write functions
Eric Joyner ixl/ixlv: Remove unused define from ixl.h.
Eric Joyner ixl: Add handling of EMP reset for nvm update purposes.
Eric Joyner ixl: Move addition of device sysctls to separate function.
Eric Joyner ixl: Fix up a couple error messages in ixl_attach().
Eric Joyner ixl: Update the hardware resource allocation debug sysctl.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.9-k.
Eric Joyner [Thu, 12 May 2016 18:20:18 +0000 (18:20 +0000)]
ixl: Update to 1.4.9-k.

Changes by author:

Kamil Krawczyk i40e-shared: use explicit cast from u16 to u8
Anjali Singhai Jain i40e-shared: Add a Virtchnl offload for RSS PCTYPE V2
Eric Joyner ixl: Remove HP device IDs.
Jesse Brandeburg i40e-shared: add small bit of debug
Mitch Williams i40e-shared: check for stopped admin queue
Mitch Williams i40e-shared: set aq count after memory allocation
Jesse Brandeburg i40e-shared: remove forever unused ID
Eric Joyner ixl: Fix bug where RSS does not hash to more than 16 queues.
Shannon Nelson i40e-shared: define function capabilities in only one place
Eric Joyner ixl: Change spacing, comments, and a single error message.
Eric Joyner ixl: Save admin queue phy interrupt mask as a define.
Eric Joyner ixl: Move callout_reset() to later in init_locked(), and stop clearing OACTIVE in driver flags.
Eric Joyner ixl: Add new reset+build flow to init() if it detects that the admin queue is stopped.
Eric Joyner ixl: Return EACCES instead of EPERM when an nvmupdate command fails.
Eric Joyner ixl: Remove KX_A device ID.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.7-k.
Eric Joyner [Thu, 12 May 2016 18:19:53 +0000 (18:19 +0000)]
ixl: Update to 1.4.7-k.

Changes by author:

Helin Zhang i40e_shared: Fix compilation error - pointer-arith
Paul M Stillwell Jr i40e-shared: Replace sprintf with i40e_debug
Anjali Singhai Jain i40e-shared: Fix an accidental error with BIT_ULL replacement
Jesse Brandeburg i40e-shared: remove useless assignments
Anjali Singhai Jain i40e-shared: Add a workaround to drop all flow control frames
Anjali Singhai Jain i40e-shared: Add new response struct from FW for AQ command i40e_aqc_lldp_set_local_mib
Anjali Singhai Jain i40e-shared: Acquire NVM, before issuing an AQ read nvm command
Eric Joyner ixl/ixlv: Remove unused MAX_LOOP define.
Eric Joyner ixl: Remove extra aq_get_link_info() call in attach().
Eric Joyner ixl: Modify a couple error messages in attach() to be more informative.
Eric Joyner ixl: Add i40e_get_link_status() call to init_locked().
Eric Joyner ixl: Move callout_stop() to earlier in ixl_stop().
Eric Joyner ixl: Add extra comments around link ITR code.
Eric Joyner ixl: Attempt to enhance link event handling.
Eric Joyner ixl: Style, spacing, and comment changes.
Eric Joyner ixl: Add I40E_NVM_ACCESS definition.
Eric Joyner ixl: Add interface for nvmupdate tool ioctl to driver.
Eric Joyner ixl: Don't strip out nvm update support from the driver anymore.
Eric Joyner ixl: Interrupts are now allocated/setup and torn down/released on init()/stop().

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.6-k.
Eric Joyner [Thu, 12 May 2016 18:19:31 +0000 (18:19 +0000)]
ixl: Update to 1.4.6-k.

Changes by author:

Greg Rose ixl: Add MAC+VLAN filter for LAA MAC assignment
Carolyn Wyborny i40e_shared: fix for PHY NVM interaction problem
Anjali Singhai Jain i40e-shared: Fix an accidental error with BIT_ULL replacement
Eric Joyner ixl: Fix potential crash when loading driver a startup and cannot setup all requested MSIX vectors.
Eric Joyner ixl: Add max of 8 queues limit to autoconfig.
Eric Joyner ixl: Re-add ian@'s sysctl fixes from upstream (r280043).

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoixl: Update to 1.4.5-k.
Eric Joyner [Thu, 12 May 2016 18:18:55 +0000 (18:18 +0000)]
ixl: Update to 1.4.5-k.

This first update will revert some upstream changes; forthcoming updates will reinstate them.

Changes, by author:

Anjali Singhai Jain i40e-shared: Add WB_ON_ITR offload support
Shannon Nelson i40e-shared: fix phy_types bitmap type
Kevin Scott i40e-shared: Store off PHY capabilities
Shannon Nelson i40e-shared: fix byteswap of phy_type
Jingjing Wu i40e-shared: Fix compile issue related to const string
Greg Bowers i40e-shared: Add AQ defines for non-willing Apps (DCB)
Greg Bowers i40e-shared: Support for non-willing Apps (DCB)
Shannon Nelson i40e-shared: use upper-32 bit macro for address
Shannon Nelson i40e-shared: grab the AQ spinlocks before clearing registers
Eric Joyner ixl: Properly strip out X722_SUPPORT (temporarily).
Eric Joyner ixl: Allow Fort Pond devices to advertise 100M in set_advertise sysctl.

Differential Revision:  https://reviews.freebsd.org/D6211
Reviewed by:    sbruno, kmacy, jeffrey.e.pieper@intel.com
MFC after:      2 weeks
Sponsored by:   Intel Corporation

8 years agoMove mutex initialization from PCI probe to PCI attach. Drivers are not
Scott Long [Thu, 12 May 2016 17:47:30 +0000 (17:47 +0000)]
Move mutex initialization from PCI probe to PCI attach.  Drivers are not
allowed to create any persistent state in their probe routine because it's
not guaranteed that they'll win the election and be allowed to attach.

Submitted by: Matthew Macy
MFC after: 3 days

8 years agoRetire net.inet.sctp.strict_sacks and net.inet.sctp.strict_data_order
Michael Tuexen [Thu, 12 May 2016 16:34:59 +0000 (16:34 +0000)]
Retire net.inet.sctp.strict_sacks and net.inet.sctp.strict_data_order
sysctl's, since they where only there to interop with non-conformant
implementations. This should not be a problem anymore.

8 years agoxen-netfront: fix feature detection
Roger Pau Monné [Thu, 12 May 2016 16:18:02 +0000 (16:18 +0000)]
xen-netfront: fix feature detection

Current netfront code relies on xs_scanf returning a value < 0 on error,
which is not right, xs_scanf returns a positive value on error.

MFC after: 3 days
Tested by: Stephen Jones <StephenJo@LivingComputerMuseum.org>
Sponsored by: Citrix Systems R&D

8 years ago[siba] add more MCS tx power offset decoding.
Adrian Chadd [Thu, 12 May 2016 16:14:16 +0000 (16:14 +0000)]
[siba] add more MCS tx power offset decoding.

This is required by the bwn(4) N-PHY support.

Obtained from: Linux ssb (definitions)

8 years agoMFV 299539:
Martin Matuska [Thu, 12 May 2016 16:07:25 +0000 (16:07 +0000)]
MFV 299539:
Record mergeinfo in contrib/libarchive for patches accepted upstream.

8 years agoUpdate vendor/libarchive/dist to git commit f48d99b
Martin Matuska [Thu, 12 May 2016 16:03:55 +0000 (16:03 +0000)]
Update vendor/libarchive/dist to git commit f48d99b
Integrates my pull requests #701, #702 and #703

8 years ago[siba] add TX power index entry parsing.
Adrian Chadd [Thu, 12 May 2016 16:01:33 +0000 (16:01 +0000)]
[siba] add TX power index entry parsing.

This is required by the bwn(4) N-PHY support.

Obtained from: Linux ssb (definitions)

8 years agoEnable SACK Immediately per default.
Michael Tuexen [Thu, 12 May 2016 15:48:08 +0000 (15:48 +0000)]
Enable SACK Immediately per default.

This has been tested for a long time and implements covered by RFC 7053.

MFC after: 1 week

8 years agoRename the internal BUC_DMA_* flags to BF_* so they won't conflict with
Andrew Turner [Thu, 12 May 2016 15:24:36 +0000 (15:24 +0000)]
Rename the internal BUC_DMA_* flags to BF_* so they won't conflict with
the flags in sys/bus_dma.h.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

8 years ago[siba] add extra methods for chipcommon access and PLL configuration.
Adrian Chadd [Thu, 12 May 2016 15:18:36 +0000 (15:18 +0000)]
[siba] add extra methods for chipcommon access and PLL configuration.

This is needed by the PHY-N bwn(4) support.

Obtained from: Linux b43 (definitions)

8 years agoUse a format string in snprintf() for consistency.
Michael Tuexen [Thu, 12 May 2016 14:41:53 +0000 (14:41 +0000)]
Use a format string in snprintf() for consistency.

This was reported by Radek Malcic when using the userland stack in
combination with MinGW.

MFC after: 1 week

8 years agoRestrict the memory barriers in bus_dmamap_sync to just the operations
Andrew Turner [Thu, 12 May 2016 12:13:28 +0000 (12:13 +0000)]
Restrict the memory barriers in bus_dmamap_sync to just the operations
where it's needed.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

8 years agoFix handling of IOCTLs in the LinuxKPI.
Hans Petter Selasky [Thu, 12 May 2016 11:38:28 +0000 (11:38 +0000)]
Fix handling of IOCTLs in the LinuxKPI.

Linux requires that all IOCTL data resides in userspace. FreeBSD
always moves the main IOCTL structure into a kernel buffer before
invoking the IOCTL handler and then copies it back into userspace,
before returning. Hide this difference in the "linux_copyin()" and
"linux_copyout()" functions by remapping userspace addresses in the
range from 0x10000 to 0x20000, to the kernel IOCTL data buffer.

It is assumed that the userspace code, data and stack segments starts
no lower than memory address 0x400000, which is also stated by "man 1
ld", which means any valid userspace pointer can be passed to regular
LinuxKPI handled IOCTLs.

Bump the FreeBSD version to force recompilation of all kernel modules.

Discussed with: kmacy @
MFC after: 1 week
Sponsored by: Mellanox Technologies

8 years agoMFV r299425:
Martin Matuska [Thu, 12 May 2016 10:16:16 +0000 (10:16 +0000)]
MFV r299425:

Update libarchive to 3.2.0

New features:
- new bsdcat command-line utility
- LZ4 compression (in src only via external utility from ports)
- Warc format support
- 'Raw' format writer
- Zip: Support archives >4GB, entries >4GB
- Zip: Support encrypting and decrypting entries
- Zip: Support experimental streaming extension
- Identify encrypted entries in several formats
- New --clear-nochange-flags option to bsdtar tries to remove noschg and
  similar flags before deleting files
- New --ignore-zeros option to bsdtar to handle concatenated tar archives
- Use multi-threaded LZMA decompression if liblzma supports it
- Expose version info for libraries used by libarchive

Patched files (fixed compiler warnings):

contrib/libarchive/cat/bsdcat.c (vendor PR #702)
contrib/libarchive/cat/bsdcat.h (vendor PR #702)
contrib/libarchive/libarchive/archive_read_support_format_mtree.c (PR #701)
contrib/libarchive/libarchive_fe/err.c (vendor PR #703)

MFC after: 1 month
Relnotes: yes

8 years agoRemove redundant "task_struct_set()".
Hans Petter Selasky [Thu, 12 May 2016 09:11:18 +0000 (09:11 +0000)]
Remove redundant "task_struct_set()".
This is done by the "linux_kthread_fn()".

MFC after: 1 week
Sponsored by: Mellanox Technologies

8 years agoCreate a dummy "task_struct" on the stack which is returned by
Hans Petter Selasky [Thu, 12 May 2016 09:06:54 +0000 (09:06 +0000)]
Create a dummy "task_struct" on the stack which is returned by
"current" inside all LinuxKPI file operation callbacks. The "current"
is frequently used for various debug prints, printing the thread name
and thread ID for example.

Obtained from: kmacy @
MFC after: 1 week
Sponsored by: Mellanox Technologies

8 years agoUse strlcpy() instead of strncpy() when copying date and subj
Don Lewis [Thu, 12 May 2016 08:55:57 +0000 (08:55 +0000)]
Use strlcpy() instead of strncpy() when copying date and subj
to ensure that these are properly NUL terminated since they
are passed to printf().

Reported by: Coverity
CID: 974770
MFH: 1 week

8 years agoUse strlcpy() instead of strncpy() when copying the encoding value
Don Lewis [Thu, 12 May 2016 08:41:22 +0000 (08:41 +0000)]
Use strlcpy() instead of strncpy() when copying the encoding value
to ensure that the destination is NUL terminated.  Length truncation
of one more character should not be an issue since encoding values
that long are not supported by libc.  The destination string is
treated as a NUL terminated string, but it is only passed to strcmp()
for comparison to a set of shorter, fixed length strings, so this
is not a serious problem.

Reported by: Coverity
CID: 974769
MFC after: 1 week

8 years agoStop hiding errors that result in failure to mount /dev. Otherwise,
Edward Tomasz Napierala [Thu, 12 May 2016 07:38:10 +0000 (07:38 +0000)]
Stop hiding errors that result in failure to mount /dev.  Otherwise,
missing /dev directory makes one end up with a completely deaf (init
without stdout/stderr) system with no hints on the console, unless
you've booted up with bootverbose.

MFC after: 1 month
Sponsored by: The FreeBSD Foundation

8 years agoUse strlcpy() instead of strncpy() to copy the string returned by
Don Lewis [Thu, 12 May 2016 06:39:13 +0000 (06:39 +0000)]
Use strlcpy() instead of strncpy() to copy the string returned by
setlocale() so that static analyzers know that the string is NUL
terminated.  This was causing a false positive in Coverity even
though the longest string returned by setlocale() is ENCODING_LEN
(31) and we are copying into a 64 byte buffer.  This change is also
a bit of an optimization since we don't need the strncpy() feature
of padding the rest of the destination buffer with NUL characters.

Reported by: Coverity
CID: 974654

8 years agoTypo in comment.
Konstantin Belousov [Thu, 12 May 2016 06:20:26 +0000 (06:20 +0000)]
Typo in comment.

8 years agosfxge(4): update multicast filter insertion algorithm
Andrew Rybchenko [Thu, 12 May 2016 06:20:26 +0000 (06:20 +0000)]
sfxge(4): update multicast filter insertion algorithm

When the multicast filters we're allowed to insert are controlled by the
hypervisor, it may be that we can insert some but not others. So we need
to have fallbacks where we insert any filters we can without rolling back
when one fails to insert.

Submitted by:   Mark Spender <mspender at solarflare.com>
Sponsored by:   Solarflare Communications, Inc.
MFC after:      1 week
Differential Revision:  https://reviews.freebsd.org/D6318

8 years agosfxge(4): cleanup: constify common code method tables
Andrew Rybchenko [Thu, 12 May 2016 06:19:06 +0000 (06:19 +0000)]
sfxge(4): cleanup: constify common code method tables

Submitted by:   Andy Moreton <amoreton at solarflare.com>
Sponsored by:   Solarflare Communications, Inc.
MFC after:      1 week
Differential Revision:  https://reviews.freebsd.org/D6317

8 years agoCheck for socket creation success before calling bind().
Don Lewis [Thu, 12 May 2016 05:43:54 +0000 (05:43 +0000)]
Check for socket creation success before calling bind().

Reported by: Coverity
CID: 1194209

8 years agortadvd(8): Fix use-after-close in cm_handler_client
Conrad Meyer [Thu, 12 May 2016 05:12:24 +0000 (05:12 +0000)]
rtadvd(8): Fix use-after-close in cm_handler_client

cm_send() closes 'fd' on error.  In that case, bail out early without trying to
recv from or close 'fd' again.

Reported by: Coverity
CID: 1006078
Sponsored by: EMC / Isilon Storage Division

8 years agonfsd: Fix use-after-free in NFS4 lock test service
Conrad Meyer [Thu, 12 May 2016 05:03:12 +0000 (05:03 +0000)]
nfsd: Fix use-after-free in NFS4 lock test service

Trivial use-after-free where stp was freed too soon in the non-error path.
To fix, simply move its release to the end of the routine.

Reported by: Coverity
CID: 1006105
Sponsored by: EMC / Isilon Storage Division

8 years agortadvd(8): Don't use-after-free
Conrad Meyer [Thu, 12 May 2016 04:54:32 +0000 (04:54 +0000)]
rtadvd(8): Don't use-after-free

This whole block of code as committed fully formed in r224144.  I'm not really
sure what the intent was, but it seems plausible that !persist ifis could need
other member cleanup.  Don't free the object until after we've finished
cleaning its members.

Reported by: Coverity
CID: 1006079
Sponsored by: EMC / Isilon Storage Division

8 years agodhclient: Fix some trivial buffer overruns
Conrad Meyer [Thu, 12 May 2016 04:28:22 +0000 (04:28 +0000)]
dhclient: Fix some trivial buffer overruns

There was some confusion about how to limit a hardware address to at most 16
bytes.  In some cases it would overrun a byte off the end of the array.
Correct the types and rectify the overrun.

Reported by: Coverity
CIDs: 10086821305550
Sponsored by: EMC / Isilon Storage Division

8 years agoprint_positional_test: Fix misuse of wchar APIs
Conrad Meyer [Thu, 12 May 2016 04:08:45 +0000 (04:08 +0000)]
print_positional_test: Fix misuse of wchar APIs

These APIs take unit length, not byte length parameters.

Reported by: Coverity
CIDs: 133854313385441338545
Sponsored by: EMC / Isilon Storage Division

8 years agolibmp: Fix trivial buffer overrun
Conrad Meyer [Thu, 12 May 2016 03:53:20 +0000 (03:53 +0000)]
libmp: Fix trivial buffer overrun

fgetln yields a non-NUL-terminated buffer and its length.  This routine
attempted to NUL-terminate it, but did not allocate space for the NUL.  So,
allocate space for the NUL.

Reported by: Coverity
CID: 1017457
Sponsored by: EMC / Isilon Storage Division

8 years agorpcgen(1): Tag crash() routine as __dead2 for static analyzers
Conrad Meyer [Thu, 12 May 2016 03:49:05 +0000 (03:49 +0000)]
rpcgen(1): Tag crash() routine as __dead2 for static analyzers

Suggested by: Coverity
CID: 1305464
Sponsored by: EMC / Isilon Storage Division

8 years agokern_descrip_test: Fix trivial buffer overrun with readlink(2)
Conrad Meyer [Thu, 12 May 2016 03:44:29 +0000 (03:44 +0000)]
kern_descrip_test: Fix trivial buffer overrun with readlink(2)

Reported by: Coverity
CID: 12299651229972
Sponsored by: EMC / Isilon Storage Division

8 years agortadvd(8): Fix a typo in full msg receive logic
Conrad Meyer [Thu, 12 May 2016 03:37:17 +0000 (03:37 +0000)]
rtadvd(8): Fix a typo in full msg receive logic

Check against the size of the struct, not the pointer.  Previously, a message
with a cm_len between 9 and 23 (inclusive) could cause int msglen to underflow
and read(2) to be invoked with msglen size (implicitly cast to signed),
overrunning the caller-provided buffer.

All users of cm_recv() supply a stack buffer.

On the other hand, the rtadvd control socket appears to only be writable by the
owner, who is probably root.

While here, correct some types to be size_t or ssize_t.

Reported by: Coverity
CID: 1008477
Security: unix socket remotes may overflow stack in rtadvd
Sponsored by: EMC / Isilon Storage Division

8 years agomxge: Setup mbuf flowid before calling tcp_lro_rx().
Sepherosa Ziehau [Thu, 12 May 2016 03:36:49 +0000 (03:36 +0000)]
mxge: Setup mbuf flowid before calling tcp_lro_rx().

Reviewed by: gallatin
MFC after: 1 week
Sponsored by: Microsoft OSTC
Differential Revision: https://reviews.freebsd.org/D6320

8 years agohyperv/stor: Enable INQUIRY result check only on WIN10 like host systems
Sepherosa Ziehau [Thu, 12 May 2016 03:29:29 +0000 (03:29 +0000)]
hyperv/stor: Enable INQUIRY result check only on WIN10 like host systems

On WIN8 like host systems, when rescan happens, the already installed
disks seem to return random invalid results for INQUIRY.

More investigation is under way to figure out why random invalid INQUIRY
results are delivered to VM on WIN8 like host systems.

Submitted by: Hongjiang Zhang <honzhan microsoft com>
Reviewed by: sephe
MFC after: 1 week
Sponsored by: Microsoft OSTC
Differential Revision: https://reviews.freebsd.org/D6316

8 years agosnd_hda(4): Don't pass bogus sizeof()s to unused sysctl arg2 parameter (again)
Conrad Meyer [Thu, 12 May 2016 02:46:29 +0000 (02:46 +0000)]
snd_hda(4): Don't pass bogus sizeof()s to unused sysctl arg2 parameter (again)

More of the same sort of issue as r299503, just missed some sysctls added in a
different place than the others.

Reported by: Coverity
CIDs: 100769210096771009678
Sponsored by: EMC / Isilon Storage Division

8 years agosnd_hda(4): Don't pass bogus sizeof()s to unused sysctl arg2 parameter
Conrad Meyer [Thu, 12 May 2016 02:41:38 +0000 (02:41 +0000)]
snd_hda(4): Don't pass bogus sizeof()s to unused sysctl arg2 parameter

None of the sysctl handlers in hdaa use the arg2 parameter, so just pass zero
instead.  Additionally, the sizes being passed in were suspect (size of the
pointer rather than the value).

Reported by: Coverity
CIDs: 10076941009679
Sponsored by: EMC / Isilon Storage Division

8 years agonss/gethostby_test: fix broken vector iteration of gethostbyaddr h_aliases
Conrad Meyer [Thu, 12 May 2016 02:32:23 +0000 (02:32 +0000)]
nss/gethostby_test: fix broken vector iteration of gethostbyaddr h_aliases

h_aliases is a NULL-terminated rather than fixed-length array.  nitems() is not
a valid way to determine its end; instead, check for NULL.

Reported by: Coverity
CID: 1346578
Sponsored by: EMC / Isilon Storage Division

8 years agotraceroute6(8): use NULL instead of zero for initializing a pointer.
Pedro F. Giffuni [Thu, 12 May 2016 02:05:50 +0000 (02:05 +0000)]
traceroute6(8): use NULL instead of zero for initializing a pointer.

8 years agochat(8): use NULL instead of zero for initializing a pointer.
Pedro F. Giffuni [Thu, 12 May 2016 02:02:16 +0000 (02:02 +0000)]
chat(8): use NULL instead of zero for initializing a pointer.

8 years agosys/boot/common: use of spaces vs. TAB.
Pedro F. Giffuni [Thu, 12 May 2016 01:19:11 +0000 (01:19 +0000)]
sys/boot/common: use of spaces vs. TAB.

No functional change.

8 years agoatf map: Fix double-free in low memory error path
Conrad Meyer [Wed, 11 May 2016 23:39:39 +0000 (23:39 +0000)]
atf map: Fix double-free in low memory error path

If atf_list_append(, X, ) fails, X is freed.  Don't free it again.

If anyone wants to walk this patch upstream, be my guest.  I literally cannot
upstream it myself due to Google's stupid CLA.

Reported by: Coverity
CID: 979936
Sponsored by: EMC / Isilon Storage Division

8 years agolibkrb5: Fix potential double-free
Conrad Meyer [Wed, 11 May 2016 23:25:59 +0000 (23:25 +0000)]
libkrb5: Fix potential double-free

If krb5_make_principal fails, tmp_creds.server may remain a pointer to freed
memory and then be double-freed.  After freeing it the first time, initialize
it to NULL, which causes subsequent krb5_free_principal calls to do the right
thing.

Reported by: Coverity
CID: 1273430
Sponsored by: EMC / Isilon Storage Division

8 years agosubr_vmem: Fix double-free in error case of vmem_create
Conrad Meyer [Wed, 11 May 2016 23:16:11 +0000 (23:16 +0000)]
subr_vmem: Fix double-free in error case of vmem_create

If vmem_init() fails, 'vm' is already destroyed and freed.  Don't free it
again.

Reported by: Coverity
CID: 1042110
Sponsored by: EMC / Isilon Storage Division

8 years agoRevert r299467 to fix the kernel build.
Conrad Meyer [Wed, 11 May 2016 23:00:12 +0000 (23:00 +0000)]
Revert r299467 to fix the kernel build.

$ svn merge -c -299467 .

Approved by: build being broken for six hours

8 years agoDocument r298695, ntp updated to 4.2.8p7.
Steven Kreuzer [Wed, 11 May 2016 22:44:00 +0000 (22:44 +0000)]
Document r298695, ntp updated to 4.2.8p7.

Approved by: gjb@ (implicit with re@ hat on)

8 years agoroute6d(8): Fix potential double-free
Conrad Meyer [Wed, 11 May 2016 22:33:20 +0000 (22:33 +0000)]
route6d(8): Fix potential double-free

In the case that the subsequent sysctl(3) call failed, 'buf' could be free(3)ed
repeatedly.  It isn't clear to me that that case is possible, but be clear and
do the right thing in case it is.

Reported by: Coverity
CID: 272537
Sponsored by: EMC / Isilon Storage Division

8 years agocamcontrol(8): Fix another trivial double-free
Conrad Meyer [Wed, 11 May 2016 22:25:14 +0000 (22:25 +0000)]
camcontrol(8): Fix another trivial double-free

Reported by: Coverity
CID: 1331222
Sponsored by: EMC / Isilon Storage Division

8 years agocamcontrol(8): Fix trival double-free
Conrad Meyer [Wed, 11 May 2016 22:22:49 +0000 (22:22 +0000)]
camcontrol(8): Fix trival double-free

Reported by: Coverity
CID: 1331223
Sponsored by: EMC / Isilon Storage Division

8 years agorandom(6): Fix double-close
Conrad Meyer [Wed, 11 May 2016 22:04:28 +0000 (22:04 +0000)]
random(6): Fix double-close

In the case where a file lacks a trailing newline, there is some "evil" code to
reverse goto the tokenizing code ("make_token") for the final token in the
file.  In this case, 'fd' is closed more than once.  Use a negative sentinel
value to guard close(2), preventing the double close.

Ideally, this code would be restructured to avoid this ugly construction.

Reported by: Coverity
CID: 1006123
Sponsored by: EMC / Isilon Storage Division

8 years agoexec.h: Move PS_STRINGS define to kernel-only section
Ed Maste [Wed, 11 May 2016 21:14:36 +0000 (21:14 +0000)]
exec.h: Move PS_STRINGS define to kernel-only section

The kern.ps_strings sysctl was introduced in r103767 and the last
use of PS_STRINGS in userspace code was removed in r297888.

PR: 208760 [exp-run]
Reviewed by: kib
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D5933

8 years agoRegen x86 assembly files for r299480.
Jung-uk Kim [Wed, 11 May 2016 20:11:21 +0000 (20:11 +0000)]
Regen x86 assembly files for r299480.

8 years agoSet CC environment variable for Perl scripts. This is for detecting
Jung-uk Kim [Wed, 11 May 2016 20:06:23 +0000 (20:06 +0000)]
Set CC environment variable for Perl scripts.  This is for detecting
assembler/compiler capabilities, e.g., AVX instructions.

8 years agoRefine comments to add its origin.
Jung-uk Kim [Wed, 11 May 2016 19:59:05 +0000 (19:59 +0000)]
Refine comments to add its origin.

8 years agoCall busdma_swi from swi_vm as is done from other architectures.
Andrew Turner [Wed, 11 May 2016 18:48:47 +0000 (18:48 +0000)]
Call busdma_swi from swi_vm as is done from other architectures.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

8 years agoAdd OF_prop_free function as a counterpart for OF_*prop_alloc
Oleksandr Tymoshenko [Wed, 11 May 2016 18:20:02 +0000 (18:20 +0000)]
Add OF_prop_free function as a counterpart for OF_*prop_alloc

- Introduce new OF API function OF_prop_free to free memory allocated by
  OF_getprop_alloc and OF_getencprop_alloc. Current code just calls free(9)
  with M_OFWPROP memory class which assumes knowledge about OF_*prop_alloc
  functions' internals and leads to unneccessary code coupling

- Convert some of the free(..., M_OFWPROP) instances to OF_prop_free

Files affected by this commit are the ones I was able to test on real
hardware. The rest of free(..., M_OFWPROP) instances will be handled with
idividual maintainers

Reviewed by: andrew
Differential Revision: https://reviews.freebsd.org/D6315

8 years agowhois(1): Fix potential double-close and logic mistakes
Conrad Meyer [Wed, 11 May 2016 18:03:51 +0000 (18:03 +0000)]
whois(1): Fix potential double-close and logic mistakes

Close the fd the poll error was detected on, rather than the last opened fd, to
fix the double-close.

Use -1 to make it explict which int variables no longer own socket file
descriptors.

Actually shrink, rather than grow, the poll timeout to match comment.

Reported by: Coverity
CID: 13048601305616
Sponsored by: EMC / Isilon Storage Division

8 years agoAdd gpiokeys driver
Oleksandr Tymoshenko [Wed, 11 May 2016 17:57:26 +0000 (17:57 +0000)]
Add gpiokeys driver

gpiokey driver implements functional subset of gpiokeys device-tree bindings:
https://www.kernel.org/doc/Documentation/devicetree/bindings/input/gpio-keys.txt

It acts as a virtual keyboard, so keys are visible through kbdmux(4)

Driver maps linux scancodes for most common keys to FreeBSD scancodes and
also extends spec by introducing freebsd,code property to specify
FreeBSD-native scancodes.

Reviewed by: mmel, jmcneill
Differential Revision: https://reviews.freebsd.org/D6279

8 years agoDeorbit ALLOW_SHARED_TEXTREL
Ed Maste [Wed, 11 May 2016 17:55:09 +0000 (17:55 +0000)]
Deorbit ALLOW_SHARED_TEXTREL

We want to avoid .text relocations in shared objects. libcrypto was the
only consumer and it is now fixed (as of r299389). Remove the now-unused
support for turning off the linker warning.

Reviewed by: kib
Differential Revision: https://reviews.freebsd.org/D6323

8 years agowhois(1): Pull out async multiple host connection code into a routine
Conrad Meyer [Wed, 11 May 2016 17:52:06 +0000 (17:52 +0000)]
whois(1): Pull out async multiple host connection code into a routine

This logic was added to the whois() function in r281959, but could easily be
its own routine.  In this case, I think the abstraction makes both functions
easier to reason about.

This precedes some Coverity-suggested cleanup.

Sponsored by: EMC / Isilon Storage Division

8 years agoDIRDEPS_BUILD: Exclude host tools for Makefile.depend.host as well.
Bryan Drewery [Wed, 11 May 2016 17:40:51 +0000 (17:40 +0000)]
DIRDEPS_BUILD: Exclude host tools for Makefile.depend.host as well.

Sponsored by: EMC / Isilon Storage Division

8 years agoResolve LINT linking issue by renaming ida_init() to ida_setup(). The
Hans Petter Selasky [Wed, 11 May 2016 17:38:09 +0000 (17:38 +0000)]
Resolve LINT linking issue by renaming ida_init() to ida_setup(). The
ida_init() symbol name is now taken for use by the LinuxKPI.

Reported by: emaste @
Discussed with: mav @

8 years agomixer(8): Style: Tag no-return usage() as __dead2
Conrad Meyer [Wed, 11 May 2016 17:27:27 +0000 (17:27 +0000)]
mixer(8): Style: Tag no-return usage() as __dead2

Coverity really should have figured this out from the exit(3) call at the end
of the routine, but just make it explicit.

No functional change.

Reported by: Coverity
CID: 1304866 (false positive double-close of 'baz')
Sponsored by: EMC / Isilon Storage Division

8 years agoMatch Linux behaviour and iterate the IDR tree unlocked. The caller is
Hans Petter Selasky [Wed, 11 May 2016 17:20:20 +0000 (17:20 +0000)]
Match Linux behaviour and iterate the IDR tree unlocked. The caller is
responsible the IDR tree stays unmodified while iterating.

MFC after: 1 week
Sponsored by: Mellanox Technologies

8 years agoThe idr_for_each() function is now part of the LinuxKPI. Use the
Hans Petter Selasky [Wed, 11 May 2016 17:17:48 +0000 (17:17 +0000)]
The idr_for_each() function is now part of the LinuxKPI. Use the
LinuxKPI's idr_for_each() function instead of the local one to avoid
compilation issues.

Discussed with: np @
MFC after: 1 week

8 years agoAdd a new get_id interface to pci and pcib. This will allow us to both
Andrew Turner [Wed, 11 May 2016 17:07:29 +0000 (17:07 +0000)]
Add a new get_id interface to pci and pcib. This will allow us to both
detect failures, and get different PCI IDs.

For the former the interface returns an int to signal an error. The ID is
returned at a uintptr_t * argument.

For the latter there is a type argument that allows selecting the ID type.
This only specifies a single type, however a MSI type will be added
to handle the need to find the ID the hardware passes to the ARM GICv3
interrupt controller.

A follow up commit will be made to remove pci_get_rid.

Reviewed by: jhb, rstone
Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D6239

8 years agobsnmpd: Fix size of trapsink::comm to match other community arrays
Conrad Meyer [Wed, 11 May 2016 17:06:03 +0000 (17:06 +0000)]
bsnmpd: Fix size of trapsink::comm to match other community arrays

This fixes a number of possible strcpy() buffer overruns between the various
community strings in trap.c.

Reported by: Coverity
CIDs: 100682010068211006822
Sponsored by: EMC / Isilon Storage Division

8 years agobsnmp: Don't overrun privkey buffer by copying wrong size
Conrad Meyer [Wed, 11 May 2016 16:54:34 +0000 (16:54 +0000)]
bsnmp: Don't overrun privkey buffer by copying wrong size

The 'priv_key' array is SNMP_PRIV_KEY_SIZ bytes, not SNMP_AUTH_KEY_SIZ.

Reported by: Coverity
CIDs: 10083261009675
Sponsored by: EMC / Isilon Storage Division

8 years agolibcrypto: add "Do not modify" comment to generated source files
Ed Maste [Wed, 11 May 2016 16:53:56 +0000 (16:53 +0000)]
libcrypto: add "Do not modify" comment to generated source files

Reviewed by: jkim
Differential Revision: https://reviews.freebsd.org/D6237

8 years agoOn arm64 always create a bus_dmamap_t object. This will be use to hold the
Andrew Turner [Wed, 11 May 2016 16:53:41 +0000 (16:53 +0000)]
On arm64 always create a bus_dmamap_t object. This will be use to hold the
list of memory that the kernel will need to sync when operating with a
non-cache coherent DMA engine.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

8 years agoEnable linker error if libcrypto.so contains a relocation against text. It
Jung-uk Kim [Wed, 11 May 2016 16:45:58 +0000 (16:45 +0000)]
Enable linker error if libcrypto.so contains a relocation against text.  It
is position independent on all platforms since r299389.

Submitted by: kib

8 years agoffs_bswap: Copy one UFS dinode member at a time
Conrad Meyer [Wed, 11 May 2016 16:42:13 +0000 (16:42 +0000)]
ffs_bswap: Copy one UFS dinode member at a time

No functional change.

Reported by: Coverity
CIDs: 974635, 974636, 977396, 977397, 977398, 977399
Sponsored by: EMC / Isilon Storage Division

8 years agofsck_ffs: Don't overrun mount device buffer
Conrad Meyer [Wed, 11 May 2016 16:20:23 +0000 (16:20 +0000)]
fsck_ffs: Don't overrun mount device buffer

Maybe this case is impossible.  Either way, when attempting to "/dev/"-prefix a
non-global device name, check that we do not overrun the f_mntfromname buffer.

In this case, truncating (with strlcpy or similar) would not be useful, since
the f_mntfromname result of getmntpt() is passed directly to open(2) later.

Reported by: Coverity
CID: 1006789
Sponsored by: EMC / Isilon Storage Division

8 years agocompat/opensolaris: Don't redefined off64_t if already defined
Conrad Meyer [Wed, 11 May 2016 16:05:32 +0000 (16:05 +0000)]
compat/opensolaris: Don't redefined off64_t if already defined

A follow-up to r299456.

Reported by: gjb
Sponsored by: EMC / Isilon Storage Division

8 years agoFix buffer overrun in gcore(1) NT_PRPSINFO
Conrad Meyer [Wed, 11 May 2016 15:31:31 +0000 (15:31 +0000)]
Fix buffer overrun in gcore(1) NT_PRPSINFO

Use size of destination buffer, rather than a constant that may or may not
correspond to the source buffer, to restrict the length of copied strings.  In
particular, pr_fname has 16+1 characters but MAXCOMLEN is 18+1.

Use strlcpy instead of strncpy to ensure the result is nul-terminated.  This
seems to be what is expected of these fields.

Reported by: Coverity
CIDs: 10113021011378
Sponsored by: EMC / Isilon Storage Division

8 years agoAdd data barriers to the arm64 bus_dmamap_sync function. We need these
Andrew Turner [Wed, 11 May 2016 14:59:54 +0000 (14:59 +0000)]
Add data barriers to the arm64 bus_dmamap_sync function. We need these
to ensure ordering between the CPU and device. As the CPU and DMA target
may be in different shareability domains they need to be full system
barriers.

Obtained from: ABT Systems Ltd
Sponsored by: The FreeBSD Foundation

8 years agolibc: Add fopencookie(3) wrapper around funopen(3)
Conrad Meyer [Wed, 11 May 2016 14:38:27 +0000 (14:38 +0000)]
libc: Add fopencookie(3) wrapper around funopen(3)

Reviewed by: jhb, oshogbo
Sponsored by: EMC / Isilon Storage Division
Differential Revision: https://reviews.freebsd.org/D6282

8 years agolibrpcsec_gss: remove redundant code.
Pedro F. Giffuni [Wed, 11 May 2016 14:37:33 +0000 (14:37 +0000)]
librpcsec_gss: remove redundant code.

We have identical code no matter the expression behind the if.
Avoid the desision altogether and keep doing what is expected.

Reviewed by: dfr
CID: 1305689

8 years agoMFV r299453: 6765 zfs_zaccess_delete() comments do not accurately reflect
Alexander Motin [Wed, 11 May 2016 13:53:29 +0000 (13:53 +0000)]
MFV r299453: 6765 zfs_zaccess_delete() comments do not accurately reflect
delete permissions for ACLs

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Kevin Crowe <kevin.crowe@nexenta.com>

openzfs/openzfs@a40149b935cbbe87bf95e2cc44b3bc99d400513a

8 years ago6765 zfs_zaccess_delete() comments do not accurately reflect
Alexander Motin [Wed, 11 May 2016 13:51:53 +0000 (13:51 +0000)]
6765 zfs_zaccess_delete() comments do not accurately reflect
delete permissions for ACLs

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Kevin Crowe <kevin.crowe@nexenta.com>

openzfs/openzfs@a40149b935cbbe87bf95e2cc44b3bc99d400513a

8 years agoMFV r299451: 6764 zfs issues with inheritance flags during chmod(2) with
Alexander Motin [Wed, 11 May 2016 13:50:34 +0000 (13:50 +0000)]
MFV r299451: 6764 zfs issues with inheritance flags during chmod(2) with
aclmode=passthrough

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Albert Lee <trisk@nexenta.com>

openzfs/openzfs@1bcf0d240bdebed61b4261f7c0ee323e07c8dfac

8 years ago6764 zfs issues with inheritance flags during chmod(2) with
Alexander Motin [Wed, 11 May 2016 13:49:50 +0000 (13:49 +0000)]
6764 zfs issues with inheritance flags during chmod(2) with
aclmode=passthrough

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Albert Lee <trisk@nexenta.com>

openzfs/openzfs@1bcf0d240bdebed61b4261f7c0ee323e07c8dfac

8 years agoMFV r299449: 6763 aclinherit=restricted masks inherited permissions by group
Alexander Motin [Wed, 11 May 2016 13:48:15 +0000 (13:48 +0000)]
MFV r299449: 6763 aclinherit=restricted masks inherited permissions by group
perms (groupmask)

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Albert Lee <trisk@nexenta.com>

openzfs/openzfs@eebb483d0cd68bdc4cf03c01fdeba9af160c17af

8 years ago6763 aclinherit=restricted masks inherited permissions by group
Alexander Motin [Wed, 11 May 2016 13:46:44 +0000 (13:46 +0000)]
6763 aclinherit=restricted masks inherited permissions by group
perms (groupmask)

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Albert Lee <trisk@nexenta.com>

openzfs/openzfs@eebb483d0cd68bdc4cf03c01fdeba9af160c17af

8 years agoMFV r299442: 6762 POSIX write should imply DELETE_CHILD on directories - and
Alexander Motin [Wed, 11 May 2016 13:43:20 +0000 (13:43 +0000)]
MFV r299442: 6762 POSIX write should imply DELETE_CHILD on directories - and
some additional considerations

Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Author: Kevin Crowe <kevin.crowe@nexenta.com>

openzfs/openzfs@d316fffc9c361532a482208561bbb614dac7f916

8 years agoAdd support for MTU chaning and Jumbo frames to VNIC
Zbigniew Bodek [Wed, 11 May 2016 13:42:20 +0000 (13:42 +0000)]
Add support for MTU chaning and Jumbo frames to VNIC

Enable previously added code for MTU handling (based on
Cavium 1.0 driver released on BSD license).
This commit enables possibility to change MTU on VNIC driver.

Obtained from: Semihalf
Sponsored by:  Cavium

8 years agoFix deadlock in VNIC when using single CPU only
Zbigniew Bodek [Wed, 11 May 2016 13:38:29 +0000 (13:38 +0000)]
Fix deadlock in VNIC when using single CPU only

Number of free Tx descriptors does not need to be locked since
it can be modified atomically between SND and CQ tasks.
It will also block Tx routine from sending packets while CQ will not
be able to free descriptors.

Obtained from: Semihalf
Sponsored by: Cavium
Differential Revision: https://reviews.freebsd.org/D6266