[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Backport request (was: [PATCH v2] tools: convert bitfields to unsigned type)
- To: Roger Pau Monné <roger.pau@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 4 Jul 2023 18:16:41 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.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=by9dNAtVxlZdy0Y9TdvuDIz6tOBX7yrl8a2VU5AbR3A=; b=L4FwodA53QSEc4Y19L55ixgKKLQG8RCOTU8cpVkDO2SG/uGWyU/iA2kMZFMuGkyHKz+2V+EQSH3hehyGfLZEOOyXRAQBu+5kVzLlOUrHgoMXSIjID48QTTdwxE1F0/g/fDlbYt5mFb/oH2/x1O1tuFcBkeWQNRyTwNbHH/anvSx/UzFJnYbOjm1Pte7c8cUiYhHRYWDpJrofeU8iQavqEIIu2EFXs4H/gnw23BHIxZb1P0nHF2iJHjQH27Hby06fqk9inenyZnbGM9I4P/dP3eJkY4oUad+duS3XmihG21NE0Y+OQzsrih9JwhotpFuXjQz2us3wbNdFHogYBmZBBQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dToEMdClDj7BfeDOGuYViJkHw9ah+c6JzxTAJfROepAGuX75Ap/cmoYIKoHBuAy2LqhxtDl0xWFSoZsql9RdQJrCyxrJQfQ0fWJKb1ohAO462oJf1hSY9t8YHnDPe85Ti0dk25kpqTaRNS/bdAF4VzuYFIrSKzHHYmbWdY25OM9cxRoOat7qyvAzjKyFtqbXtE+0D4IwYy8aZzh105Dntw3eh5O88nr+MyQjUscCU1hGVQt0fC7Qn7NOviFyBhuIBLR1W2wB25jZ77Ft6/hFQ4OtDMf2oxUbta/7+zjP2SE7ld3uCO+KjxQgX4y39BRzmrx4tDcutlVAtdWzliFomw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>
- Delivery-date: Tue, 04 Jul 2023 16:16:48 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 04.07.2023 18:10, Roger Pau Monné wrote:
> On Tue, Jul 04, 2023 at 06:04:36PM +0200, Jan Beulich wrote:
>> On 04.07.2023 17:55, Roger Pau Monné wrote:
>>> On Tue, Jul 04, 2023 at 05:42:33PM +0200, Jan Beulich wrote:
>>>> Plus is the mentioned
>>>> "potential ABI change" safe to take on a stable branch? There's not going
>>>> to
>>>> be any SONAME change ...
>>>
>>> Is there any ABI change in practice? Both fields will still have a 1bit
>>> size.
>>
>> But what a consumer of the interface reads out of such a field would change
>> in case their compiler settings arrange for signed bitfields when signedness
>> isn't explicit. We don't dictate, after all, what compiler settings to use
>> with our interfaces (which generally is good, but which bites us here).
>
> Hm, I see. I would argue that sign doesn't matter here, as those are
> intended to be booleans, so anything different than 0 would map to
> `true`. But implementation might have hard coded TRUE to -1, and the
> change would then break them?
That's a possible scenario I'm wary of here, yes.
> I'm failing to see that, because those implementations would still use
> the old struct declarations they have been built with, and hence would
> still threat it as signed?
Until they rebuild against the updated header, without any change to
their code.
Anthony - do you have any thoughts here?
Jan
|