From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f53.google.com (mail-lf1-f53.google.com [209.85.167.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5A98E41C73 for ; Wed, 15 May 2024 08:11:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715760692; cv=none; b=LHUhWSUnVhlBCrhRdTizljUkX0OKmMnCz3e99zv/veQDc6gHaNa8Cyfvou5+ba9g6nXLNBnxohhc5dRblSOVCYghqL/V2W/JcVozlGoz9ICaGJiQs9RlkDStumt8+W14ZiXMRGVPK6UC9S3Es7ZA4SJBxf8SxTxOTJF73OTPnJQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715760692; c=relaxed/simple; bh=wgLVDZfo7OUNUCa/aFvVw7eI10FXyDQIuqG4Ml5oaIg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=NC6Pe0IJe7+rVBMAyh/xHYvJWV6Bk33XTRVgvuklFbPc6EgS/ja6lnd+phO6BdqfSygzL3+CxcBMnYIXSg2cH17fiSrx81oZbucOhXYUC2pAye0hSp8kUXckj5FX3Vu0dilKRnfZP/kVlPNKOQLoSY/bQXadMvJi836j1HjnM4g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cryptogams.org; spf=pass smtp.mailfrom=cryptogams.org; dkim=pass (2048-bit key) header.d=cryptogams.org header.i=@cryptogams.org header.b=hqJCZA4D; arc=none smtp.client-ip=209.85.167.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=cryptogams.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cryptogams.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=cryptogams.org header.i=@cryptogams.org header.b="hqJCZA4D" Received: by mail-lf1-f53.google.com with SMTP id 2adb3069b0e04-51fdc9af005so10543181e87.3 for ; Wed, 15 May 2024 01:11:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cryptogams.org; s=gmail; t=1715760687; x=1716365487; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Nadn3nJhbpbUcGRwW1AG+AKX8Tqss/4B5lqeqiiCsI4=; b=hqJCZA4DdZA/CwAcI7ltBYUJyTvK1J/SEZbazsgOzYKYjDUlS+OUbPPhKqMpDiIgef IGKgkbgq8EVhlWqMGBWnXx5mh9GXVazftKK+hVplkKYaRzgViruau5k00yUvDFiunAdk R4mASkiFQ/qz3W8Y1kS2J++GjSWQy8p+IlgVN11rCcjM0/ztUWJGnT9wVgyIbknsLcM7 M8CUBm3LxXSCX94W6gJT3a82BSK1fp5foSDgCcJCmHPyrdSZqJnoQiATlvYk3dptUvpK 0131mV52A34vO26Lp8ZHFXXrbm/+o+uY98syQzXW6xFxpVNWj4iWYS9Z8ECP/S391oKZ I9Hw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715760687; x=1716365487; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Nadn3nJhbpbUcGRwW1AG+AKX8Tqss/4B5lqeqiiCsI4=; b=gHAkg2UPVMUGvQ+UPbAaAfd1mgk+PgInpnyC92xaoak3xXNRdCiuswS93dlAKulphv AYtjO9v5fvH8z/tLe1cAGerxstxyBtl5qW3DlfOyRQBzV9x4MZhon3YIc3gDQfq/cyYg rRuqkZ/ezD31NWrhJtlPGcD+wAGwa84X9pqc+ju9pe8YyAIa77NVyXi3w5uIrH3dW8pL S4sor8C/fSjAdYCSnsseTIOv/wDdCOzeMcEyKEw5mXXPV5Jx7YVmYEao86KVXqd+F7YC 5M4xUJx5OPyzFFlAWjdzCdU7JBINbTtg7ft5z5vIZaaY4K4tuRoIgz6ra2DOy4NjAawB ipgQ== X-Forwarded-Encrypted: i=1; AJvYcCW17YEoZg3epU4FwKrNDod5MvrE8Ng01kWtqJ3nHEonoEOTThvQRDjBtc+S5B3FbvikrD6pXhcizXemfXNfcBcnWKuWgQmtx0lzLngk X-Gm-Message-State: AOJu0YwlbkkFiHE1VvF4BvH24QuGjnZoelU34mF0z8RnF9XdH1WP2Oo8 JCzNZSKUOvH/2c4wkZtdkiOyk5LlodLKaPvwFaJPTslFTNk+Yl6+gg9RAtQ0wrQKTszoJGAeZ2e Z X-Google-Smtp-Source: AGHT+IEm4tt3o20ljc2e1O4yFuBQYTb/i4s4EPsBk0XZkzFkv52SwrrVPTrrVXZR+8C3wJP6qscL5Q== X-Received: by 2002:a05:6512:61:b0:51c:5570:f570 with SMTP id 2adb3069b0e04-52210473d4fmr11529720e87.59.1715760687525; Wed, 15 May 2024 01:11:27 -0700 (PDT) Received: from [10.0.1.129] (c-922370d5.012-252-67626723.bbcust.telenor.se. [213.112.35.146]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-521f35ba8e6sm2400086e87.108.2024.05.15.01.11.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 15 May 2024 01:11:27 -0700 (PDT) Message-ID: Date: Wed, 15 May 2024 10:11:26 +0200 Precedence: bulk X-Mailing-List: linux-crypto@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/3] crypto: X25519 low-level primitives for ppc64le. To: Danny Tsen , linux-crypto@vger.kernel.org Cc: herbert@gondor.apana.org.au, leitao@debian.org, nayna@linux.ibm.com, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, mpe@ellerman.id.au, ltcgcw@linux.vnet.ibm.com, dtsen@us.ibm.com References: <20240514173835.4814-1-dtsen@linux.ibm.com> <20240514173835.4814-2-dtsen@linux.ibm.com> Content-Language: en-US From: Andy Polyakov In-Reply-To: <20240514173835.4814-2-dtsen@linux.ibm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi, Couple of remarks inline. > +# [1] https://www.openssl.org/~appro/cryptogams/ https://github.com/dot-asm/cryptogams/ is arguably better reference. > +SYM_FUNC_START(x25519_fe51_mul) > +.align 5 The goal is to align the label, not the first instruction after the directive. It's not a problem in this spot, in the beginning of the module that is, but further below it's likely to inject redundant nops between the label and meaningful code. But since the directive in question is not position-sensitive one can resolve this by changing the order of the directive and the SYM_FUNC_START macro. Cheers.