[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 7/9] gnttab: no need to translate handle for gnttab_get_status_frames()


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • Date: Fri, 7 Oct 2022 18:24:23 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FiRCk3Y/NObYQimek7mR4D1ZlH2EpOD+gibFR2m8lis=; b=j4PFLe1rh4GO4FNS+9Ld10WwAAoTzHq7HTRk+2idPSI+561DGJeUP2Yk94b4xMB3Q1MF8+b2RNflZsT90dxQiB4ZO8nwCsxNs8jDfBCNeAuJ4GfvyXDHegv7vePW2CGofW16wJVXjA20w7Ap6lDWN42iHPvuvC1mSHPBnYk9z/tqFj3Rm9z9Ag38S3G0/DWKoDJSETYWTe691ULoAdBN0iagEcNThxybCWgR1YGqsaEDupKr/d9TCNVJAqWZ0u+xiTxP2l9OXLo8oV5fa2CvZYABUu2LGMVOcy7JHmWq4+weNLZyWbwj5mgpIcCKEAyygUYjtpQPk9qohqE+ZrhImQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=B/HYTodbG6cL9pE3tLtSWeAH/m77vd9gj+Nk16AnQZO3/X9yK9Aw5FazJqMTn2r9dfdnkFccE8flq3f5rWU9D3QMJVYyAelR0mRdCevLDwuezMW2Tde6mR0ZObxl0+7C2TNVEmEB8EhuVX2t6bEDl71VRnj4b0POXUiwfNiZfcbwcF/jKXlBJ6j2wdu4BavbghqVae97zaPIS772CNRO6wsAbIH0ozN70Uuf8biHMMkFZw/7zAzND5WKqwvgx0bxjhWZN0+LuTi+JmUCJyY4AetlRL3S0ibTyBDN/X4AdI9ORXI7bwRjH+SmidKmeqQ+TAcE0p52zdKYSdIB4EB/2A==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: George Dunlap <George.Dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 07 Oct 2022 18:24:37 +0000
  • Ironport-data: A9a23:oJQ/UqykBx2GsqYRQON6t4n5zX0fS1M+ZkaijkTw14qbGYmMLcPOrxiLQjTXvW5WxI8XJcG4SpgwjnZAGzTh9pmuRSG8MG3qtOxfgcBBnHBoNelWH2QmE+eZMoxCFCeqqBMByRJRY04IJcp5V3XXbqvlQFmpiBbEbdW9en4LWj7smpYBGba0kFjq/Wp8AHgmszjG/sgnm/LezZO/HL1ujS/Jy0FPFlyiifuutNrbikIgvHESZZ0YLKRHX4K7Qk832uHIYpqDB6M/4VriAa+G6ea3XXI/zRdZJ2Fkg8ZKvrBtPNKxWUip9q0CFvERvgL714PwVQTyjjBpWA9XVfHH8xZo6viNvLFJAybQ3fv8EaP8U+APlGvluvjE5b2ieAp6WhW2lxGS9+pLPh7Xw7HfeYzYuXgz7lbKCjVHdF1dNdkSkwGGe2pCATlVRSv4CkCsaEuqhCkS1fAEuNsLPYQ/V31YECmMxsSm3j6CQWqfQ8aYnM50vSD1OpS11it6VHWpjbY9PxD1jPuOknwckZnztARLQD6VtgBhJJF+axBM8OR40Th7xpQH4NT/SyqDBNQQaKD1xkHSj6jMQrSPwhFgO0LtNdD3lZuiyd5w4/aVqJwjCp2NrJ6L/ghj5PgXeAygI4+OmOL8RPQ6lHewgAkzUYx9wY5dWWeI+6uz0cqklum/gZB/nxpC1QS/EvwJD7J18C50xuvS/xFFOB/czcnu++R3NZ0XrP3L/qyLxriWfD5OOHKgNEaKcZHYF0k2WUufngtT709AcFBGfpCh5Bd8WQdH6yhhBMXcdWldRlE4KGsvGSEWxGXBlSv0DElRQTP39e9TBPmC2rDWyBTb8JaCpD3I9hIkogF+Qh7ZmzneC2loXNA3oG/l+YnWGjKzaV56qThSIbZKm0jCuqBIyC38pWERrZnM24RBonzNlohA1HQ7phtOcBwOmNIA8EuLuwZzp6pAwoPFAfEJC+4m2VZVPxfPZlrMKT6d5BX00xEpn/5xRT1VAWSbtphsELuiyNCFUEHMmLuAAjWOickA2wgZ7tQw0nTk2PTXLphkIhVUbChRwbNh/fm1ppBDLLhEnr9IzlYfE9FJjAPE3QmFFhJZBntgaM59soVlTwrreXxEZ/YQMRITkWL3fUq8TrZOkvTh2YnAuUpU7Qx/IWrPmJhBuaedymGMZDCi6pF1UGoZUhXEB1rUkwALllr674VKL12ap7mB/I9dOjjwmuxlQh5ubjfvQDqhaEDYNoifAeWGiCJlIhVVdF1pPvUfClGzV9z/YKOx2M6FLxMvxhLy3+JW4bqqWy+qltlqvtAkdeoVXGtxQmrD/FfIikR2bNWUpkvel6OlX+gcGJ24xe6ktxTTt+DeWfg08JPYy5ZqDZxuIfMEtCYWBrHrHAAyPvvTicWccYat2zJ2C34GgACxpUGlNHztXxPelviZjR54NdyxooR7ognlhT2DVCNiArNnsRxzMakIagB5/WMUX4Yxzoq74EhRlye4PVSbgzUiyZMFJc1qxfXtwPpX36fnN00eAyQsKI5lauqNGPIIYbG+cJJJXXCj4f6IBcfu40qZjpz8UZfKsJ9T6er7KIy6Jb7vOjLZ6riPXQu+E0/lZtii83+9gtRbC3zZRPPf/h6yyVj6k8WcGv/Oh6L0kFyvAUCa5T3XzrYc80fUnw3lMRYwXVcoIQrvPzdrZhizppEXFUTNlMI0mO9TIC42HLJjZJezaP2r+G2BjhvhxEo+byhnbc78QYgQ6CM9FZ0FPJMC6ZENe1Q+YGj+n084KpfCMubK2z3UsTiwSML9klv9gmGLSshRs7ityrfcKPmSCctl8eMzdWrCsweU2kbmXZCtFYLM925ebCOg1jKspdL0MYwBXKc1riytVOdE0xUl3AOLYnjju8Y8v1YCOVeYjR6wvm5RCWNR2V3AvExhugGWt4lXRzkrseCB/uR7t1Z7JU2hO+5/cTH0yvLNIxOIxswOASXX3b2X0uRQXLRVCUmpkRIQ8FUKl/1yYy5jIxDZRw6E8/V84kTay1pT79qiZiKdlBuIG+v76n7P3angNnItPVogiMXg6z2daQcA7juE7v49SDh9RLbehKoVjVzQg83fOU9VLEwfzFL08g0eyn+qIPwVUxdf0JXlzGvowuP51ogCZdiflBGY5evmF/Kf3i7OI8YFgxWKl1bhoQvClr/hnoZadyRyrjAslFXWB87QAJszQP2O76x2BtTn+tY1vDxAPC8GEx6fyNUhbfPiZ6dAcgX/+rc8Z4TefhY/zSvHxFd92GBcHY0a/cyzHHvcql4YYhdK8zD1cizy3i52g7DlRHnSkKuv72MLCnJ1mYgo8fP+nQP5GsZoG5TjMgu0SPxoICWG8jhf6VKtZ/SXgUdfKGx8+/QqL88gQu/4x8173UOPU/Nucwr1VHvuY8eGYmdu3QppE9LbvGojGezgbxzdbKAwEY4TIJoQw15zmEsPACxxwISCJxrb0jKhvT8LSIADvL16VzmSlxxE6Fs5AR0Vnuj1uKTQnapuwwihQuJ9yEDSamsii8WlrnCCFOz5P2iIgEHbGO9mUJdEmC09vfjtJ9GJpavSLrRi5jwEda4I
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHXmmM3wQfSlkpB0kOBlQDRimPR0q4Fv0QA
  • Thread-topic: [PATCH 7/9] gnttab: no need to translate handle for gnttab_get_status_frames()

On 26/08/2021 11:14, Jan Beulich wrote:
> Unlike for GNTTABOP_setup_table native and compat frame lists are arrays

"GNTTABOP_setup_table, native"

But I think it would also be clearer to follow with

"frame lists for GNTTABOP_get_status_frames are of".

> of the same type (uint64_t). Hence there's no need to translate the frame
> values. This then also renders unnecessary the limit_max parameter of
> gnttab_get_status_frames().
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.