Re: [Xen-devel] [PATCH v9 00/25] Enable L2 Cache Allocation Technology & Refactor psr.c

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 17.03.17 at 02:29, <yi.y.sun@xxxxxxxxxxxxxxx> wrote:
> On 17-03-16 05:20:35, Jan Beulich wrote:
>> >>> On 16.03.17 at 12:07, <yi.y.sun@xxxxxxxxxxxxxxx> wrote:
>> > Acked and Reviewed list before V9:
>> > 
>> > a - Acked-by
>> > r - Reviewed-by
>> > 
>> >   r  patch 1  - docs: create Cache Allocation Technology (CAT) and Code and
>> >                 Data Prioritization (CDP) feature document
>> >   ar patch 2  - x86: refactor psr: remove L3 CAT/CDP codes.
>> >   r  patch 3  - x86: refactor psr: implement main data structures.
>> >   r  patch 6  - x86: refactor psr: L3 CAT: implement Domain init/free and
>> >                 schedule flows.
>> >   r  patch 7  - x86: refactor psr: L3 CAT: implement get hw info flow.
>> >   r  patch 8  - x86: refactor psr: L3 CAT: implement get value flow.
>> 
>> I didn't check whether you've kept them, but the changes you've
>> listed certainly are such that the last four of the above pretty likely
>> would need to have any prior tags dropped (in which case the
>> overview above is misleading).
>> 
> Oh, sorry. I thought I should keep all ack/review history. But I am wrong.
> For any reworked patch, I should remove a or r. But one question, if only
> some minor changes added, shall I remove a or r? Thanks!

Cosmetic changes don't normally require dropping tags.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel