Feature #8330

Add svc_tp_create_addr to libnsl

Added by Gordon Ross 6 months ago. Updated 4 months ago.

Status:ClosedStart date:2017-06-05
Priority:NormalDue date:
Assignee:Gordon Ross% Done:

100%

Category:-
Target version:-
Difficulty:Medium Tags:needs-triage

Description

In support of these issues:
#7577 mountd support to run on a fixed port
#7569 statd support to run on a fixed port


Related issues

Blocks illumos gate - Bug #7577: mountd support to run on a fixed port Closed 2016-11-11
Blocks illumos gate - Bug #7569: statd support to run on a fixed port Closed 2016-11-11

History

#2 Updated by Gordon Ross 4 months ago

  • Blocks Bug #7577: mountd support to run on a fixed port added

#3 Updated by Gordon Ross 4 months ago

  • Blocks Bug #7569: statd support to run on a fixed port added

#4 Updated by Gordon Ross 4 months ago

  • Status changed from New to In Progress

#5 Updated by Marcel Telka 4 months ago

Why do we need to add a new function to libnsl(3nsl) that would save us two lines of code only?

Detailed analysis is at https://illumos.topicbox.com/groups/developer/discussions/Tced6bc9c4704e405-M2c3f66fa76b3e00d89f18c78

#6 Updated by Gordon Ross 4 months ago

The work submitted for RTI is a compromize between "no changes in libnsl" and "substantial changes in libnsl".
Where to settle between those bounds is a matter of judgement, and not worth arguing about.
I'm going ahead without Marcel's review.

#7 Updated by Electric Monk 4 months ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Closed

git commit dec98d2aa2912b1fbd0abd2574307a695b566692

commit  dec98d2aa2912b1fbd0abd2574307a695b566692
Author: Gordon Ross <gwr@nexenta.com>
Date:   2017-07-21T01:54:31.000Z

    8330 Add svc_tp_create_addr to libnsl
    Reviewed by: Paul Dagnelie <pcd@delphix.com>
    Reviewed by: Evan Layton <evan.layton@nexenta.com>
    Reviewed by: Sebastien Roy <sebastien.roy@delphix.com>
    Approved by: Prakash Surya <prakash.surya@delphix.com>

Also available in: Atom