这段代码还有办法优化么

blackcat242

我们需要减少加锁的时间，目前已经精简到如下的代码了，大家还有什么办法进行减少这段代码的执行时间恩么

LWLockAcquire(ProcArrayLock, LW_SHARED);

xmax = ShmemVariableCache->latestCompletedXid;
numProcs = arrayP->numProcs;
for (index = 0; index < numProcs; ++index)
{
        pgprocno = pgprocnos[index];
        localpgxact[index] = allPgXact[pgprocno];
        memcpy(localpgproc[index].subxids.xids, allProcs[pgprocno].subxids.xids, allPgXact[pgprocno].nxids*sizeof(TransactionId));
}

LWLockRelease(ProcArrayLock);

sxcong

如果能只在单线程里用，去掉锁更精简

blackcat242

sxcong 发表于 2014-10-24 13:38
如果能只在单线程里用，去掉锁更精简

这个不可能了，如果只能在单线程里调用，我还优化他干嘛

hellioncu

关键在memcpy，看看能不能去掉拷贝或者尽量减少拷贝的数据量

blackcat242

hellioncu 发表于 2014-10-24 13:45
关键在memcpy，看看能不能去掉拷贝或者尽量减少拷贝的数据量

这个memcpy已经是优化后的了，只拷贝了结构里面一个字段。这个字段内容是必须要的了

hellioncu

blackcat242 发表于 2014-10-24 13:51
这个memcpy已经是优化后的了，只拷贝了结构里面一个字段。这个字段内容是必须要的了

我的意思是能不能改成指针赋值，“浅拷贝”

hanxin83

顶楼上, 这种情况一般都是尽量变成浅拷贝, 用"多缓冲"技术吧, 数据准备完毕后, 再加锁, 把指针复制过去, 解锁.
略改一下数据结构就行.
不太了解使用场景, 并发情况, 随便说说, 仅供参考.

sxcong

回复 3# blackcat242


问题就在这，其实很多情况其实不需要多线程。
不知道你的业务情况，不过模式是这样的:

线程1---> get       put<--  单线程 process...
线程2---> get
....

如果数据处理不复杂，比如读文件，读数据库，读网络数据，那完全可以单线程处理。
其他线程异步请求，比如UI上面的操作(点一个按键之类的)，并不具体处理业务，避免影响用户体验。

对程序来讲，优化逻辑比优化算法更有效。