[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [virtio-dev] Re: [Qemu-devel] [PATCH v19 0/2] virtio-crypto: virtio crypto device specification
On 09/06/2017 03:52 PM, Michael S. Tsirkin wrote: > On Fri, Sep 01, 2017 at 01:45:45PM +0200, Halil Pasic wrote: >> >> >> On 09/01/2017 02:47 AM, Longpeng (Mike) wrote: >>> Ping... >>> >>> Stefan, Halil, do you have any suggestion ? >>> >> >> Hi Longpeng, >> >> I've ran trough your patch, and it reads much better that >> what I recall v18 used to read like. Because it's been a while >> since v18 doing a conscious review on this will take a considerable >> amount of time (my memories of the issues identified back then >> are very sketchy/vague now). It's on my todo list, but it ain't the >> only item there. > > I don't think it's a fair reason to delay the merging though. > No comments in a while says time to merge to me. > There's always a public review period during which more comments > can be addressed. > Well, the problem is that there was no new version for a couple of months. That's why there were no comments. > So I think we should start a ballot on this one. > Mike - if you agree please open an issue in TC issue > tracker. As a reminder here are the guidelines: > I think, starting a now ballot is a bit rushed: this version is out for less than two weeks, and there was no substantial feedback. That could be OK if we had only minor changes in this version, but I don't think that's the case. Nevertheless, I'm fine with balloting. Will just have to re-prioritize myself, have a thorough look, and if in doubt abstain. Regards, Halil > > -- > > When you open an issue (can be fixed afterwards as well): > > 1. Fill in the reporter in the Environment: field. > 2. Shortly describe the issue in the description field > Preferably add a URL to discussion here or > in the comments. > If the mail was copied to virtio-comment or virtio-dev, you can use > mid.gmane.org/<message-id> > to quickly locate a mail in the archives. > 2. Preferably, fill in all affected versions: in > "Affects Version/s:". > This might not apply to e.g. improvement requests. > > --- > > When you propose the issue before the TC meeting: > > 1. Mark issue as Open > 2. Fill in a summary of the final proposed change and > link to full proposal in "Proposal" > field. > [any historical abandoned proposals and extra info can go > into comments field] > --- > > Then mail me and the TC and I will start a ballot. > > Thanks! > >> Btw. I like to have a reference implementation at hand when reviewing >> a spec. What is the status of the (reference) implementation (I mean >> the new stuff like stateless/mux)? I think it would be nice to provide >> this info in the cover letter (e.g. next time, should we need another >> iteration). >> >> Regards, >> Halil >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org >> For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org > > --------------------------------------------------------------------- > To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org > For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org >
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]