11282 port ZoL send/recv fixes

Review Request #2190 — Created July 17, 2019 and submitted

jjelinek
illumos-gate
general

This is a port from ZoL of several send/recv commits which are listed in the bug report.

ZFS test run, including several additional tests which previously did not work on illumos.

  • 0
  • 0
  • 0
  • 1
  • 1
Description From Last Updated
kkantor
  1. I think this looks good. I just have the one question about a possible 11309 regression. Do you think you could add the new tests to the smartos runfile as well?

  2. It looks like this came in from ZoL, so we shouldn't change it, but we do have the fnvlist_ set of functions that pretty much do exactly what the VERIFY() stuff is doing here and below.

  3. Would this be a regression for 11309, or is this handled below (where you note the diff between ZoL and illumos)?

    1. Yes, the change you noted will handle this and the specific test I added for this still passes with this code.

  4. 
      
jjelinek
kkantor
  1. Ship It!
  2. 
      
jjelinek
Review request changed

Status: Closed (submitted)

Loading...