>>> On 04.08.16 at 18:16, <tamas.leng...@zentific.com> wrote:
> On Thu, Aug 4, 2016 at 1:51 AM, Jan Beulich <jbeul...@suse.com> wrote:
>>>>> On 04.08.16 at 08:36, <tamas.leng...@zentific.com> wrote:
>>> On Thu, Aug 4, 2016 at 12:29 AM, Jan Beulich <jbeul...@suse.com> wrote:
>>>>>>> On 03.08.16 at 20:41, <tamas.leng...@zentific.com> wrote:
>>>>> @@ -1846,11 +1846,15 @@ int hvm_hap_nested_page_fault(paddr_t gpa,
>>>>> unsigned
> long gla,
>>>>> }
>>>>> }
>>>>>
>>>>> - if ( p2m_mem_access_check(gpa, gla, npfec, &req_ptr) )
>>>>> - {
>>>>> + sync = p2m_mem_access_check(gpa, gla, npfec, &req_ptr);
>>>>> +
>>>>> + if ( !sync )
>>>>> fall_through = 1;
>>>>> - } else {
>>>>> - /* Rights not promoted, vcpu paused, work here is done */
>>>>> + else
>>>>> + {
>>>>> + /*
>>>>> + * Rights not promoted (aka. sync event), work here is
>>>>> done
>>>>> + */
>>>>
>>>> Comment style.
>>>
>>> Alright, you had an issue with the pre-existing comment style but you
>>> also have an issue with this. What exactly is the issue here?
>>
>> See ./CODING_STYLE: This is a single line comment, and what was
>> and is missing is the full stop at the end.
>
> Well, thanks for spotting it but I'm not going to resend resend the
> patch for this. If that's seriously all that's missing and you feel
> really strongly about it things like that have routinely been
> corrected when the patch is applied.
Sure, no problem if I end up committing it. As it stands, it's not fully
acked, so I can't apply it just yet.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
