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

Re: [PATCH 2/6] common: move standard C fixed width type declarations to common header


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Wed, 15 Feb 2023 18:54:29 +0000
  • 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=XhA+FWabA9LtS20hnqX6OA9FcC9yOgKyCbXRh/vos0w=; b=TIB+VUkI2hqahYnFNktcSqER9pBGCloHJ04Thifkxs7jGZJYxQO5w57tvZIdfhYYetAmqGGD1HanTq1VWYABHhb3xY4OOpzjhzbSgv5cLjGgbKvEwTVNH/8/2Jz3q2qC88qUeOGU+sJ7Z+WhjIjxh+y7id9z9pJ1cq8OugtrjJ6fwV3UWFT4Xxf4BcQ1Rp+dtKQT58Ak4+8o6I4FnI7Ykq5YUZeUk0w76ewDu3cTajrMqAFeTyf19hTzHwznwmb+EhkyImJAmbfSokpfXDqKTbxTmjHLeUeDlR2ulJ4R/w46i6MjM+eV56BUjxRmIOtB4OmJADjuqekk0w3GR24BQQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=a2WDlKWFZDbM4O2ZeMGhurJ44FiTWRL/Fc4vUnX7jEODs7RAe5Hm33Zya1+6UvBLQjh4HmHmrMoyEm3Q8cOrR7WipXADK1Br+e+WfYRuKySm+YS5VSq6KjJ6aTIhr4YhPc+PmwczmcZm4cz/73xy1ER2AG2QWZnHVpSgdi1oND5zIWbOQJ0wXRcZGvJ9UdIPxEpLne7d90d4eOsEoknOpdAFCA5tZ+IH3Ds7syLqT7nXnkVWSqyA39qlQRi4Lz7wzcxtelQqpyaVfOqK0I5JGo7EOZmF3FhhVNUlqi4aXNlC5Wn9Ofo9bUW+bN6LHGpHjMosZDKxKqqDbZbsgNZpUQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Bobby Eshleman <bobbyeshleman@xxxxxxxxx>, Alistair Francis <alistair.francis@xxxxxxx>, Connor Davis <connojdavis@xxxxxxxxx>, Daniel Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>
  • Delivery-date: Wed, 15 Feb 2023 18:55:04 +0000
  • Ironport-data: A9a23:ExyTzaLqFeBsx/T+FE+Ri5UlxSXFcZb7ZxGr2PjKsXjdYENS0DdUx mYYCjuCbK6KN2HxLt51Pong8UMFsZXTm9ZgHgBlqX01Q3x08seUXt7xwmUcnc+xBpaaEB84t ZV2hv3odp1coqr0/0/1WlTZhSAgk/rOHvykU7Ss1hlZHWdMUD0mhQ9oh9k3i4tphcnRKw6Ws Jb5rta31GWNglaYCUpJrfPcwP9TlK6q4mhA5AVjPaojUGL2zBH5MrpOfcldEFOgKmVkNrbSb /rOyri/4lTY838FYj9yuu+mGqGiaue60Tmm0hK6aYD76vRxjnVaPpIAHOgdcS9qZwChxLid/ jnvWauYEm/FNoWU8AgUvoIx/ytWZcWq85efSZSzXFD6I+QrvBIAzt03ZHzaM7H09c5vACZD5 +IzDwotbyu8rfya0pb8asZz05FLwMnDZOvzu1lG5BSAV7MKZM6GRK/Ho9hFwD03m8ZCW+7EY NYUYiZuaxKGZABTPlAQC9Q1m+LAanvXKmUE7g7K4/dmpTGMlGSd05C0WDbRUvWMSd9YgQCzo WXe8n6iKhobKMae2XyO9XfEaurnzHyiCdJOT+DQGvhCnWCB3HAsDxEvXFqqn9P6jB75VfN5J BlBksYphe1onKCxdfH/VRClpH+PvjYHRsFdVeY97Wml2qfSpgqUGGUAZjpAc8A98t87QyQw0 V2ElM+vAiZg2JWXVHac+7G8vT60fy8PIgcqfjQYRAEI593ipoAbjR/VSNtnVqmvgbXdBjXY0 z2M6i8kiN07h8MRy7+y+1yBhju2v4XIVSY8/ACRVWWghitzY4SvdoGz6VzW6P9GBImcR1iF+ nMDnqC25eQPFo2ElTbLTv8EGrqo/N6aPDaaillqd7Eq8DKr/HyvdJ5R+xlxIU5oNoAPfjqBS EPLogZQ+J97NWOnd7NqeJm2D9k2zK/mDpLuUfW8RsJHf5xZZAKBuiZ0aiatM3vFlUEtleQzP MmdeMP1V3ICU/04lHyxWvsX1qItymYm32TPSJvnzhOhl72DeHqSTrRDO1yLBgwk0J65TMzu2 443H6O3J993C4USvgG/HVYvEG03
  • Ironport-hdrordr: A9a23:qYWPCKnpQlkgoGTL7wkx2iktTSDpDfPjiWdD5ihNYBxZY6Wkfp +V7ZEmPE7P+VQssS8b6LO90fG7MADhHZ4c2/hrAV7QZniVhILIFvAj0WKG+UyFJ8SQzJ8j6U 4NSdkGNDS0NykLsS+Y2nj6Lz9D+qj4zEnAv463pB0BIW4FCsEQiHYeNu/YKDwGeOAsP+tCKH Po3Ls6m9PWQwVrUi3UPAh8Y8Hz4/nw0L72ax8PABAqrCGIkDOT8bb/VzSIwxsEVDtL4LE6tU zIiRbw6KmPu+yyjka07R6X071m3P/ajvdTDs2FjcYYbh3qlwaTfYxkH5GSoTwvp+mryVAy1P 3BuQ0pMchf427YOku1vRzu8Q/91ytG0Q6h9XaoxV/Y5eDpTjMzDMRMwahDdAHC1kYmtNZglI pWwmOwrfNsfFf9tRW4w+KNewBhl0Kyr3Znu/UUlWZjXYwXb6IUhZAD/XlSDIwLEEvBmcca+a hVfYfhDcRtAAunhkPizzJSKRuXLy4O9yK9MxE/UwquomVrdT5CvhclLYck7wI9HdoGOu15Dq L/Q/tVvYALd9QRa61lAucHXI+YNkzhKCi8bl66EBDfD6cANGvKq5nrpJMPxM/vVqAp4fIJ6c X8uGkzjx9gR6srM7zf4LRbth/KW2mzRjLr14VX4IV4oKT1QP7xPTSEU01GqbrunxyxOLySZx +fAuMaPxbYFxqjJa9ZmwnlH5VCI3gXV8MY/t49RlKVu8rObonnrPbSfvrfLKfkVW9MYBK/Pl ITGDzoYMlQ5EGiXXH1xBDXRnP2Y0T6uZZ9CrLT8eQfwJUEcodMrg8Wg1Kk4dzjE0wNjoUmOE 9lZL/3mKKyome7uW7O8mVyIxJYSl1Y5b3xOkk67zPi83mEOorrl+/vMFy6hkH3USOXZ/mmZj Jimw==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 09/02/2023 10:38 am, Jan Beulich wrote:
> Have these in one place, for all architectures to use. Also use the C99
> types as the "original" ones, and derive the Linux compatible ones
> (which we're trying to phase out). For __s<N>, seeing that no uses exist
> anymore, move them to a new Linux compatibility header (as an act of
> precaution - we don't have any uses of these types right now).
>
> Modern compilers supply __{,U}INT<n>_TYPE__ - use those if available.
> Otherwise fall back to using "mode" attributes, but this can be relied
> upon only when bytes are 8 bits wide. Should there ever be a port to an
> architecture not matching this, it would need to define the fixed width
> types locally by some other means.

These types were added in GCC 4.5.  It is 12 years old.  We even use a
newer C standard than this compiler...

At this point, it is an unreasonable burden to be continuing to support
compilers this obsolete, not to mention that the mode attributes are
unreadable to anyone who isn't a GCC developer and clearly unnecessary
to begin with.

~Andrew



 


Rackspace

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