[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH v4 0/4] xentop: add physical CPU usage support


  • To: Anthony PERARD <anthony@xxxxxxxxxxxxxx>
  • From: Jahan Murudi <jahan.murudi.zg@xxxxxxxxxxx>
  • Date: Wed, 3 Sep 2025 18:01:14 +0000
  • Accept-language: en-IN, kn-IN, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com; dkim=pass header.d=renesas.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=L8l4gIdEqhE1cmUACxb3kdeS4deI+iS6Qx7kCU94diQ=; b=wRly8STEC8ku4PaozAOAQiuPBa0T/o0xNZiv3ZaMgfK5pFBsdcjuAS1b2Z9nAuTHLPo1bm4ldZjI6PXM3VMwCdxL96OtpgoORWciuDZPK09gQYLGQmo4vwfJISM0SI27jgkW8Cbb2KBBvk9NbQh2SSUgPhaZItVfH5/DNKso4AB1ZkWMzUZikEp2Bde69bDdVZ2h+uHxgjo4TC7hVFDefKp/v/EGlgF9lZvNCEFulu4hGJQa37dTrCEYKhpaz2v77XtkyglLLpRo0ySye0JQ77Z1b1IGv0uKSH2eGCAVVJqnsbNqrQugayd3DMSOQb4oTTyOxhaUO+Ytw/7Nq0p5ig==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=a99O0zeIa6R2KcrQggsIUSVylnziOZ7KeDdmmtHUItn3PN7p9wCB4i2aPZfsYjzILTIKMJ+7j3GQae/dtC4L62ttjGf5w4hFow2sfP68ZWx1/Pkuwu3ZmWdrjKvnOB1EiqnAzRVMfP1AsaV6ztxw0g79k7BzcKyVEW74zAe71fGuvC0XvDI0KAebm1QUaWvWlS84em/wZl7ZTPfajutd4V8LgepQfaQh4kapS5Fjz0h3xTB4KtsgJkG0dCJUmYLbaHkjkl33+grFunRi0cwLI7yM7YCPqVY05mynLgxwxybelTQdvdTRrZMAYO38HQ/cjekJbFSy3PMgYMGsths35g==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=renesas.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Delivery-date: Wed, 03 Sep 2025 18:01:45 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHcHLzc0wf0/CGB+EiRtdeisPpMILSBn8yAgAARDACAAA3CgA==
  • Thread-topic: [PATCH v4 0/4] xentop: add physical CPU usage support

Hi Anthony,

>> Anyway, squashing back all the patch is the way to go I think.
>> 
>> I'll have a look at the changes.

> So the code looks good, but only if it is a single patch like on v3.
> Would you be fine if I merge the change, but squash all patch of v4 together 
> and take the patch description from v3? (or the patch description of the last 
> patch with "integrate" replace by "introduce".)

> If I do the squashing, I'll just ignore the remark to use "unsigned int"
> instead of "int".

> With all patch squash together, more or less: Reviewed-by: Anthony PERARD 
> <anthony.perard@xxxxxxxxxx>

Thank you for the review and feedback.
I completely understand. Please feel free to squash the v4 series back into a 
single patch for merging, using the patch description from v3 as you suggested.

Thanks again for your time and guidance.

Best regards,
Jahan



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.