From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3vSQ1B3TzJzDqBV for ; Wed, 22 Feb 2017 02:49:50 +1100 (AEDT) Received: from pps.filterd (m0098396.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v1LFnfqc084050 for ; Tue, 21 Feb 2017 10:49:48 -0500 Received: from e34.co.us.ibm.com (e34.co.us.ibm.com [32.97.110.152]) by mx0a-001b2d01.pphosted.com with ESMTP id 28rpx1w9gt-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Tue, 21 Feb 2017 10:49:44 -0500 Received: from localhost by e34.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 21 Feb 2017 08:49:15 -0700 Subject: Re: [PATCH] powerpc/pseries: advertise Hot Plug Event support to firmware To: Michael Roth , linuxppc-dev@lists.ozlabs.org References: <1487639538-25922-1-git-send-email-mdroth@linux.vnet.ibm.com> Cc: David Gibson From: Nathan Fontenot Date: Tue, 21 Feb 2017 09:49:11 -0600 MIME-Version: 1.0 In-Reply-To: <1487639538-25922-1-git-send-email-mdroth@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252 Message-Id: List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 02/20/2017 07:12 PM, Michael Roth wrote: > With the inclusion of: > > powerpc/pseries: Implement indexed-count hotplug memory remove > powerpc/pseries: Implement indexed-count hotplug memory add > > we now have complete handling of the RTAS hotplug event format > as described by PAPR via ACR "PAPR Changes for Hotplug RTAS Events". > > This capability is indicated by byte 6, bit 5 of architecture > option vector 5, and allows for greater control over cpu/memory/pci > hot plug/unplug operations. > > Existing pseries kernels will utilize this capability based on the > existence of the /event-sources/hot-plug-events DT property, so we > only need to advertise it via CAS and do not need a corresponding > FW_FEATURE_* value to test for. > > Cc: Michael Ellerman > Cc: Nathan Fontenot > Cc: David Gibson > Signed-off-by: Michael Roth Reviewed-by: Nathan Fontenot > --- > arch/powerpc/include/asm/prom.h | 1 + > arch/powerpc/kernel/prom_init.c | 2 +- > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/powerpc/include/asm/prom.h b/arch/powerpc/include/asm/prom.h > index 2c8001c..4a90634 100644 > --- a/arch/powerpc/include/asm/prom.h > +++ b/arch/powerpc/include/asm/prom.h > @@ -153,6 +153,7 @@ struct of_drconf_cell { > #define OV5_XCMO 0x0440 /* Page Coalescing */ > #define OV5_TYPE1_AFFINITY 0x0580 /* Type 1 NUMA affinity */ > #define OV5_PRRN 0x0540 /* Platform Resource Reassignment */ > +#define OV5_HP_EVT 0x0604 /* Hot Plug Event support */ > #define OV5_RESIZE_HPT 0x0601 /* Hash Page Table resizing */ > #define OV5_PFO_HW_RNG 0x1180 /* PFO Random Number Generator */ > #define OV5_PFO_HW_842 0x1140 /* PFO Compression Accelerator */ > diff --git a/arch/powerpc/kernel/prom_init.c b/arch/powerpc/kernel/prom_init.c > index f3c8799..1a835e7 100644 > --- a/arch/powerpc/kernel/prom_init.c > +++ b/arch/powerpc/kernel/prom_init.c > @@ -839,7 +839,7 @@ struct ibm_arch_vec __cacheline_aligned ibm_architecture_vec = { > 0, > #endif > .associativity = OV5_FEAT(OV5_TYPE1_AFFINITY) | OV5_FEAT(OV5_PRRN), > - .bin_opts = OV5_FEAT(OV5_RESIZE_HPT), > + .bin_opts = OV5_FEAT(OV5_RESIZE_HPT) | OV5_FEAT(OV5_HP_EVT), > .micro_checkpoint = 0, > .reserved0 = 0, > .max_cpus = cpu_to_be32(NR_CPUS), /* number of cores supported */ >