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

Re: [PATCH 2/3] tools/libs: Stash the 4.14 API/ABIs for the stable libraries


  • To: Jürgen Groß <jgross@xxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Tue, 2 Mar 2021 11:17:50 +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-SenderADCheck; bh=1+3UQ8r09124ZiDtcgSdGhpoQNNN1AuuzpSWBpCoAGg=; b=JRHPn8C3+FiGrIWO8PssI3VMBEETyDxXCA7E/EbfI95L+nrB3CxMZgg9DGVJQX8cTdhm0X99BKdGoxawUSKdyI42NRwnJTzY858eU4VU+JZpg1uZ6CJWNVmqUmN5bTDmQ0uTDLWQounY5Fj7c8AvvfLD8xgpsVxf3DZpMAHSytH8uc4N7ZeeDb/sbULxhUSm0y/O/knPnkg7/qoQC7VibDiEHUXwuPWVdje1SpGXlpuu1wrvicOq/mFjGeLct48iiUj5GIxRLKmU07qNaxPmqHN/BDRDu2mexHG8ZG5be7rGxXdQSIvneNY8HY6Z8QV/zUrFx3uVooabgq5TLn+z4Q==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=D0vNL4CRtJqFbQtLicS9X1UmdVhrpPVT7BfkIllNwTsYdnbBCzgO6WNf9fogwQwpDxDKwlmetlRfDcTrIrA7RK2S3Vu9C7ItFJzSZ60GmxV8Wyv8VNhStVgPNq8ClJ98kR99sZO3rXdDYaP1G5Rv7TwRF+9VMdgNBjMoMZze8jFBUZ1yNWAa1oIr33NmrwTSExGSI7VxbWD/S9vNJD68DNPKKePQYfLmyxQEF2BXK99yUkJe8IyDkZu3dqFz0cbqkB+uFpIMB5JdfrAF3FFToLz+ozLJ9POplISQSllH3FKKKf3mn0tNbr12jt+gIkuAXvzfshs/P3LBCLpD0PI1nQ==
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Tue, 02 Mar 2021 11:18:04 +0000
  • Ironport-sdr: kBAGlCPNB7DpegBl8LUtnmwHpjFU/zQmuX++L11Rh2ZdYCn/RnYTJbCRLlTUZT3Cc4AqjcITLy hLxY8H1xOV7nt8USP2bwpnBXcL+TpEBWdAA87bq4T+09DyuJ4ZKalzPXo4Y2m++LaHuc50/3wc P+7grxi+MTDcKWV1Lgw4emSxfxXQ4DRrSck0958QY6N1X/U9jRdZMS8cNkDFCT5RzPD/4p+BJk U9ERWcDgAujI4VMjdgywovUXbWOhlEddYnrF4FMMjiatmhfN5aqrrqnsyBfBe0Ur45iW0ch4by whU=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 02/03/2021 10:45, Jürgen Groß wrote:
> On 01.03.21 18:00, Andrew Cooper wrote:
>> These dumps were produced from the RELEASE-4.14.0 tag, with the
>> abi-dumper
>> tooling backported from staging.
>>
>> For each stable library, add a PKG_OLD_ABI variable pointing at the
>> 4.14 ABI.
>>
>> No functional change.
>>
>> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>> ---
>> CC: Ian Jackson <iwj@xxxxxxxxxxxxxx>
>> CC: Wei Liu <wl@xxxxxxx>
>> CC: Juergen Gross <jgross@xxxxxxxx>
>> ---
>>   tools/libs/abi/libxencall.so.1.2-x86_64-abi.dump   |  924 +++++++++++
>>   .../abi/libxendevicemodel.so.1.3-x86_64-abi.dump   | 1491
>> +++++++++++++++++
>>   tools/libs/abi/libxenevtchn.so.1.1-x86_64-abi.dump |  719 ++++++++
>>   .../abi/libxenforeignmemory.so.1.3-x86_64-abi.dump |  847 ++++++++++
>>   tools/libs/abi/libxengnttab.so.1.2-x86_64-abi.dump | 1199
>> ++++++++++++++
>>   tools/libs/abi/libxenhypfs.so.1.0-x86_64-abi.dump  |  597 +++++++
>>   .../libs/abi/libxenstore.so.3.0.3-x86_64-abi.dump  | 1711
>> ++++++++++++++++++++
>>   .../libs/abi/libxentoolcore.so.1.0-x86_64-abi.dump |  239 +++
>>   .../libs/abi/libxentoollog.so.1.0-x86_64-abi.dump  |  882 ++++++++++
>>   tools/libs/call/Makefile                           |    2 +
>>   tools/libs/devicemodel/Makefile                    |    2 +
>>   tools/libs/evtchn/Makefile                         |    2 +
>>   tools/libs/foreignmemory/Makefile                  |    2 +
>>   tools/libs/gnttab/Makefile                         |    2 +
>>   tools/libs/hypfs/Makefile                          |    2 +
>>   tools/libs/store/Makefile                          |    2 +
>>   tools/libs/toolcore/Makefile                       |    2 +
>>   tools/libs/toollog/Makefile                        |    2 +
>>   18 files changed, 8627 insertions(+)
>>   create mode 100644 tools/libs/abi/libxencall.so.1.2-x86_64-abi.dump
>>   create mode 100644
>> tools/libs/abi/libxendevicemodel.so.1.3-x86_64-abi.dump
>>   create mode 100644 tools/libs/abi/libxenevtchn.so.1.1-x86_64-abi.dump
>>   create mode 100644
>> tools/libs/abi/libxenforeignmemory.so.1.3-x86_64-abi.dump
>>   create mode 100644 tools/libs/abi/libxengnttab.so.1.2-x86_64-abi.dump
>>   create mode 100644 tools/libs/abi/libxenhypfs.so.1.0-x86_64-abi.dump
>>   create mode 100644 tools/libs/abi/libxenstore.so.3.0.3-x86_64-abi.dump
>>   create mode 100644
>> tools/libs/abi/libxentoolcore.so.1.0-x86_64-abi.dump
>>   create mode 100644 tools/libs/abi/libxentoollog.so.1.0-x86_64-abi.dump
>>
>
>> diff --git a/tools/libs/call/Makefile b/tools/libs/call/Makefile
>> index 4ed201b3b3..37a7db5395 100644
>> --- a/tools/libs/call/Makefile
>> +++ b/tools/libs/call/Makefile
>> @@ -11,4 +11,6 @@ SRCS-$(CONFIG_SunOS)   += solaris.c
>>   SRCS-$(CONFIG_NetBSD)  += netbsd.c
>>   SRCS-$(CONFIG_MiniOS)  += minios.c
>>   +PKG_OLD_ABI =
>> ../abi/libxen$(LIBNAME).so.1.2-$(XEN_TARGET_ARCH)-abi.dump
>> +
>
> Any reason you don't add
>
> PKG_OLD_ABI =
> ../abi/libxen$(LIBNAME).so.$(MAJOR).$(MINOR)-$(XEN_TARGET_ARCH)-abi.dump
>
> to tools/libs/libs.mk, maybe with some way to override/disable the
> setting (e.g. by setting a different value for PKG_OLD_ABI just
> after including $(XEN_ROOT)/tools/libs/libs.mk) ?

The problem is with libraries which have changed in staging, where
$MINOR differs by 1.  I chose not to wildcard in ../abi/ to reduce the
chance of picking up the wrong ABI to check against.

Something needs to be a statement of which is the appropriate $MINOR to
use, and it shouldn't be the change to bump the soname, as that is a
change we want to be tested.

(That said, I'm actually struggling to get abi-compliance-checker to
catch all the things it ought to catch.  Its better than nothing, but
has some shortcomings.)

> You could even disable the ABI check if there is no map file
> provided, which will be the case for all the unstable libs.

That would be nice, but I don't see any way to do it.

~Andrew



 


Rackspace

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