http://defect.opensolaris.org/bz/show_bug.cgi?id=10832


amaguire <alan.maguire at sun.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |CAUSEKNOWN


--- Comment #4 from amaguire <alan.maguire at sun.com> 2009-08-26 15:25:35 UTC 
---
(In reply to comment #3)
> (In reply to comment #2)
> > > 
> > > hmm, I can't reproduce this one:
> > > 
> > > # nwamadm disable -p ncu -c ip iwk0
> > > Disabling ncu 'iwk0'
> > > #
> > > 
> > > It should always be possible to disable IP NCUs in the User NCP. I tried 
> > > the
> > > above action where the underlying link NCU was both prioritized and 
> > > manual. Not
> > > sure why you're seeing this...
> > 
> > Should I set the underlying link NCU to 'manual' first ? The case didn't do
> > that, while it passed before.
> 
> You shouldn't have to (I tried both prioritized and manual underlying link 
> NCUs
> and both worked). I'm totally stumped by this one to be honest - I can't see
> why it would fail like it does. I could see that a prioritized link NCU would
> fail with that message, or that it would fail if you just did
> 
> # nwamadm disable -p ncu iwk0
> 
> ...since leaving out the class means that nwamadm assumes both the link and ip
> ncus are to be disabled, but I can't figure out why this should fail at all.


I've made some progress here - I've managed to reproduce this issue on a sparc
test machine. It seems that the problem is nwamadm/libnwam is interpreting the
request to disable the IP NCU as a request to disable the underlying phys NCU.
I've confirmed that if we convert the underlying phys NCU to manual rather than
prioritized (in Lizhong's case it was prioritized since the User NCP was
constructed from the Automatic NCP template) we can indeed disable the IP NCU.
This confirms the suspicion that we're misinterpreting the request to disable
an IP NCU as a request to disable the phys NCU. The problem is we're casting
an enumerated type into an uint64_t in parse_argv(). We should simply make the
variables "ncu" and "class" into uint64_ts. I've verified doing this makes this
problem go away, and things work on x86 and sparc.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.

Reply via email to