7734 uts: update properties with correct type

Review Request #299 — Created Dec. 19, 2016 and submitted

tsoome
illumos-gate
7734
307
ad58e2f...
general

7734 uts: update properties with correct type

prtconf -v and visual verification.

tsoome
yuripv
  1. Ship It!
  2. 
      
andy_js
  1. Ship It!
  2. 
      
rm
  1. Ship It!
    1. Actually, I should have looked a little closer. Are there other consumers of these properties other than those in bootfs? Are progarms that consume these properties going to break because the lookup of the type doesn't work?

    2. the boot-args/bootargs is string, so no actual problem with it. the bootp-response is already treated as byte array http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/io/strplumb.c#557 so it is also ok.

      the module properties need fix of bootfs, and this patch and bootfs patch shall be squashed, so we get no chance for getting out of sync.

      ramdisk_start and ramdisk_end are read only from BOP propery area in:

      http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/i86pc/os/ddi_impl.c#2500

      So I do not think anything else is affected than the bootfs code.

  2. 
      
tsoome
tsoome
andy_js
  1. Ship It!
  2. 
      
tsoome
Review request changed

Status: Closed (submitted)

Loading...