Re: [Qemu-devel] [PATCH 0/5] add CCW indirect data access support

[Cornelia Huck]

On Fri, 8 Sep 2017 12:45:25 +0200
Halil Pasic <address@hidden> wrote:

> The discussion seems to have settled down quite a bit. Since there weren't
> many complaints, I would like to opt for a v2 fixing the things pointed out
> during the review early next week (I was thinking Tuesday maybe some late 
> birds
> are going to join in).
> 
> @Connie 
> ========
> 
> Does that sound reasonable or would you like more time for v1?

No, sounds good.

> 
> What do you think, would it make more sense to omit or to keep the testing
> stuff for v2 (I mean patch 5 and the kernel module in the cover letter)?

Can you maybe split this out? It makes it easier if you don't have to
go hunt in a cover letter.

> 
> You probably haven't found the time to look at have a glance at "s390x/css: 
> drop
> data-check in interpretation" (http://patchwork.ozlabs.org/patch/810995/). We
> have said it would make some things more straight forward here, and I could
> drop that ugly TODO comment. I think it's quite straight-forward, and I would
> not mind having a decision on it before v2 or putting it as preparation into
> v2. What do you prefer?

It is marked for my attention. I don't know whether I find time to look
at it today, but probably early next week.