qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [RFC PATCH] target/s390x: Remove KVM stubs in cpu_models.h
@ 2024-04-19  9:06 Philippe Mathieu-Daudé
  2024-04-19  9:15 ` [PATCH] " Philippe Mathieu-Daudé
  0 siblings, 1 reply; 2+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-04-19  9:06 UTC (permalink / raw)
  To: qemu-devel, Zhao Liu
  Cc: David Hildenbrand, Richard Henderson, Thomas Huth,
	Ilya Leoshkevich, qemu-s390x, Philippe Mathieu-Daudé

Since the calls are elided when KVM is not available,
we can remove the stubs (which are never compiled).

Inspired-by: Thomas Huth <thuth@redhat.com>>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
RFC: untested =) Inspired by
https://lore.kernel.org/qemu-devel/0ae778e1-b4aa-4021-bdbc-4c6a663db41a@redhat.com/
---
 target/s390x/cpu_models.h | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/target/s390x/cpu_models.h b/target/s390x/cpu_models.h
index d7b8912989..a89c2a15ab 100644
--- a/target/s390x/cpu_models.h
+++ b/target/s390x/cpu_models.h
@@ -114,23 +114,8 @@ static inline uint64_t s390_cpuid_from_cpu_model(const S390CPUModel *model)
 S390CPUDef const *s390_find_cpu_def(uint16_t type, uint8_t gen, uint8_t ec_ga,
                                     S390FeatBitmap features);
 
-#ifdef CONFIG_KVM
 bool kvm_s390_cpu_models_supported(void);
 void kvm_s390_get_host_cpu_model(S390CPUModel *model, Error **errp);
 void kvm_s390_apply_cpu_model(const S390CPUModel *model,  Error **errp);
-#else
-static inline void kvm_s390_get_host_cpu_model(S390CPUModel *model,
-                                               Error **errp)
-{
-}
-static inline void kvm_s390_apply_cpu_model(const S390CPUModel *model,
-                                            Error **errp)
-{
-}
-static inline bool kvm_s390_cpu_models_supported(void)
-{
-    return false;
-}
-#endif
 
 #endif /* TARGET_S390X_CPU_MODELS_H */
-- 
2.41.0



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] target/s390x: Remove KVM stubs in cpu_models.h
  2024-04-19  9:06 [RFC PATCH] target/s390x: Remove KVM stubs in cpu_models.h Philippe Mathieu-Daudé
@ 2024-04-19  9:15 ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 2+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-04-19  9:15 UTC (permalink / raw)
  To: qemu-devel, Zhao Liu
  Cc: David Hildenbrand, Richard Henderson, Thomas Huth,
	Ilya Leoshkevich, qemu-s390x

On 19/4/24 11:06, Philippe Mathieu-Daudé wrote:
> Since the calls are elided when KVM is not available,
> we can remove the stubs (which are never compiled).
> 
> Inspired-by: Thomas Huth <thuth@redhat.com>>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> RFC: untested =)

Now tested!

Inspired by
> https://lore.kernel.org/qemu-devel/0ae778e1-b4aa-4021-bdbc-4c6a663db41a@redhat.com/
> ---
>   target/s390x/cpu_models.h | 15 ---------------
>   1 file changed, 15 deletions(-)



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-04-19  9:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-19  9:06 [RFC PATCH] target/s390x: Remove KVM stubs in cpu_models.h Philippe Mathieu-Daudé
2024-04-19  9:15 ` [PATCH] " Philippe Mathieu-Daudé

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).