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

Re: [PATCH] libs/foreignmemory: fix MiniOS build


  • To: Roger Pau Monne <roger.pau@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Fri, 29 Jan 2021 08:11:28 +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=oTrwzwkYidd2+wpoJ8V4Ag5UDFciWH9ldQ3ocyeNAqA=; b=lPs1gSH4G2bP8OlAx3NxQmDhbs4Gh2JSkuVvF+b3E5XCfbmmgn/3yXklUO0wz5Oz61mJSFSpVnSkXuCXn+eVIFELjfL6y67ZiaFgFaCLuumkFuM6lcS15PIC3PnxUxRuBWIFy9F69NhlMCTR4nBVGZ6DlDkzClqmpkmYZH6dXMgU+scVZCCXG11QXxL3zLxHhFxFAkGw/0bqSwsLIqhc+F0DgL4ct+z/F8AId4MC/HnaUM7cOrlKuQ/aERWomM9+7nyB8kQ+X//VDM8JaYmyhdrPob/4e9OL23ZNDZXyUKOkYW2cmDFjGsBSO5r8nno507y9m+SCy3LTQFt7RXMnWQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RNLJJ1zR/qLE2zj/A47/KM4ejyQrtkDa+7JDbPqNRHbU1yc8ENhlXIyNPTAFrCSPi5Li7Qf0BEoQsxtaEGuIUw1Wg5WIx/KCd+QTiWZg0cuJMBfwwhZRUrUMongzdbegPv8Ni3JtL5urw4cARnGy5byfLEm2pgIj25o/CyRTjL8ZP3fNQEgrhSjpLjF6sXIX4ShaiLZboKwD+epYVT+0EIl2vuysCJIs1jFv9k2Ye6tuTS+WOIH3Zt4MKoZMVCX/qRA+bAG75Z+MiLOEL6K6WwJYevR8jC4I4iBzwgTEwA4zUV9W4rz9nK5Zysj6SvxWRbmqOphCBcZ4ZSDDYWi1xQ==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 29 Jan 2021 08:12:27 +0000
  • Ironport-sdr: 0gnr3sPFR/V6BQHcThx/t/G5MA0FIFqzE0MAEvY4tDg26fhUY1FAmM5ar3MvspmA5KJhDaG4Z3 jrN95hb6GOesSSn/oSkAjbuaVLuT/PV5b/Ygqcb0dW87YYkKGA7B9Y1SZGndOsrOW5F92j85W0 JTHk+kODW5Pd5VwMgD4e/04YbsVxTSaXFLT3M38L8HnWsLJ+rIpQnXHa5OgK+mYxP1DuMQBSc3 o7FwJvlL+ku7FLMtAZ+8p6uIOZrXq6nIPXwq1MYaGWUJ9lQ3gQ0TilIJHONrUxZLoyDqFgNn6g BzU=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 29/01/2021 08:09, Roger Pau Monne wrote:
> Keep the dummy handlers for restrict, map_resource and unmap_resource
> for MiniOS, or else the build breaks with:
>
> ld: 
> /home/osstest/build.158759.build-amd64/xen/stubdom/mini-os-x86_64-xenstore/mini-os.o:
>  in function `xenforeignmemory_restrict':
> /home/osstest/build.158759.build-amd64/xen/stubdom/libs-x86_64/foreignmemory/core.c:137:
>  undefined reference to `osdep_xenforeignmemory_restrict'
> ld: 
> /home/osstest/build.158759.build-amd64/xen/stubdom/mini-os-x86_64-xenstore/mini-os.o:
>  in function `xenforeignmemory_map_resource':
> /home/osstest/build.158759.build-amd64/xen/stubdom/libs-x86_64/foreignmemory/core.c:171:
>  undefined reference to `osdep_xenforeignmemory_map_resource'
> ld: 
> /home/osstest/build.158759.build-amd64/xen/stubdom/mini-os-x86_64-xenstore/mini-os.o:
>  in function `xenforeignmemory_unmap_resource':
> /home/osstest/build.158759.build-amd64/xen/stubdom/libs-x86_64/foreignmemory/core.c:185:
>  undefined reference to `osdep_xenforeignmemory_unmap_resource'
> ld: 
> /home/osstest/build.158759.build-amd64/xen/stubdom/mini-os-x86_64-xenstore/mini-os.o:
>  in function `xenforeignmemory_resource_size':
> /home/osstest/build.158759.build-amd64/xen/stubdom/libs-x86_64/foreignmemory/core.c:200:
>  undefined reference to `osdep_xenforeignmemory_map_resource'
>
> Fixes: 2b4b33ffe7d67 ('libs/foreignmemory: Implement on NetBSD')
> Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Thanks for getting to this.  I hadn't had time thus far.

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx> and I'll put it in
straight away to unblock things.



 


Rackspace

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