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

Re: [PATCH v5 05/15] pmstat&xenpm: Re-arrage for cpufreq union


  • To: Jason Andryuk <jandryuk@xxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 10 Jul 2023 14:09:55 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=QF3cSdk6rYS+BzEnzl8gS/3nPiFhCHZ6zpoXcZV0KUc=; b=XIvmSYR1t7a4TfvLVVJlLsGJIAjMXZPBesxXPh+K4xXPZWFhqKBYAXCZYHjSzH3ksu7l4QWfn1N4PGHkhYIbyArg/Z/dCM+8yNOfpKxomybuPaaeGraZER+cd0sDRX6HFQnHyIHhWejf/8nIRpvZ+R9JzPtMvUgRsu5O8OHSQSBxM8JlI3zmPmZjJaOUV8/L8oalF94DXNVCYFUOjYvA0TBCB8GeKDjjn9R9+aX1SrmH08B/Zs8hUUDdh5HRrqtcODhmuWRkGYx9uM/4rN68U8NxGJ7RLiWIumCq7ZOhfQOdbZTRjjUQvrlyyszlcYG0Lo9BtYr1DlwWX+JfjPQ2oA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UiaJQFZqpc3T7T6/kKZHyBPqP0C5A3poo1hoh2aKehk9TNx8O3avfynaKuYlr+kSokfOs/RKA+Z+rT7osnAPqOSOpXZWPbH3Pb5YpURSuG68aCZv18XPlKwmFqDM7suUqUwnJty+7ciLmB/c5f70MVzt4CPAD1s/0zvRkB2hdQ9E3c8hgyaeu57M8SRaHcnxh5RzNG9Wh3B4mD5u+wArZHYxjasKBa831hgLbvDLxz+JYbzVlj1qMp3bXrl8ADzX4mJ07vrgEV7HHTRsa7wXAtLYQftBiy3azhCjqXRMECvfUbgc3j7O0iFyqcYrlgQ78sxlLJTjoYdN2SkY5uZ5dA==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Mon, 10 Jul 2023 12:10:20 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 06.07.2023 20:54, Jason Andryuk wrote:
> Rearrange code now that xen_sysctl_pm_op's get_para fields has the
> nested union and struct.  In particular, the scaling governor
> information like scaling_available_governors is inside the union, so it
> is not always available.  Move those fields (op->u.get_para.u.s.u.*)
> together as well as the common fields (ones outside the union like
> op->u.get_para.turbo_enabled).
> 
> With that, gov_num may be 0, so bounce buffer handling needs
> to be modified.
> 
> scaling_governor and other fields inside op->u.get_para.u.s.u.* won't be
> used for hwp, so this will simplify the change when hwp support is
> introduced and re-indents these lines all together.
> 
> Signed-off-by: Jason Andryuk <jandryuk@xxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>





 


Rackspace

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