[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



 


Rackspace

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