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

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


  • To: Jason Andryuk <jandryuk@xxxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Thu, 27 Jul 2023 11:45:51 +0100
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>
  • Delivery-date: Thu, 27 Jul 2023 10:46:08 +0000
  • Ironport-data: A9a23:A8vgiKBOASEfwxVW/+rjw5YqxClBgxIJ4kV8jS/XYbTApDJz3zMGz jRLCzqOOvaOZWCgKNkiPt61/E0EvpTRn4IyQQY4rX1jcSlH+JHPbTi7wuUcHAvJd5GeExg3h yk6QoOdRCzhZiaE/n9BCpC48T8nk/nOHuGmYAL9EngZbRd+Tys8gg5Ulec8g4p56fC0GArIs t7pyyHlEAbNNwVcbCRMsspvlDs15K6p4GxC4QRnDRx2lAS2e0c9Xcp3yZ6ZdxMUcqEMdsamS uDKyq2O/2+x13/B3fv8z94X2mVTKlLjFVDmZkh+AsBOsTAbzsAG6Y4pNeJ0VKtio27hc+ada jl6ncfYpQ8BZsUgkQmGOvVSO3kW0aZuoNcrLZUj2CA6IoKvn3bEmp1T4E8K0YIw3+l0E2BX1 NwiDx8/UBKzifu325uERbw57igjBJGD0II3v3hhyXfSDOo8QICFSKLPjTNa9G5u3IYUR6+YP pdHL2M1N3wsYDUWUrsTIJs4gOevgGi5azBCoUiZjaE2/3LS3Ep6172F3N/9I4bSH5gJwBjJz o7A11viUxcga/GP8gjG4iK1q6zMxjLmR41HQdVU8dY12QbOlwT/EiY+V1K9reOol02WVNdWK khS8S0rxYAi+UruQtTjUhmQpH+fogVaS9dWC/c96gyG1uzT+QnxLncAZi5MbpohrsBebT4lz FiShPvyGCdi9raSTBq19L2ZsDezMig9NnIZaGkPSg5ty9D7rZs6lB7nUtdpG6nzhdrwcQwc2 BjT8nJ43e9Ky5dWiePipwuvby+QSobhawE29ATOf0KfySxLWoyVaZWp8nHk4qMVRGqGdWVtr EToiuDHsrFWVsDWxHTRKAkeNOr3vqjYaVUwlXYqRsB8rGr1phZPaKgKuFlDyFFV3tHokNMDS Gvaoktv6ZBaJxNGhocnMtvqW6zGIUUNfOkJt8w4jfIUOPCdjCfdoElTibSs927silMwtqo0J I2Wd82hZV5DV/U/l2voHbhMiud0rszb+Y81bcqlp/hA+ePADEN5tJ9faAfeBgzHxP3sTPrpH yZ3aJLRlkQ3vBzWaSjL648DRW3m3lBiba0aX/d/L7bZSiI/QTFJNhMk6e95E6R/gb9vn/vFl lnkHBcwJKzX2SeWdm1nqxlLNNvSYHqIhStkZndxbAnwgCNLjETGxP53SqbbtIIPrIRLpcOYh dFcEylcKpyjkgj6xgk=
  • Ironport-hdrordr: A9a23:daG0N6tYnxT7Wy4/AJ0OfYop7skDxNV00zEX/kB9WHVpmwKj5q STdZUgpGzJYVMqMx8dcL+7WJVoPkmsj6KdjbNxAV7AZniWhILXFvAB0WKK+VSJcREWndQttp uIHZIObeEZBjBB/LvHCGHTKbodKLLsys+VbSi19RpQZDAvUoUlyzpQTj+cFEgefng3OXL6fq DsmvauY1CbCAoqhgHQPAh7YwG5naytqLvWJSQeAgIh6k2kkTOs5dfBYmOl9yZbaSpL3bAhtV LkvmXCl8OemsD+8AbYy2jQq65bg8Ts0fxKAMDksLltFhzczjywYZhnWfm8sCsuoOepgWxa6O Xxnw==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Wed, Jul 26, 2023 at 01:09:35PM -0400, 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>

Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Thanks,

-- 
Anthony PERARD



 


Rackspace

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