优化你的DiscuzNT3.0，让它跑起来（6）在线人数和Regex.IsMatch()引发的hang

最新推荐文章于 2024-07-10 18:43:48 发布

weixin_30597269

最新推荐文章于 2024-07-10 18:43:48 发布

阅读量151

点赞数

文章标签： php 数据库 javascript ViewUI

原文链接：http://www.cnblogs.com/gezifeiyang/archive/2011/06/26/2090836.html

版权

注：本文仅针对 DiscuzNT3.0, sqlserver 2000版本，其他版本请勿对号入座。

你没看错标题，的确是在线人数和Regex.IsMatch()引发的hang。事情是这样的，就在今天我们的论坛出现的挂起问题，当时刚好赶上了抓dump文件。于是就有了今天这篇文章。

我们先用windbg看看论坛当时在干什么吧。

1. 打开文件，运行 .load sos, 因为是hang，所以当然是要运行 !syncblk ，下面是运行结果：

0:000> .load sos

0:000> !syncblk

Index SyncBlock MonitorHeld Recursion Owning Thread Info SyncBlock Owner

202 000fe87c 1 1 1c3d2c98 d28 48 02413704 System.Collections.Generic.LinkedList`1[[System.Text.RegularExpressions.CachedCodeEntry, System]]

-----------------------------

Total 419

CCW 2

RCW 0

ComClassFactory 0

Free 354

从上面看到，持有锁的线程是48 ，持有的对象是System.Collections.Generic.LinkedList，

2. 我们看看线程48在干什么, ~48s 切换到线程，!clrstack 看看执行的代码。

0:000> ~48s

eax=000006d7 ebx=1c3d2c98 ecx=000fe87c edx=044ec5c3 esi=00000354 edi=00000000

eip=7c9585ec esp=1d8eeee4 ebp=1d8eef54 iopl=0 nv up ei pl zr na pe nc

cs=001b ss=0023 ds=0023 es=0023 fs=003b gs=0000 efl=00000246

ntdll!KiFastSystemCallRet:

7c9585ec c3 ret

0:048> !clrstack

OS Thread Id: 0xd28 (48)

ESP EIP

1d8ef060 7c9585ec [GCFrame: 1d8ef060]

1d8ef0fc 7c9585ec [HelperMethodFrame: 1d8ef0fc] System.Threading.Monitor.Enter(System.Object)

1d8ef150 7a4aed85 System.Text.RegularExpressions.Regex.LookupCachedAndUpdate(System.String)

1d8ef180 7a4aec91 System.Text.RegularExpressions.Regex..ctor(System.String, System.Text.RegularExpressions.RegexOptions, Boolean)

1d8ef1b4 7a4bf480 System.Text.RegularExpressions.Regex.IsMatch(System.String, System.String)

1d8ef1c4 1ba6c3cc Discuz.Common.TypeConverter.StrToInt(System.String, Int32)

1d8ef1e0 1ba6f6c2 Discuz.Common.TypeConverter.ObjectToInt(System.Object)

1d8ef1e4 1d50a215 Discuz.Data.OnlineUsers.LoadSingleOnlineUser(System.Data.IDataReader)

1d8ef1f4 1d50a111 Discuz.Data.OnlineUsers.GetOnlineUserCollection()

1d8ef204 1d509f73 Discuz.Forum.OnlineUsers.GetOnlineUserCollection(Int32 ByRef, Int32 ByRef, Int32 ByRef, Int32 ByRef)

1d8ef25c 1d30b291 Discuz.Web.website.ShowPage()

1d8ef274 1ba6d837 Discuz.Forum.PageBase..ctor()

1d8ef318 1d37e2d1 Discuz.Web.website..ctor()

1d8ef324 1d37e0f0 ASP.aspx_2_website_aspx..ctor()

。。。。。。这里省略若干字

--------------------------

我们从上面看到程序调用了

Discuz.Common.TypeConverter.StrToInt() 这个方法，然后进入

System.Text.RegularExpressions.dll，最后停留在

System.Text.RegularExpressions.Regex.LookupCachedAndUpdate() 方法, 我们从dnt3.0的程序一步步来看看。

1          /// <summary>
2          /// 将对象转换为Int32类型
3          /// </summary>
4          /// <param name="str"> 要转换的字符串 </param>
5          /// <param name="defValue"> 缺省值 </param>
6          /// <returns> 转换后的int类型结果 </returns>
7          public static int StrToInt( string str, int defValue)
8         {
9              if ( string .IsNullOrEmpty(str) || str.Trim().Length >= 11 || ! Regex.IsMatch (str.Trim(), @" ^([-]|[0-9])[0-9]*(\.\w*)?$ " ))
10                  return defValue;
11
12              int rv;
13              if (Int32.TryParse(str, out rv))
14                  return rv;
15
16              return Convert.ToInt32(StrToFloat(str, defValue));

17 }

请出reflector

1 public static bool IsMatch( string input, string pattern)
2 {
3 return new Regex(pattern, RegexOptions.None, true).IsMatch(input);

4 }

继续reflector

1 private Regex( string pattern, RegexOptions options, bool useCache)
2 {
3     CachedCodeEntry cachedAndUpdate = null ;
4      string threeLetterWindowsLanguageName = null ;
5      if (pattern == null )
6     {
7          throw new ArgumentNullException( " pattern " );
8     }
9      if ((options < RegexOptions.None) || (((( int ) options) >> 10 ) != 0 ))
10     {
11          throw new ArgumentOutOfRangeException( " options " );
12     }
13      if (((options & RegexOptions.ECMAScript) != RegexOptions.None) && ((options & ~ (RegexOptions.CultureInvariant | RegexOptions.ECMAScript | RegexOptions.Compiled | RegexOptions.Multiline | RegexOptions.IgnoreCase)) != RegexOptions.None))
14     {
15          throw new ArgumentOutOfRangeException( " options " );
16     }
17      if ((options & RegexOptions.CultureInvariant) != RegexOptions.None)
18     {
19         threeLetterWindowsLanguageName = CultureInfo.InvariantCulture.ThreeLetterWindowsLanguageName;
20     }
21      else
22     {
23         threeLetterWindowsLanguageName = CultureInfo.CurrentCulture.ThreeLetterWindowsLanguageName;
24     }
25      string [] strArray = new string [] { (( int ) options).ToString(NumberFormatInfo.InvariantInfo), " : " , threeLetterWindowsLanguageName, " : " , pattern };
26      string key = string .Concat(strArray);
27     cachedAndUpdate = LookupCachedAndUpdate (key);
28      this .pattern = pattern;
29      this .roptions = options;
30      if (cachedAndUpdate == null )
31     {
32         RegexTree t = RegexParser.Parse(pattern, this .roptions);
33          this .capnames = t._capnames;
34          this .capslist = t._capslist;
35          this .code = RegexWriter.Write(t);
36          this .caps = this .code._caps;
37          this .capsize = this .code._capsize;
38          this .InitializeReferences();
39         t = null ;
40          if (useCache)
41         {
42             cachedAndUpdate = this .CacheCode(key);
43         }
44     }
45      else
46     {
47          this .caps = cachedAndUpdate._caps;
48          this .capnames = cachedAndUpdate._capnames;
49          this .capslist = cachedAndUpdate._capslist;
50          this .capsize = cachedAndUpdate._capsize;
51          this .code = cachedAndUpdate._code;
52          this .factory = cachedAndUpdate._factory;
53          this .runnerref = cachedAndUpdate._runnerref;
54          this .replref = cachedAndUpdate._replref;
55          this .refsInitialized = true ;
56     }
57      if ( this .UseOptionC() && ( this .factory == null ))
58     {
59          this .factory = this .Compile( this .code, this .roptions);
60          if (useCache && (cachedAndUpdate != null ))
61         {
62             cachedAndUpdate.AddCompiled( this .factory);
63         }
64          this .code = null ;
65     }

66 }

这个代码量较大，找到关键点 LookupCachedAndUpdate

private static CachedCodeEntry LookupCachedAndUpdate( string key)
{
     lock (livecode)
    {
for (LinkedListNode < CachedCodeEntry > node = livecode.First; node != null ; node = node.Next)
        {
             if (node.Value._key == key)
            {
                livecode.Remove(node);
                livecode.AddFirst(node);
                 return node.Value;
            }
        }
    }
     return null ;

}

终于找到这个lock的对象了，看看他是什么类型的，和我们通过windbg看到的一样吗

internal static LinkedList<CachedCodeEntry> livecode;

果然一样，这下应该放心了，就是这里的lock引起了hang，但是我们应该经常用Regex.IsMatch()的，也没见引起这个问题啊，为什么这里？？？

我们看看在线人数有多少，如果在线人数比较多，访问的人数也多，那可能性就很大了，我们来看看到底有多少人在线。

运行 !dso,看看本线程对象。

0:048> !dso

OS Thread Id: 0xd28 (48)

ESP/REG Object Name

1d8ef094 02413704 System.Collections.Generic.LinkedList`1[[System.Text.RegularExpressions.CachedCodeEntry, System]]

1d8ef0b8 074b8c54 System.String 0:CHS:^([-]|[0-9])[0-9]*(\.\w*)?$

1d8ef0bc 074b94ac System.String 114.247.10.127

1d8ef0d0 074b94ac System.String 114.247.10.127

1d8ef150 02413704 System.Collections.Generic.LinkedList`1[[System.Text.RegularExpressions.CachedCodeEntry, System]]

1d8ef170 074b8c20 System.Text.RegularExpressions.Regex

1d8ef184 0a3fcd1c System.String ^([-]|[0-9])[0-9]*(\.\w*)?$

1d8ef190 074b8c54 System.String 0:CHS:^([-]|[0-9])[0-9]*(\.\w*)?$

1d8ef19c 074b8c08 System.String -1

1d8ef1a0 074b8c20 System.Text.RegularExpressions.Regex

1d8ef1a4 0a3fcd1c System.String ^([-]|[0-9])[0-9]*(\.\w*)?$

1d8ef1b4 068e2dc4 Discuz.Common.Generic.List`1[[Discuz.Entity.OnlineUserInfo, Discuz.Entity]]

1d8ef1b8 074b8c08 System.String -1

1d8ef1d4 074b8bac Discuz.Entity.OnlineUserInfo

1d8ef1d8 068e2f64 System.Data.SqlClient.SqlDataReader

。。。。。。省略若干字

----------------------

找到上面的

Discuz.Common.Generic.List 的地址

068e2dc4 ，运行 !do

068e2dc4

0:048> !do 068e2dc4

Name: Discuz.Common.Generic.List`1[[Discuz.Entity.OnlineUserInfo, Discuz.Entity]]

MethodTable: 1ceb7084

EEClass: 1bb39dd0

Size: 28(0x1c) bytes

(C:\WINDOWS\Microsoft.NET\Framework\v2.0.50727\Temporary ASP.NET Files\root\9932c999\b8dfff59\assembly\dl3\21614a1e\486ed665_6d88ca01\Discuz.Common.DLL)

Fields:

MT Field Offset Type VT Attr Value Name

79124228 400099d 4 System.Object[] 0 instance 0f1b254c _items

790fed1c 400099e c System.Int32 0 instance 472 _size

790fed1c 400099f 10 System.Int32 0 instance 472 _version

790f9c18 40009a0 8 System.Object 0 instance 00000000 _syncRoot

79124228 40009a1 0 System.Object[] 0 shared static _emptyArray

>> Domain:Value dynamic statics NYI

000e1008:NotInit dynamic statics NYI

00107038:NotInit <<

790fed1c 400000c 14 System.Int32 0 instance 0 _fixedsize

从上面的size可以看到在线人数是472人，就是说这个48这个线程要lock 472 次，如果有n个人访问那后面的人真的要等不少时候了。

话说StrToInt()这个方法为什么要用Regex.IsMatch()呢，string 转换成 int 一般 int.TryParse()也足够了。不过从这里我才发现Regex.IsMatch() 里面原来还有个lock，不然还真不知道，也算是收获不小啊。

转载于:https://www.cnblogs.com/gezifeiyang/archive/2011/06/26/2090836.html

weixin_30597269

关注

0
点赞
踩
0

收藏

觉得还不错? 一键收藏
0
评论
优化你的DiscuzNT3.0，让它跑起来（6）在线人数和Regex.IsMatch()引发的hang

注：本文仅针对 DiscuzNT3.0, sqlserver 2000版本，其他版本请勿对号入座。你没看错标题，的确是在线人数和Regex.IsMatch()引发的hang。事情是这样的，就在今天我们的论坛出现的挂起问题，当时刚好赶上了抓dump文件。于是就有了今天这篇文章。我们先用windbg看看论坛当时在干什么吧。1. 打开文件，运行 .load sos, 因为是hang，所以当然是要运行...
复制链接

扫一扫