[nfs-discuss] webrev for CR 6368645

2009-02-06 Thread Jeff A. Smith
Hi Dai,

Changes look good.

Jeff

On Feb 6, 2009, at 12:35 AM, Dai Ngo wrote:

> Hi,
>
> I need a code review for the fix of CR 6368645.
> The webrev is here:  http://cr.opensolaris.org/~dain/6368645/
> 
>
> The root cause of the problem is the failure to cleanup the client
> handle and its endpoint, used for lock requests, when the zone is
> rebooted and lockd is not running.
>
> This fix addresses the specific system assert condition raised by this
> CR. The larger issue of proper cleanup of client handle used by lock
> manager in zoning environment will be addressed in a separate CR.
>
> Thanks,
> -Dai
> ___
> nfs-discuss mailing list
> nfs-discuss at opensolaris.org




[nfs-discuss] webrev for CR 6368645

2009-02-06 Thread Mahesh Siddheshwar

Dai Ngo wrote:
> Hi,
>
> I need a code review for the fix of CR 6368645.
> The webrev is here:  http://cr.opensolaris.org/~dain/6368645/ 
> 
>
>   
Dai, the changes look good.

Mahesh
> The root cause of the problem is the failure to cleanup the client
> handle and its endpoint, used for lock requests, when the zone is
> rebooted and lockd is not running.
>
> This fix addresses the specific system assert condition raised by this
> CR. The larger issue of proper cleanup of client handle used by lock
> manager in zoning environment will be addressed in a separate CR.
>
> Thanks,
> -Dai
> ___
> nfs-discuss mailing list
> nfs-discuss at opensolaris.org
>   




[nfs-discuss] webrev for CR 6368645

2009-02-05 Thread Dai Ngo
Hi,

I need a code review for the fix of CR 6368645.
The webrev is here:  http://cr.opensolaris.org/~dain/6368645/ 


The root cause of the problem is the failure to cleanup the client
handle and its endpoint, used for lock requests, when the zone is
rebooted and lockd is not running.

This fix addresses the specific system assert condition raised by this
CR. The larger issue of proper cleanup of client handle used by lock
manager in zoning environment will be addressed in a separate CR.

Thanks,
-Dai