Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <7daa7a8d-f125-5d86-6537-68551319157c@intel.com>
Date:   Thu, 8 Sep 2022 15:43:20 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.2.1
Subject: Re: [PATCH v2 2/5] selftests/sgx: Move ENCL_HEAP_SIZE_DEFAULT to
 main.c
Content-Language: en-US
To:     Jarkko Sakkinen <jarkko@kernel.org>, <linux-sgx@vger.kernel.org>
CC:     Haitao Huang <haitao.huang@linux.intel.com>,
        Vijay Dhanraj <vijay.dhanraj@intel.com>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "Shuah Khan" <shuah@kernel.org>,
        "open list:KERNEL SELFTEST FRAMEWORK" 
        <linux-kselftest@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>
References: <20220905020411.17290-1-jarkko@kernel.org>
 <20220905020411.17290-3-jarkko@kernel.org>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20220905020411.17290-3-jarkko@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RUlXeEhvWTZET0hneDdtUzFUZFQ1ai82bnlndUxLY1ErS2ZPQmRva3k4Wkp6?=
 =?utf-8?B?M0JFN2psQWpkNmYybVJVYXhBaXFkRVk4VmR4MDRiQ21reVBFZm9sNnd2TFVH?=
 =?utf-8?B?S2VFK2xhUFBLaDNuTDFNUitHWXFDOTBIS1k1TWxBVUFDQkZRekdNdTd6ZlpV?=
 =?utf-8?B?b2F1SklaV2tmYWZGbitZejJMQzZaRGN0ekFWTS84UjJDT0tDVzdyRldDT21n?=
 =?utf-8?B?b3pTRzlsTittV0ZuTHNaNHhXdmRCVXEzbXZBRHZheURFMU1IRnFsTEk1Vkpz?=
 =?utf-8?B?TUpXV2p0K1g1U2ZJT21YcDNMT1hFRkZSNEk2NGF6Y0NkbHBhblVjVFZQNlYr?=
 =?utf-8?B?Nk1ERmNjZWk4UkEwMEtaamdiVm9RMUhZVkpSZ05qeEFleC9vN1U0TzFxQ3pX?=
 =?utf-8?B?RXREZDBrRmRLQUI3dXpNbi9CR2JNcGxSemlNVzFPNk5pVlF0SzI3cExoT3ky?=
 =?utf-8?B?a2VwNURkWVladGhoNXduUSs2bGh4ZkNhbGVKd2l5eHpPY3ZIRExCQmlyT3Bl?=
 =?utf-8?B?ZkNlamlwRExVdVRuOWxiV21OOWNYQjV6NjZITUdWZ3R6N2ZxMTNubldNUzhJ?=
 =?utf-8?B?T0J3RVZuSTN3UmFOa3p5anI3cjNhejFKUlpYZFcwRkR6cW9FK1hDbnFKdUlQ?=
 =?utf-8?B?Rm8wSUtpLzlvOEd5T09vVnhvWkRSN0pZZTVadytzMzVwb3UwUXRZMG83VHFX?=
 =?utf-8?B?M3poOTgzUU1yUEd6TkppdFhNeWpHa2UyOC9zQUJkSlZHdS9hR0Q0U2RpaXR1?=
 =?utf-8?B?L3FGaWZPV1k0TUdQK1Z4bWQxRS9tNllpZHBzMjVydmF4MUo4RTFEdGxBYlV1?=
 =?utf-8?B?WUxFT0oxVlZheVQ2bjE4cDN6RjFDS0lETklDV1ZMRWlhVXZVQU1nWEtEaVQr?=
 =?utf-8?B?VnBwb2MwRHZYTmVRaGIzajZpS0RuOHEyYVRibHFZcEdsZ0hFcVNIN2hYZWNG?=
 =?utf-8?B?Y1Q1SW00NU9HYmpGMHFteWhBbVRFdkpoREZjQnJQRE9EbmtRaEVUOTNDbFkr?=
 =?utf-8?B?ODluMFo0OGNCcER2UHBSQVpPTXZObHgvQWltRG9qTmtSSmZDME45NmdDUjE5?=
 =?utf-8?B?MnZhVUVPcmJxbHhDRDBpNlF4eU9ONnNDaDVRSkZUUFQzcUhRcGxzYmxvVko5?=
 =?utf-8?B?NE5udTVxODNGWHRUUG4wcGVycWtuZXBCbG1ybDVvcmRad292TGNrSHYwVHZW?=
 =?utf-8?B?VmRBKzduNVJkNTk4d2V5bEpQbEZhakhLSzlab0JxOWlibE1qRDJTejVxbU9Z?=
 =?utf-8?B?VjJpelFFNjZJcEFsdkc5bVR1ZWhDMGJadERub1FYS2FTYTZrRGNmWVF4cUQ2?=
 =?utf-8?B?S3hKYUlIRHpUKzM2bEJiSEJoMXN0eWYxZjVJbGFuTUE0YVdVNDIyTEhYZnlO?=
 =?utf-8?B?UnhXQ2NKMmlHM2MyOHpwNUY2dDhkZTM5QnpSM1B5NElyS1l4R3pJbTljelBN?=
 =?utf-8?B?VUFPbEZ5bmw3WXRidXdrYzVXNjRLNzNYc0QydmN4NGRBSWk2eWI4dmE2dVdO?=
 =?utf-8?B?c3hzSjc5MWJMdWVoNlNJeFNoZ3YrNC9PT3Y5U2FEODJrZzFSYnJid3p2cWVq?=
 =?utf-8?B?N1dpNHcvaGkxcVJZYjlLQVRjOTFnbWo4WFhrNnM2OFZ3QXV3NXdVQmttVEp5?=
 =?utf-8?B?TmdldWlBZ2dMMTN0RC9SVWZNMERyWklvN05qWCtaTzdZaXlIYXE4WDFmd2dv?=
 =?utf-8?B?anlVSHVkeTIvM2NnSERrTWdidUpqUUxKRWppYVlqMi8yVUpoRWx5eUdaMnVN?=
 =?utf-8?B?b3hLWXE0ejZIdE1rZWR4V3RNajhLUktMUkpWNmZUbDJlTVF1dHJwbzRjVmRM?=
 =?utf-8?B?VmdrZEZrZHJUZW90T3Z0Z0dkSzl4SU0yMTlZNm9Ma2dwNFVxbkx5Tm5nZzlp?=
 =?utf-8?B?cDN2Z0hOdFNHVUdFR04xQmxaOXoyQW1CS1B0VXM2UU9aRWdDbHhYVzl4QVV3?=
 =?utf-8?B?amxyRVQxRWtkREN6NnQ3UmkxK3ZUd2dwcEJ5TDBuZVdacmNxcFFnM0psWmRL?=
 =?utf-8?B?cDBLTGliWkdBazFWTnBtd291UnB0YjBwZUFoWmRwU3hUVndIakNDMW9WOTlv?=
 =?utf-8?B?UUVDT2YrRElLeExFSWt4MmppaTJLSHBMblZwTkRGRTlVRnJJVDBGNURaVndS?=
 =?utf-8?B?R09TM3A5S0cyYm5vb0pVOTFRV0MwYVJjYzQzY00zRkJJNklERXBweUlMdFhG?=
 =?utf-8?B?QWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 02118759-9fb7-4f00-c7fd-08da91eb8888
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Sep 2022 22:43:22.2228
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wtdz3X0+XjkZRzBNhYXd7fawa0EJQWhs4m+xNrKoCrDzdArwpnMieeFqPzjucbXXQtDq/qlXPBnqFOI1jzFo8rsVS/2ris4m/P0Ex/Cghuk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4075
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 9/4/2022 7:04 PM, Jarkko Sakkinen wrote:
> Move ENCL_HEAP_SIZE_DEFAULT to main.c because all the other constants
> are also there, and it is only used locally there.
> 
> Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>

Acked-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette