Message-ID: <8700d72f-a995-8ab5-45a5-cc08e34e57f6@intel.com>
Date:   Wed, 1 Mar 2023 17:36:44 -0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.11.0
Subject: Re: [PATCH v4 5/6] iommu/ioasid: Rename INVALID_IOASID
Content-Language: en-US
To:     Jacob Pan <jacob.jun.pan@linux.intel.com>,
        LKML <linux-kernel@vger.kernel.org>, <iommu@lists.linux.dev>,
        Jason Gunthorpe <jgg@nvidia.com>,
        Lu Baolu <baolu.lu@linux.intel.com>,
        Joerg Roedel <joro@8bytes.org>,
        Jean-Philippe Brucker <jean-philippe@linaro.com>,
        "Dave Hansen" <dave.hansen@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        "X86 Kernel" <x86@kernel.org>, <bp@alien8.de>,
        "H. Peter Anvin" <hpa@zytor.com>,
        Peter Zijlstra <peterz@infradead.org>, <corbet@lwn.net>,
        <vkoul@kernel.org>, <dmaengine@vger.kernel.org>,
        <linux-doc@vger.kernel.org>
CC:     Robin Murphy <robin.murphy@arm.com>, Will Deacon <will@kernel.org>,
        "David Woodhouse" <dwmw2@infradead.org>,
        Raj Ashok <ashok.raj@intel.com>,
        "Tian, Kevin" <kevin.tian@intel.com>, Yi Liu <yi.l.liu@intel.com>,
        Dave Jiang <dave.jiang@intel.com>,
        Kirill Shutemov <kirill.shutemov@linux.intel.com>
References: <20230301235646.2692846-1-jacob.jun.pan@linux.intel.com>
 <20230301235646.2692846-6-jacob.jun.pan@linux.intel.com>
From:   Fenghua Yu <fenghua.yu@intel.com>
In-Reply-To: <20230301235646.2692846-6-jacob.jun.pan@linux.intel.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bHBpdStmYVhPQjd1cldWM0NEVnpaVzF5aGRnRHNGR0pCWGlpUy8xTHE4cTUy?=
 =?utf-8?B?bVFtQks1MTJ3NEsyRjA0UE5uSUVvMTNUcmkya0VXTnlUVjlkaDdFTkp3WU54?=
 =?utf-8?B?UXJOcTI4N2VzaUU2SkRWb0RnYlhXN3RzbEJhdGFydFVOSGZ1ZTFvb0NvU2pP?=
 =?utf-8?B?VXhPYmhSSjRVTS9wUjZKU0JSN25JdWJYdzhIdmZhU05MQlZicW9UanB6RlY5?=
 =?utf-8?B?cnI1alpuM25KQWtjRUJRb1A5ejByN3BZT3o1ZXFGSGZ1YktYTEFPL0tVSGYy?=
 =?utf-8?B?VUphbWFlQVFZTnNvbU82d3I4T3g0U1d6YTNuWHhKZ0hoZjYzZzIxQkVwZmk1?=
 =?utf-8?B?NUZtYityRUhCSGpnZTVnQ3ZwRXhrYTVDU3ZqelB3UHJEVVQ2dThESkkwNXRZ?=
 =?utf-8?B?VEdaZHp0Rk5Ga1lnMXJDN0VSNU1lcktZY0VCOVh2UDdDTjlZNUsyWVhjcnZG?=
 =?utf-8?B?QWVnbU1qTkpkWEpwbHFDZ2hGZkhHdmsrTXg4bjZleVo1N0VMWXJ0MUxXYmdp?=
 =?utf-8?B?ZUJhQkNnV0pLMDg0amNKQVlXU0JvZGlPZHNEQW12Z1phK0F5L1pUV0diWHpv?=
 =?utf-8?B?TE1wOUprVm9iMXFBdXc3WE45VStHVmNFOE5jWGZNQ2doTGpTL0JYdGVmYllJ?=
 =?utf-8?B?MnJMVDlDRWxOT3lMZCtCSDVWTElPdys4N3RDZmY5bXBVY25BYnd5YnNsVWJh?=
 =?utf-8?B?NnlqeGdBWUQ3cytDRnU0OVJDWnlkanFLZlI4ODQzS2dZaXRhVGk2eGYvTHRM?=
 =?utf-8?B?c2NTc00zS25CVGEvUy9nMG5lU0ZjQWd3TGNtUTIzcHQ1TktZMWkrU3FSVTFH?=
 =?utf-8?B?N0hwbUVyL2c5VzMxemRWaTgvNEdnVjRtRGpVR3dxREhhbWhyMm1jUnJRbzJr?=
 =?utf-8?B?Y3N0TVF1ZUJQNGVMUWU5aEhWM0QyUU91N0pDMzFhM2JnZncybHlMMWZRdnNw?=
 =?utf-8?B?dXM2bHBLdmlJb2dnZk9ER0dqRFBidkpiQzhISkpPbmwzdzd4cHNXZmFTQjRS?=
 =?utf-8?B?NmxZN05ZR0pMTFFrUklkMnVkQmZ1T2YrM2loWEZpNzNNejBJN3RwS1l4WFFS?=
 =?utf-8?B?RmZ0S1pmdENPQ0JzTGpTek42ZXUrZUxTelhrRjhiaTFkK09GZCtSOHJoS2ZI?=
 =?utf-8?B?OWtTM3R5cnlkWUdSL2ZZYWNjN3RnY0dKQldCeE03azQ0bnVCcnFjalBPaDBr?=
 =?utf-8?B?eUwvS0k4a0ZBMjltQjNpQlRIY0VxVkRLMDg4cnAvRGpFK3pESUZCZVBtVDJL?=
 =?utf-8?B?YjBaci9KWE0zTCtWRURuRlJZeC9kb1A1TTRNM2s1cUVvbEpjcnhsVXpPTnpS?=
 =?utf-8?B?Z3VDMjg3bHBUUzQvSWx0dmpHTnhoVm1BREtZNnRqRFl5ZTRYaUVMdFQydUM3?=
 =?utf-8?B?K21CUVU4RHdzeDVtanZkMitQaHYzK3NDUkl6MWFmdFdycWVnT1dYbHFEeVNQ?=
 =?utf-8?B?MGdycXdLNlY5eDlKc3FTdXpsUHZraWY5aytkc0o0cVQyWGJpcEwwQzRJRG9k?=
 =?utf-8?B?WmJjYlZncnNtSU5LS2U1MnRram5saFM0YjFtNEs1Yk1aeFprL2IxNFA2YnRu?=
 =?utf-8?B?cFEwbHQ4VjJaMXI2SXJkTWt5bnNOQkNyR1ZzZmpmT2dwQ1B3MTVLaEhqZTJE?=
 =?utf-8?B?WjJFdEJaQTd3NGl3ajFZY3k2YUZ5Ni9uOWFhYnlwZWZNZ3pkTEYwV0xNZjdW?=
 =?utf-8?B?U0phcGVDOFZsTkMrdDlRdU92QXZVS1dDNVRKVWgxdFVpMlJjMWFxVG96KzFv?=
 =?utf-8?B?S1ByNXVGK2hjWW1YKzBMUzFLV3pxa1FVRFBpOTdRajVDQWg5N3pScElwKzNC?=
 =?utf-8?B?RUFmdEJySTMwZFc5aXBkN1Zva3FkY1ltTk1ES1FCVXJzS2NQS1A2ck1LcTVB?=
 =?utf-8?B?ZTJYWWZVRTNaejg0dUVjWjNnRFpoSkEvUGZMOXhmdkhGK2JEZTAwTlRmblI3?=
 =?utf-8?B?T0RvaUhubjdIZXYrdDd6M0NTenB0SnpLVVhFN3YwS01NbGdDZGlzOE4xOEor?=
 =?utf-8?B?dS93YXFxU2k1ZndoeXRFby82VFI0QmlHdjNmdmxEQVlDbE9MNEhNS0VNYllB?=
 =?utf-8?B?N0hOY2s5c2duWm1VWEVTclVIM0dTTTNyN1plVWhidFRXUUVQY2U5V0ZtSjls?=
 =?utf-8?Q?knyruYgbvNrkRrRrzRoq5zgd4?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4008490c-af4b-4efe-4ab9-08db1abe96fb
X-MS-Exchange-CrossTenant-AuthSource: IA1PR11MB6097.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Mar 2023 01:36:48.5965
 (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: GAvc0kf1lTQpWyWoIQ6oJQxMlUuRi3sUcOU3WIGahbhunI2/KbdvWKuGHhX8HRx3VXAjp1U4biBH83BDGnJclw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5641
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 3/1/23 15:56, Jacob Pan wrote:
> INVALID_IOASID and IOMMU_PASID_INVALID are duplicated. Rename
> INVALID_IOASID and consolidate since we are moving away from IOASID
> infrastructure.
> 
> Reviewed-by: Dave Jiang <dave.jiang@intel.com>
> Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>

Reviewed-by: Fenghua Yu <fenghua.yu@intel.com>

Thanks.

-Fenghua