On 09/29/2016 02:17 PM, Paolo Bonzini wrote:


On 29/09/2016 19:02, John Snow wrote:


On 09/29/2016 06:25 AM, Kevin Wolf wrote:
John, can you have a look at the IDE code and check whether we can get
rid of the deep recursion? It seems that the test issues a large request
that is then split into many small requests. But it should be possible
to do this iteratively rather than recursively.

http://wiki.qemu.org/Features/IDE

"Rewrite ide_atapi_cmd_reply_end to be fully non-recursive; and ideally
less confusing."

Guess I have to float this one to the top.

Would it be enough to hide the call to s->bus->dma->ops->start_transfer
behind a bottom half?

Paolo


Probably the simplest way, even if not the prettiest.

Reply via email to