(please don't top post) yiboz <yiboz@xxxxxxxxxxxxxx> writes: > We have met performance issue on our two-core system after applying > your patch. In WDS mode, we found that the peak throughput in TCP-DL > and UDP-DL dropped more than 10% compared with previous one. And in > some cases, though throughput stays the same, one CPU usage rises > about 20% which leads to 10% in total CPU usage. With your change, I > think driver will try its best to push as many packets as it can. > During this time, the driver's queue lock will be held for too much > time in one CPU and as a result, the other CPU will be blocked if it > wants to acquired the same lock. Working in this way seems not > efficiency. > > So I think it is better to revert the change till we come up with a > new solution. I don't think reverting is a clear option at this stage because that again creates problems for SDIO. IIRC without this patch SDIO was sending one packet a time (or something like that, can't remember all the details right now). Why does this happen only WDS mode? Did you test other modes, like AP or client mode? -- Kalle Valo
