Re: [PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-11 Thread Mike Travis




On 9/10/2019 11:04 PM, Ingo Molnar wrote:


* Mike Travis  wrote:


@@ -1596,7 +1687,7 @@ static void __init uv_system_init_hub(vo
uv_nmi_setup();
uv_cpu_init();
uv_scir_register_cpu_notifier();
-   proc_mkdir("sgi_uv", NULL);
+   uv_setup_proc_files(0);


This slipped through previously: platform drivers have absolutely no
business mucking in /proc.

Please describe the hardware via sysfs as pretty much everyone else does.

Thanks,

Ingo



If I was doing it now I definitely would put it in the sysfs realm.  The 
problem is Jack did it back in (I think) 2007.  The earliest commit I 
could find:


commit a3d732f93785da17e0137210deadb4616f5536fc
Author: Cliff Wickman 
Date:   Mon Nov 10 16:16:31 2008 -0600

x86, UV: fix redundant creation of sgi_uv

Impact: fix double entry creation in /proc

And in the past 12 years probably a hundred user programs are now keying 
of the presence of /proc/sgi_uv to signal this is indeed a UV system. 
Changing the location of this node also affects all the UV utilities 
including those not written by us.


Re: [PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-11 Thread Ingo Molnar


* Mike Travis  wrote:

> @@ -1596,7 +1687,7 @@ static void __init uv_system_init_hub(vo
>   uv_nmi_setup();
>   uv_cpu_init();
>   uv_scir_register_cpu_notifier();
> - proc_mkdir("sgi_uv", NULL);
> + uv_setup_proc_files(0);

This slipped through previously: platform drivers have absolutely no 
business mucking in /proc.

Please describe the hardware via sysfs as pretty much everyone else does.

Thanks,

Ingo


[PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-10 Thread Mike Travis
Indicate to UV user utilities that UV hubless support is available on
this system via the existing /proc infterface.  The current interface is
maintained with the addition of new /proc leaves ("hubbed", "hubless",
and "oemid") that contain the specific type of UV arch this one is.

Signed-off-by: Mike Travis 
Reviewed-by: Steve Wahl 
Reviewed-by: Dimitri Sivanich 
---
V2: Remove is_uv_hubbed define
Remove leading '_' from _is_uv_hubbed
---
 arch/x86/include/asm/uv/uv.h   |4 +
 arch/x86/kernel/apic/x2apic_uv_x.c |   93 -
 2 files changed, 96 insertions(+), 1 deletion(-)

--- linux.orig/arch/x86/include/asm/uv/uv.h
+++ linux/arch/x86/include/asm/uv/uv.h
@@ -12,6 +12,8 @@ struct mm_struct;
 #ifdef CONFIG_X86_UV
 #include 
 
+#defineUV_PROC_NODE"sgi_uv"
+
 static inline int uv(int uvtype)
 {
/* uv(0) is "any" */
@@ -28,6 +30,7 @@ static inline bool is_early_uv_system(vo
return uv_systab_phys && uv_systab_phys != EFI_INVALID_TABLE_ADDR;
 }
 extern int is_uv_system(void);
+extern int is_uv_hubbed(int uvtype);
 extern int is_uv_hubless(int uvtype);
 extern void uv_cpu_init(void);
 extern void uv_nmi_init(void);
@@ -40,6 +43,7 @@ extern const struct cpumask *uv_flush_tl
 static inline enum uv_system_type get_uv_system_type(void) { return UV_NONE; }
 static inline bool is_early_uv_system(void){ return 0; }
 static inline int is_uv_system(void)   { return 0; }
+static inline int is_uv_hubbed(int uv) { return 0; }
 static inline int is_uv_hubless(int uv) { return 0; }
 static inline void uv_cpu_init(void)   { }
 static inline void uv_system_init(void){ }
--- linux.orig/arch/x86/kernel/apic/x2apic_uv_x.c
+++ linux/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -26,6 +26,7 @@
 static DEFINE_PER_CPU(int, x2apic_extra_bits);
 
 static enum uv_system_type uv_system_type;
+static int uv_hubbed_system;
 static int uv_hubless_system;
 static u64 gru_start_paddr, gru_end_paddr;
 static u64 gru_dist_base, gru_first_node_paddr = -1LL, 
gru_last_node_paddr;
@@ -309,6 +310,24 @@ static int __init uv_acpi_madt_oem_check
if (uv_hub_info->hub_revision == 0)
goto badbios;
 
+   switch (uv_hub_info->hub_revision) {
+   case UV4_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x11;
+   break;
+
+   case UV3_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x9;
+   break;
+
+   case UV2_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x5;
+   break;
+
+   case UV1_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x3;
+   break;
+   }
+
pnodeid = early_get_pnodeid();
early_get_apic_socketid_shift();
 
@@ -359,6 +378,12 @@ int is_uv_system(void)
 }
 EXPORT_SYMBOL_GPL(is_uv_system);
 
+int is_uv_hubbed(int uvtype)
+{
+   return (uv_hubbed_system & uvtype);
+}
+EXPORT_SYMBOL_GPL(is_uv_hubbed);
+
 int is_uv_hubless(int uvtype)
 {
return (uv_hubless_system & uvtype);
@@ -1457,6 +1482,68 @@ static void __init build_socket_tables(v
}
 }
 
+/* Setup user proc fs files */
+static int proc_hubbed_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubbed_system);
+   return 0;
+}
+
+static int proc_hubless_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubless_system);
+   return 0;
+}
+
+static int proc_oemid_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "%s/%s\n", oem_id, oem_table_id);
+   return 0;
+}
+
+static int proc_hubbed_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubbed_show, (void *)NULL);
+}
+
+static int proc_hubless_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubless_show, (void *)NULL);
+}
+
+static int proc_oemid_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_oemid_show, (void *)NULL);
+}
+
+/* (struct is "non-const" as open function is set at runtime) */
+static struct file_operations proc_version_fops = {
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static const struct file_operations proc_oemid_fops = {
+   .open   = proc_oemid_open,
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static __init void uv_setup_proc_files(int hubless)
+{
+   struct proc_dir_entry *pde;
+   char *name = hubless ? "hubless" : "hubbed";
+
+   pde = proc_mkdir(UV_PROC_NODE, NULL);
+   proc_create("oemid", 0, pde, _oemid_fops);
+   proc_create(name, 0, pde, _version_fops);
+   if (hubless)
+   proc_version_fops.open = proc_hubless_open;
+   else
+   proc_version_fops.open = proc_hubbed_open;
+}
+
 /* Initialize UV 

[PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-10 Thread Mike Travis
Indicate to UV user utilities that UV hubless support is available on
this system via the existing /proc infterface.  The current interface is
maintained with the addition of new /proc leaves ("hubbed", "hubless",
and "oemid") that contain the specific type of UV arch this one is.

Signed-off-by: Mike Travis 
Reviewed-by: Steve Wahl 
Reviewed-by: Dimitri Sivanich 
---
V2: Remove is_uv_hubbed define
Remove leading '_' from _is_uv_hubbed
---
 arch/x86/include/asm/uv/uv.h   |4 +
 arch/x86/kernel/apic/x2apic_uv_x.c |   93 -
 2 files changed, 96 insertions(+), 1 deletion(-)

--- linux.orig/arch/x86/include/asm/uv/uv.h
+++ linux/arch/x86/include/asm/uv/uv.h
@@ -12,6 +12,8 @@ struct mm_struct;
 #ifdef CONFIG_X86_UV
 #include 
 
+#defineUV_PROC_NODE"sgi_uv"
+
 static inline int uv(int uvtype)
 {
/* uv(0) is "any" */
@@ -28,6 +30,7 @@ static inline bool is_early_uv_system(vo
return uv_systab_phys && uv_systab_phys != EFI_INVALID_TABLE_ADDR;
 }
 extern int is_uv_system(void);
+extern int is_uv_hubbed(int uvtype);
 extern int is_uv_hubless(int uvtype);
 extern void uv_cpu_init(void);
 extern void uv_nmi_init(void);
@@ -40,6 +43,7 @@ extern const struct cpumask *uv_flush_tl
 static inline enum uv_system_type get_uv_system_type(void) { return UV_NONE; }
 static inline bool is_early_uv_system(void){ return 0; }
 static inline int is_uv_system(void)   { return 0; }
+static inline int is_uv_hubbed(int uv) { return 0; }
 static inline int is_uv_hubless(int uv) { return 0; }
 static inline void uv_cpu_init(void)   { }
 static inline void uv_system_init(void){ }
--- linux.orig/arch/x86/kernel/apic/x2apic_uv_x.c
+++ linux/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -26,6 +26,7 @@
 static DEFINE_PER_CPU(int, x2apic_extra_bits);
 
 static enum uv_system_type uv_system_type;
+static int uv_hubbed_system;
 static int uv_hubless_system;
 static u64 gru_start_paddr, gru_end_paddr;
 static u64 gru_dist_base, gru_first_node_paddr = -1LL, 
gru_last_node_paddr;
@@ -309,6 +310,24 @@ static int __init uv_acpi_madt_oem_check
if (uv_hub_info->hub_revision == 0)
goto badbios;
 
+   switch (uv_hub_info->hub_revision) {
+   case UV4_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x11;
+   break;
+
+   case UV3_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x9;
+   break;
+
+   case UV2_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x5;
+   break;
+
+   case UV1_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x3;
+   break;
+   }
+
pnodeid = early_get_pnodeid();
early_get_apic_socketid_shift();
 
@@ -359,6 +378,12 @@ int is_uv_system(void)
 }
 EXPORT_SYMBOL_GPL(is_uv_system);
 
+int is_uv_hubbed(int uvtype)
+{
+   return (uv_hubbed_system & uvtype);
+}
+EXPORT_SYMBOL_GPL(is_uv_hubbed);
+
 int is_uv_hubless(int uvtype)
 {
return (uv_hubless_system & uvtype);
@@ -1457,6 +1482,68 @@ static void __init build_socket_tables(v
}
 }
 
+/* Setup user proc fs files */
+static int proc_hubbed_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubbed_system);
+   return 0;
+}
+
+static int proc_hubless_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubless_system);
+   return 0;
+}
+
+static int proc_oemid_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "%s/%s\n", oem_id, oem_table_id);
+   return 0;
+}
+
+static int proc_hubbed_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubbed_show, (void *)NULL);
+}
+
+static int proc_hubless_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubless_show, (void *)NULL);
+}
+
+static int proc_oemid_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_oemid_show, (void *)NULL);
+}
+
+/* (struct is "non-const" as open function is set at runtime) */
+static struct file_operations proc_version_fops = {
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static const struct file_operations proc_oemid_fops = {
+   .open   = proc_oemid_open,
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static __init void uv_setup_proc_files(int hubless)
+{
+   struct proc_dir_entry *pde;
+   char *name = hubless ? "hubless" : "hubbed";
+
+   pde = proc_mkdir(UV_PROC_NODE, NULL);
+   proc_create("oemid", 0, pde, _oemid_fops);
+   proc_create(name, 0, pde, _version_fops);
+   if (hubless)
+   proc_version_fops.open = proc_hubless_open;
+   else
+   proc_version_fops.open = proc_hubbed_open;
+}
+
 /* Initialize UV 

[PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-10 Thread Mike Travis
Indicate to UV user utilities that UV hubless support is available on
this system via the existing /proc infterface.  The current interface is
maintained with the addition of new /proc leaves ("hubbed", "hubless",
and "oemid") that contain the specific type of UV arch this one is.

Signed-off-by: Mike Travis 
Reviewed-by: Steve Wahl 
Reviewed-by: Dimitri Sivanich 
---
V2: Remove is_uv_hubbed define
Remove leading '_' from _is_uv_hubbed
---
 arch/x86/include/asm/uv/uv.h   |4 +
 arch/x86/kernel/apic/x2apic_uv_x.c |   93 -
 2 files changed, 96 insertions(+), 1 deletion(-)

--- linux.orig/arch/x86/include/asm/uv/uv.h
+++ linux/arch/x86/include/asm/uv/uv.h
@@ -12,6 +12,8 @@ struct mm_struct;
 #ifdef CONFIG_X86_UV
 #include 
 
+#defineUV_PROC_NODE"sgi_uv"
+
 static inline int uv(int uvtype)
 {
/* uv(0) is "any" */
@@ -28,6 +30,7 @@ static inline bool is_early_uv_system(vo
return uv_systab_phys && uv_systab_phys != EFI_INVALID_TABLE_ADDR;
 }
 extern int is_uv_system(void);
+extern int is_uv_hubbed(int uvtype);
 extern int is_uv_hubless(int uvtype);
 extern void uv_cpu_init(void);
 extern void uv_nmi_init(void);
@@ -40,6 +43,7 @@ extern const struct cpumask *uv_flush_tl
 static inline enum uv_system_type get_uv_system_type(void) { return UV_NONE; }
 static inline bool is_early_uv_system(void){ return 0; }
 static inline int is_uv_system(void)   { return 0; }
+static inline int is_uv_hubbed(int uv) { return 0; }
 static inline int is_uv_hubless(int uv) { return 0; }
 static inline void uv_cpu_init(void)   { }
 static inline void uv_system_init(void){ }
--- linux.orig/arch/x86/kernel/apic/x2apic_uv_x.c
+++ linux/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -26,6 +26,7 @@
 static DEFINE_PER_CPU(int, x2apic_extra_bits);
 
 static enum uv_system_type uv_system_type;
+static int uv_hubbed_system;
 static int uv_hubless_system;
 static u64 gru_start_paddr, gru_end_paddr;
 static u64 gru_dist_base, gru_first_node_paddr = -1LL, 
gru_last_node_paddr;
@@ -309,6 +310,24 @@ static int __init uv_acpi_madt_oem_check
if (uv_hub_info->hub_revision == 0)
goto badbios;
 
+   switch (uv_hub_info->hub_revision) {
+   case UV4_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x11;
+   break;
+
+   case UV3_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x9;
+   break;
+
+   case UV2_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x5;
+   break;
+
+   case UV1_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x3;
+   break;
+   }
+
pnodeid = early_get_pnodeid();
early_get_apic_socketid_shift();
 
@@ -359,6 +378,12 @@ int is_uv_system(void)
 }
 EXPORT_SYMBOL_GPL(is_uv_system);
 
+int is_uv_hubbed(int uvtype)
+{
+   return (uv_hubbed_system & uvtype);
+}
+EXPORT_SYMBOL_GPL(is_uv_hubbed);
+
 int is_uv_hubless(int uvtype)
 {
return (uv_hubless_system & uvtype);
@@ -1457,6 +1482,68 @@ static void __init build_socket_tables(v
}
 }
 
+/* Setup user proc fs files */
+static int proc_hubbed_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubbed_system);
+   return 0;
+}
+
+static int proc_hubless_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubless_system);
+   return 0;
+}
+
+static int proc_oemid_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "%s/%s\n", oem_id, oem_table_id);
+   return 0;
+}
+
+static int proc_hubbed_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubbed_show, (void *)NULL);
+}
+
+static int proc_hubless_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubless_show, (void *)NULL);
+}
+
+static int proc_oemid_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_oemid_show, (void *)NULL);
+}
+
+/* (struct is "non-const" as open function is set at runtime) */
+static struct file_operations proc_version_fops = {
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static const struct file_operations proc_oemid_fops = {
+   .open   = proc_oemid_open,
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static __init void uv_setup_proc_files(int hubless)
+{
+   struct proc_dir_entry *pde;
+   char *name = hubless ? "hubless" : "hubbed";
+
+   pde = proc_mkdir(UV_PROC_NODE, NULL);
+   proc_create("oemid", 0, pde, _oemid_fops);
+   proc_create(name, 0, pde, _version_fops);
+   if (hubless)
+   proc_version_fops.open = proc_hubless_open;
+   else
+   proc_version_fops.open = proc_hubbed_open;
+}
+
 /* Initialize UV 

[PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-09 Thread Mike Travis
Indicate to UV user utilities that UV hubless support is available on
this system via the existing /proc infterface.  The current interface is
maintained with the addition of new /proc leaves ("hubbed", "hubless",
and "oemid") that contain the specific type of UV arch this one is.

Signed-off-by: Mike Travis 
Reviewed-by: Steve Wahl 
Reviewed-by: Dimitri Sivanich 
To: Thomas Gleixner 
To: Ingo Molnar 
To: H. Peter Anvin 
To: Andrew Morton 
To: Borislav Petkov 
To: Christoph Hellwig 
To: Sasha Levin 
Cc: Dimitri Sivanich 
Cc: Russ Anderson 
Cc: Hedi Berriche 
Cc: Steve Wahl 
Cc: Justin Ernst 
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
V2: Remove is_uv_hubbed define
Remove leading '_' from _is_uv_hubbed
---
 arch/x86/include/asm/uv/uv.h   |4 +
 arch/x86/kernel/apic/x2apic_uv_x.c |   93 -
 2 files changed, 96 insertions(+), 1 deletion(-)

--- linux.orig/arch/x86/include/asm/uv/uv.h
+++ linux/arch/x86/include/asm/uv/uv.h
@@ -12,6 +12,8 @@ struct mm_struct;
 #ifdef CONFIG_X86_UV
 #include 
 
+#defineUV_PROC_NODE"sgi_uv"
+
 static inline int uv(int uvtype)
 {
/* uv(0) is "any" */
@@ -28,6 +30,7 @@ static inline bool is_early_uv_system(vo
return uv_systab_phys && uv_systab_phys != EFI_INVALID_TABLE_ADDR;
 }
 extern int is_uv_system(void);
+extern int is_uv_hubbed(int uvtype);
 extern int is_uv_hubless(int uvtype);
 extern void uv_cpu_init(void);
 extern void uv_nmi_init(void);
@@ -40,6 +43,7 @@ extern const struct cpumask *uv_flush_tl
 static inline enum uv_system_type get_uv_system_type(void) { return UV_NONE; }
 static inline bool is_early_uv_system(void){ return 0; }
 static inline int is_uv_system(void)   { return 0; }
+static inline int is_uv_hubbed(int uv) { return 0; }
 static inline int is_uv_hubless(int uv) { return 0; }
 static inline void uv_cpu_init(void)   { }
 static inline void uv_system_init(void){ }
--- linux.orig/arch/x86/kernel/apic/x2apic_uv_x.c
+++ linux/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -26,6 +26,7 @@
 static DEFINE_PER_CPU(int, x2apic_extra_bits);
 
 static enum uv_system_type uv_system_type;
+static int uv_hubbed_system;
 static int uv_hubless_system;
 static u64 gru_start_paddr, gru_end_paddr;
 static u64 gru_dist_base, gru_first_node_paddr = -1LL, 
gru_last_node_paddr;
@@ -309,6 +310,24 @@ static int __init uv_acpi_madt_oem_check
if (uv_hub_info->hub_revision == 0)
goto badbios;
 
+   switch (uv_hub_info->hub_revision) {
+   case UV4_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x11;
+   break;
+
+   case UV3_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x9;
+   break;
+
+   case UV2_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x5;
+   break;
+
+   case UV1_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x3;
+   break;
+   }
+
pnodeid = early_get_pnodeid();
early_get_apic_socketid_shift();
 
@@ -359,6 +378,12 @@ int is_uv_system(void)
 }
 EXPORT_SYMBOL_GPL(is_uv_system);
 
+int is_uv_hubbed(int uvtype)
+{
+   return (uv_hubbed_system & uvtype);
+}
+EXPORT_SYMBOL_GPL(is_uv_hubbed);
+
 int is_uv_hubless(int uvtype)
 {
return (uv_hubless_system & uvtype);
@@ -1457,6 +1482,68 @@ static void __init build_socket_tables(v
}
 }
 
+/* Setup user proc fs files */
+static int proc_hubbed_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubbed_system);
+   return 0;
+}
+
+static int proc_hubless_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubless_system);
+   return 0;
+}
+
+static int proc_oemid_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "%s/%s\n", oem_id, oem_table_id);
+   return 0;
+}
+
+static int proc_hubbed_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubbed_show, (void *)NULL);
+}
+
+static int proc_hubless_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubless_show, (void *)NULL);
+}
+
+static int proc_oemid_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_oemid_show, (void *)NULL);
+}
+
+/* (struct is "non-const" as open function is set at runtime) */
+static struct file_operations proc_version_fops = {
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static const struct file_operations proc_oemid_fops = {
+   .open   = proc_oemid_open,
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static __init void uv_setup_proc_files(int hubless)
+{
+   struct proc_dir_entry *pde;
+   char *name = hubless ? "hubless" : "hubbed";
+
+   pde = 

[PATCH V2 5/8] x86/platform/uv: Add UV Hubbed/Hubless Proc FS Files

2019-09-09 Thread Mike Travis
Indicate to UV user utilities that UV hubless support is available on
this system via the existing /proc infterface.  The current interface is
maintained with the addition of new /proc leaves ("hubbed", "hubless",
and "oemid") that contain the specific type of UV arch this one is.

Signed-off-by: Mike Travis 
Reviewed-by: Steve Wahl 
Reviewed-by: Dimitri Sivanich 
To: Thomas Gleixner 
To: Ingo Molnar 
To: H. Peter Anvin 
To: Andrew Morton 
To: Borislav Petkov 
To: Christoph Hellwig 
To: Sasha Levin 
Cc: Dimitri Sivanich 
Cc: Russ Anderson 
Cc: Hedi Berriche 
Cc: Steve Wahl 
Cc: Justin Ernst 
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
V2: Remove is_uv_hubbed define
Remove leading '_' from _is_uv_hubbed
---
 arch/x86/include/asm/uv/uv.h   |4 +
 arch/x86/kernel/apic/x2apic_uv_x.c |   93 -
 2 files changed, 96 insertions(+), 1 deletion(-)

--- linux.orig/arch/x86/include/asm/uv/uv.h
+++ linux/arch/x86/include/asm/uv/uv.h
@@ -12,6 +12,8 @@ struct mm_struct;
 #ifdef CONFIG_X86_UV
 #include 
 
+#defineUV_PROC_NODE"sgi_uv"
+
 static inline int uv(int uvtype)
 {
/* uv(0) is "any" */
@@ -28,6 +30,7 @@ static inline bool is_early_uv_system(vo
return uv_systab_phys && uv_systab_phys != EFI_INVALID_TABLE_ADDR;
 }
 extern int is_uv_system(void);
+extern int is_uv_hubbed(int uvtype);
 extern int is_uv_hubless(int uvtype);
 extern void uv_cpu_init(void);
 extern void uv_nmi_init(void);
@@ -40,6 +43,7 @@ extern const struct cpumask *uv_flush_tl
 static inline enum uv_system_type get_uv_system_type(void) { return UV_NONE; }
 static inline bool is_early_uv_system(void){ return 0; }
 static inline int is_uv_system(void)   { return 0; }
+static inline int is_uv_hubbed(int uv) { return 0; }
 static inline int is_uv_hubless(int uv) { return 0; }
 static inline void uv_cpu_init(void)   { }
 static inline void uv_system_init(void){ }
--- linux.orig/arch/x86/kernel/apic/x2apic_uv_x.c
+++ linux/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -26,6 +26,7 @@
 static DEFINE_PER_CPU(int, x2apic_extra_bits);
 
 static enum uv_system_type uv_system_type;
+static int uv_hubbed_system;
 static int uv_hubless_system;
 static u64 gru_start_paddr, gru_end_paddr;
 static u64 gru_dist_base, gru_first_node_paddr = -1LL, 
gru_last_node_paddr;
@@ -309,6 +310,24 @@ static int __init uv_acpi_madt_oem_check
if (uv_hub_info->hub_revision == 0)
goto badbios;
 
+   switch (uv_hub_info->hub_revision) {
+   case UV4_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x11;
+   break;
+
+   case UV3_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x9;
+   break;
+
+   case UV2_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x5;
+   break;
+
+   case UV1_HUB_REVISION_BASE:
+   uv_hubbed_system = 0x3;
+   break;
+   }
+
pnodeid = early_get_pnodeid();
early_get_apic_socketid_shift();
 
@@ -359,6 +378,12 @@ int is_uv_system(void)
 }
 EXPORT_SYMBOL_GPL(is_uv_system);
 
+int is_uv_hubbed(int uvtype)
+{
+   return (uv_hubbed_system & uvtype);
+}
+EXPORT_SYMBOL_GPL(is_uv_hubbed);
+
 int is_uv_hubless(int uvtype)
 {
return (uv_hubless_system & uvtype);
@@ -1457,6 +1482,68 @@ static void __init build_socket_tables(v
}
 }
 
+/* Setup user proc fs files */
+static int proc_hubbed_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubbed_system);
+   return 0;
+}
+
+static int proc_hubless_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "0x%x\n", uv_hubless_system);
+   return 0;
+}
+
+static int proc_oemid_show(struct seq_file *file, void *data)
+{
+   seq_printf(file, "%s/%s\n", oem_id, oem_table_id);
+   return 0;
+}
+
+static int proc_hubbed_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubbed_show, (void *)NULL);
+}
+
+static int proc_hubless_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_hubless_show, (void *)NULL);
+}
+
+static int proc_oemid_open(struct inode *inode, struct file *file)
+{
+   return single_open(file, proc_oemid_show, (void *)NULL);
+}
+
+/* (struct is "non-const" as open function is set at runtime) */
+static struct file_operations proc_version_fops = {
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static const struct file_operations proc_oemid_fops = {
+   .open   = proc_oemid_open,
+   .read   = seq_read,
+   .llseek = seq_lseek,
+   .release= single_release,
+};
+
+static __init void uv_setup_proc_files(int hubless)
+{
+   struct proc_dir_entry *pde;
+   char *name = hubless ? "hubless" : "hubbed";
+
+   pde =