Microsoft MVP성태의 닷넷 이야기
글쓴 사람
정성태 (techsharer at outlook.com)
홈페이지
첨부 파일

C# - lock (this), lock (typeof(...))를 사용하면 안 되는 이유

마침 질문도 있고,

스레드 lock 키워드 관련 질문드립니다
; https://www.sysnet.pe.kr/3/0/5341

한 번도 관련 글을 직접적으로 소개한 적이 없어서 이참에 다뤄보겠습니다. ^^ 사실 이번 글은 이미 다음의 자료에서 잘 설명하고 있습니다.

Reliability Best Practices
; https://docs.microsoft.com/en-us/dotnet/framework/performance/reliability-best-practices




우선 질문과 관계된 항목을 먼저 풀어보겠습니다.

A note about lock(this)
; https://docs.microsoft.com/en-us/dotnet/framework/performance/reliability-best-practices#a-note-about-lockthis

예를 들어 다음과 같은 코드가 있다고 했을 때,

// 첨부 파일의 ConsoleApp1 프로젝트
class MyTest
{
    public void Do()
    {
        lock (this)
        {
            // ...
        }
    }
}

여기서 사용한 this는 그 객체 자신의 참조를 가리킨다는 점이 문제가 되는데, 왜냐하면 해당 타입의 인스턴스를 생성했을 때 그 인스턴스로 외부에서 lock을 걸 수 있기 때문입니다.

MyTest tinst = new MyTest();
lock (tinst) // 내부의 lock (this)와 동일한 잠금 획득 시도
{
    // ...
}

이를 바탕으로 간단하게 재현 코드를 만들어 볼까요?

class Program
{
    static void Main(string[] args)
    {
        MyTest inst = new MyTest();

        Thread t = new Thread((objInst) =>
        {
            inst.Do();
        });

        t.Start(inst);

        Thread.Sleep(1000);

        lock (inst)
        {
            Console.WriteLine("This code");
        }
    }
}

class MyTest
{
    public void Do()
    {
        lock (this)
        {
            string txt = Console.ReadLine();
            Console.WriteLine(txt);
        }
    }
}

위의 코드는 (MyTest 타입이 3rd-party 라이브러리에 정의되었다고 가정하고) 개발자가 왜 "This code" 문자열이 화면에 출력되지 않는지 이해할 수 없게 만들어 버립니다. (만약 초보 개발자라면 정말 신기한 현상이라며 어쩌면 닷넷 프레임워크의 버그라고까지 생각할 수도 있을 것입니다.)

결국, 내부의 잠금에 관계된 lock 인스턴스를 외부에 제공한 것이 문제가 된 것으로, 마치 다음과 같은 식으로 프로그램한 것과 동일한 효력을 가집니다.

class MyTest
{
    public object LockInstance = new object();

    public void Do()
    {
        lock (LockInstance)
        {
            // ...
        }
    }
}

아마 lock (this)라고 코딩했을 때 Visual Studio 편집기가 자동으로 저렇게 public object로 확장해 코드를 변경해 준다면, 누구도 public 접근자를 그대로 두는 것을 원치 않았을 것입니다.




위의 lock (this) 문제점을 이해했으면 이제 다음의 글도 동일한 맥락으로 파악할 수 있습니다.

Avoid lock(typeof(MyType))
; https://docs.microsoft.com/en-us/dotnet/framework/performance/reliability-best-practices#avoid-locktypeofmytype

lock (this)가 잠금 객체를 public하게 접근할 수 있게 만들었던 것이 문제였던 것처럼, lock (typeof(...)) 역시 해당 타입을 접근할 수 있다면 모든 곳에서 사용할 수 있기 때문에 동일한 문제가 발생합니다. 이외에도 lock (typeof(...))에는 한 가지 더 중요한 문제점이 있습니다.

Private and public Type objects in shared assemblies with only one copy of the code shared across all application domains also present problems.


"shared assemblies"는 "SharedDomain"에 로드되는 어셈블리들로 다음의 글에 설명한,

.NET - 눈으로 확인하는 SharedDomain의 동작 방식
; https://www.sysnet.pe.kr/2/0/10948

(AppDomain의 동작 방식에 따라 다르지만,) 기본 값인 SingleDomain의 경우에는 mscorlib.dll만 shared이므로 그 안에 있는 타입은 AppDomain 전체에 걸쳐 lock 잠금이 발생하게 됩니다. 예를 들어 아래는 재현 코드입니다.

// 첨부 파일의 ConsoleApp2 프로젝트

class Program
{
    static void Main(string[] args)
    {
        Thread t = new Thread(() =>
        {
            AppDomain appDomain = AppDomain.CreateDomain("testDomain");
            appDomain.DoCallBack(new CrossAppDomainDelegate(() =>
            {
                lock (typeof(Console))
                {
                    string txt = Console.ReadLine();
                    Console.WriteLine(txt);
                }
            }));
        });

        t.Start();
        Thread.Sleep(1000);

        lock (typeof(Console))
        {
            Console.WriteLine("This code");
        }

    }
}

(shared assembly인) mscorlib.dll의 Console 타입에 대해 lock을 걸었으므로 상이한 AppDomain에서 lock을 걸었는데도 "This code" 텍스트가 출력되지 않는 것을 확인할 수 있습니다. 반면, 위의 "lock (typeof(Console))"을 "lock (typeof(Program))"으로 변경하면 이번에는 "This code" 출력이 발생합니다. 왜냐하면, SingleDomain의 경우 위의 코드를 구현한 어셈블리는 공유되지 않으므로 typeof(Program)이 해당 AppDomain 내에서 고유한 인스턴스로 나오기 때문입니다.

당연히 이런 상황은 사용하는 측에 의존하게 됩니다. 만약 Main 메서드에 다음과 같이 MultiDomain 속성을 주면,

class Program
{
    [LoaderOptimization(LoaderOptimization.MultiDomain)]
    static void Main(string[] args)
    {
        // ...
    }
}

Program 타입을 정의한 어셈블리도 shared가 되므로, 이번에는 "lock (typeof(Program))"에서도 "This code"가 출력되지 않습니다. 그러니까, 아무리 의도해서 typeof(...)를 사용했다고 해도 그것을 사용하는 측에서 AppDomain을 어떤 식으로 운영하느냐에 따라 다른 결과를 가져오므로 typeof(...)를 lock에 사용하는 것은 피해야 합니다.

참고로, AppDomain은 .NET Core 환경에서는 개발자가 임의로 만들 수 없기 때문에 저런 식의 문제는 이제 더 이상 볼 일이 없을 것입니다. (적어도 .NET 5.0 Preview 4까지는 AppDomain.CreateDomain을 지원하지 않습니다.)




프로세스 전체에 걸쳐서 유일한 인스턴스로 다뤄지는 것은 shared assemblies에서만 발생하는 것은 아닙니다. 가령 다음과 같이 문자열을 직접적으로 사용하는 경우에도, 프로세스에 고유하게 interning시킨 문자열 풀의 인스턴스가 사용되기 때문에,

// 첨부 파일의 ConsoleApp3 프로젝트
static void Main(string[] args)
{
    Thread t = new Thread(() =>
    {
        AppDomain appDomain = AppDomain.CreateDomain("testDomain");

        appDomain.DoCallBack(new CrossAppDomainDelegate(() =>
        {
            lock ("mytest")
            {
                string txt = Console.ReadLine();
                Console.WriteLine(txt);
            }
        }));
    });

    t.Start();
    Thread.Sleep(1000);

    lock ("mytest")
    {
        Console.WriteLine("This code");
    }
}

"lock (typeof(Console))"과 동일한 문제가 발생합니다. 사실 저런 식으로 lock을 쓰는 개발자는 거의 없을 것이므로 문제가 될 여지가 없겠지만 그냥 이해 차원에서 봐두면 되겠습니다.




[이 글에 대해서 여러분들과 의견을 공유하고 싶습니다. 틀리거나 미흡한 부분 또는 의문 사항이 있으시면 언제든 댓글 남겨주십시오.]



donaricano-btn



[최초 등록일: ]
[최종 수정일: 6/1/2020 ]

Creative Commons License
이 저작물은 크리에이티브 커먼즈 코리아 저작자표시-비영리-변경금지 2.0 대한민국 라이센스에 따라 이용하실 수 있습니다.
by SeongTae Jeong, mailto:techsharer at outlook.com

비밀번호

댓글 쓴 사람
 




1  [2]  3  4  5  6  7  8  9  10  11  12  13  14  15  ...
NoWriterDateCnt.TitleFile(s)
12609정성태4/22/2021138.NET Framework: 1045. C# - 런타임 시점에 이벤트 핸들러를 만들어 Reflection을 이용해 구독하는 방법파일 다운로드1
12608정성태4/22/2021337.NET Framework: 1044. C# - Generic Host를 이용해 .NET 5로 리눅스 daemon 프로그램 만드는 방법 [6]파일 다운로드1
12607정성태4/21/2021141.NET Framework: 1043. C# - 실행 시점에 동적으로 Delegate 타입을 만드는 방법파일 다운로드1
12606정성태4/21/2021215.NET Framework: 1042. C# - enum 값을 int로 암시적(implicit) 형변환하는 방법? [2]파일 다운로드1
12605정성태4/18/2021118.NET Framework: 1041. C# - AssemblyID, ModuleID를 관리 코드에서 구하는 방법파일 다운로드1
12604정성태4/19/2021133VS.NET IDE: 163. 비주얼 스튜디오 속성 창의 "Build(빌드)" / "Configuration(구성)"에서의 "활성" 의미
12603정성태4/16/2021184VS.NET IDE: 162. 비주얼 스튜디오 - 상속받은 컨트롤이 디자인 창에서 지원되지 않는 문제
12602정성태4/19/2021117VS.NET IDE: 161. x64 DLL 프로젝트의 컨트롤이 Visual Studio의 Designer에서 보이지 않는 문제
12601정성태4/15/2021129.NET Framework: 1040. C# - REST API 대신 github 클라이언트 라이브러리를 통해 프로그래밍으로 접근
12600정성태4/15/2021151.NET Framework: 1039. C# - Kubeconfig의 token 설정 및 인증서 구성을 자동화하는 프로그램
12599정성태4/14/2021153.NET Framework: 1038. C# - 인증서 및 키 파일로부터 pfx/p12 파일을 생성하는 방법파일 다운로드1
12598정성태4/14/2021105.NET Framework: 1037. openssl의 PEM 개인키 파일을 .NET RSACryptoServiceProvider에서 사용하는 방법 (2)파일 다운로드1
12597정성태4/14/2021130개발 환경 구성: 569. csproj의 내용을 공통 설정할 수 있는 Directory.Build.targets / Directory.Build.props 파일
12596정성태4/12/2021200개발 환경 구성: 568. Windows의 80 포트 점유를 해제하는 방법
12595정성태4/12/2021192.NET Framework: 1036. SQL 서버 - varbinary 타입에 대한 문자열의 CAST, CONVERT 변환을 C# 코드로 구현
12594정성태4/15/2021257.NET Framework: 1035. C# - kubectl 명령어 또는 REST API 대신 Kubernetes 클라이언트 라이브러리를 통해 프로그래밍으로 접근 [1]파일 다운로드1
12593정성태4/15/2021239개발 환경 구성: 567. Docker Desktop for Windows - kubectl proxy 없이 k8s 대시보드 접근 방법
12592정성태4/15/2021161개발 환경 구성: 566. Docker Desktop for Windows - k8s dashboard의 Kubeconfig 로그인 및 Skip 방법
12591정성태4/9/2021298.NET Framework: 1034. C# - byte 배열을 Hex(16진수) 문자열로 고속 변환하는 방법파일 다운로드1
12590정성태4/9/2021194.NET Framework: 1033. C# - .NET 4.0 이하에서 Console.IsInputRedirected 구현
12589정성태4/8/2021205.NET Framework: 1032. C# - Environment.OSVersion의 문제점 및 윈도우 운영체제의 버전을 구하는 다양한 방법
12588정성태4/7/2021148개발 환경 구성: 565. PowerShell - New-SelfSignedCertificate를 사용해 CA 인증서 생성 및 인증서 서명 방법
12587정성태4/7/2021269개발 환경 구성: 564. Windows 10 - ClickOnce 배포처럼 사용할 수 있는 MSIX 설치 파일
12586정성태4/22/2021155오류 유형: 710. Windows - Restart-Computer / shutdown 명령어 수행 시 Access is denied(E_ACCESSDENIED)
12585정성태4/5/2021159개발 환경 구성: 563. 기본 생성된 kubeconfig 파일의 내용을 새롭게 생성한 인증서로 구성하는 방법
12584정성태4/12/2021155개발 환경 구성: 562. kubeconfig 파일 없이 kubectl 옵션만으로 실행하는 방법
1  [2]  3  4  5  6  7  8  9  10  11  12  13  14  15  ...