From nobody Tue Jan 24 22:09:38 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4P1h365Lzvz3bdjM; Tue, 24 Jan 2023 22:09:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4P1h364yPjz4GXt; Tue, 24 Jan 2023 22:09:38 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1674598178; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=Mdt5HlQpI124TdYefIqJDa2+bDcdY2RAH6q9NNl6UgM=; b=dmdIHbqW1lnika5XJEZjtYiwP2q7Fre87No+ilgSoDDILmnkofmYE2A3fodTj3prLLWtZK /R1IfI7/tUxWgMKJsPCtdjn4ndqRIv557reiscdUif983fSUFYa6zoE906SraF8ztIbiXd bLmedH1oBdMNgV50iGBUIDuuulT8OTK47h5PWlsZmLNKethKG6BBYybVH3ZRBtFU8B8UEA gIo9Ma1A1BRtrvp1z6LCf6DiCNSZ58znvPy6ImSVqeO3WGCuarX3gLRPCmbSzu9IUKEeiY TBwWIkYW9yaqKVMg1HE48ik+MJJpe6FXAtroj4xX5J7EccQL91cIUvIfgUpQvQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1674598178; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=Mdt5HlQpI124TdYefIqJDa2+bDcdY2RAH6q9NNl6UgM=; b=ZZ9hiNtKTah+1SrTkH8QX+7qE5cAhxlBRxmrvFpKYRDdEB8wDdZ/3kB3tWOAsABM//p0lI dI+hlpxytwMXFDGLQDPC82eaS0yulBsLoSD6q7xmCJikAsNe2qDBFoZg1qSezS3GaXoSN5 jZnXolSWgV5wtcPSTCI4smXJj+K39JavOXwL0tCAVP48gK8pdGba2Y/4HjXabJ316WQItT fkg8Wy95tj6mBGANqgp1Okw3/rrReIegEbB4rEdMmZ14bKkyTvg/57qx9HrL8Nl9Ixj9PB jGFATdnTV7gQNNg8FKalAsZYcVEC6aVvX16qyOE/rxcs0VX9GKBRLeS11+uROw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1674598178; a=rsa-sha256; cv=none; b=n+XM/0uLVwEiKih52Us+T4OeteSrOHSpJD7Xb/tcTgSJ4Tb3SEoReZ4mdc/7jqIwCal68K fLBnijBkuM7Z2TXNtvHJrYl14XI7Q3UfODoGuj23lySYqscb7LIVj6AA65t0KGQC1Rbrxi CQX76Pk6FPIL78kZLXxETdPDihQ0A8FRIxiU3j2sewiIXNmvGCX5jdGGJHeq5k/+uhC8LC 1A0kREB56MDKT5Vnn1/yJJ6AZOoJ9NNkNzGp/yf1HySv5NW1Tt0Q4+Wo8UAeLqcupyv9yj RVTAGSHdlH7n5k2eDKou+wkPSMqRa/fx3vfdnNE+swqF4F3uVFUmNeoJrOUyAw== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4P1h3644VzzQF5; Tue, 24 Jan 2023 22:09:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 30OM9c8f070439; Tue, 24 Jan 2023 22:09:38 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 30OM9cXq070437; Tue, 24 Jan 2023 22:09:38 GMT (envelope-from git) Date: Tue, 24 Jan 2023 22:09:38 GMT Message-Id: <202301242209.30OM9cXq070437@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Warner Losh Subject: git: 50ceb4089843 - stable/13 - stand/amd64: remove unused addr parameter from bi_load64 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: imp X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: 50ceb408984322133cec4e9627a86315a9d84ff4 Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by imp: URL: https://cgit.FreeBSD.org/src/commit/?id=50ceb408984322133cec4e9627a86315a9d84ff4 commit 50ceb408984322133cec4e9627a86315a9d84ff4 Author: Roger Pau Monné AuthorDate: 2021-01-29 11:29:37 +0000 Commit: Warner Losh CommitDate: 2023-01-24 21:49:15 +0000 stand/amd64: remove unused addr parameter from bi_load64 All callers of bi_load64 pass 0 as the addr parameter, so just remove it and always calculate the last load address from the module chain. No functional change. Sponsored by: Citrix Systems R&D Reviewed by: tsoome, imp Differential revision: https://reviews.freebsd.org/D28412 (cherry picked from commit 45ecda8ebf80bb341aa3b4955810518ae6fab158) --- stand/i386/libi386/bootinfo64.c | 14 +++++++------- stand/i386/libi386/elf64_freebsd.c | 2 +- stand/i386/libi386/libi386.h | 2 +- stand/i386/libi386/multiboot.c | 2 +- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/stand/i386/libi386/bootinfo64.c b/stand/i386/libi386/bootinfo64.c index 9f038f2a4f2b..1ffa4188d926 100644 --- a/stand/i386/libi386/bootinfo64.c +++ b/stand/i386/libi386/bootinfo64.c @@ -182,7 +182,7 @@ bi_checkcpu(void) * - Module metadata are formatted and placed in kernel space. */ int -bi_load64(char *args, vm_offset_t addr, vm_offset_t *modulep, +bi_load64(char *args, vm_offset_t *modulep, vm_offset_t *kernendp, int add_smap) { struct preloaded_file *xp, *kfp; @@ -191,6 +191,7 @@ bi_load64(char *args, vm_offset_t addr, vm_offset_t *modulep, uint64_t kernend; uint64_t envp; uint64_t module; + uint64_t addr; vm_offset_t size; char *rootdevname; int howto; @@ -217,12 +218,11 @@ bi_load64(char *args, vm_offset_t addr, vm_offset_t *modulep, /* Try reading the /etc/fstab file to select the root device */ getrootmount(i386_fmtdev((void *)rootdev)); - if (addr == 0) { - /* find the last module in the chain */ - for (xp = file_findfile(NULL, NULL); xp != NULL; xp = xp->f_next) { - if (addr < (xp->f_addr + xp->f_size)) - addr = xp->f_addr + xp->f_size; - } + addr = 0; + /* find the last module in the chain */ + for (xp = file_findfile(NULL, NULL); xp != NULL; xp = xp->f_next) { + if (addr < (xp->f_addr + xp->f_size)) + addr = xp->f_addr + xp->f_size; } /* pad to a page boundary */ addr = roundup(addr, PAGE_SIZE); diff --git a/stand/i386/libi386/elf64_freebsd.c b/stand/i386/libi386/elf64_freebsd.c index 6bf884045eea..1108a9e4d713 100644 --- a/stand/i386/libi386/elf64_freebsd.c +++ b/stand/i386/libi386/elf64_freebsd.c @@ -80,7 +80,7 @@ elf64_exec(struct preloaded_file *fp) return(EFTYPE); ehdr = (Elf_Ehdr *)&(md->md_data); - err = bi_load64(fp->f_args, 0, &modulep, &kernend, 1); + err = bi_load64(fp->f_args, &modulep, &kernend, 1); if (err != 0) return(err); diff --git a/stand/i386/libi386/libi386.h b/stand/i386/libi386/libi386.h index dbd6565f8d1c..a52a1190e7ec 100644 --- a/stand/i386/libi386/libi386.h +++ b/stand/i386/libi386/libi386.h @@ -151,7 +151,7 @@ void bi_setboothowto(int howto); vm_offset_t bi_copyenv(vm_offset_t addr); int bi_load32(char *args, int *howtop, int *bootdevp, vm_offset_t *bip, vm_offset_t *modulep, vm_offset_t *kernend); -int bi_load64(char *args, vm_offset_t addr, vm_offset_t *modulep, +int bi_load64(char *args, vm_offset_t *modulep, vm_offset_t *kernend, int add_smap); void pxe_enable(void *pxeinfo); diff --git a/stand/i386/libi386/multiboot.c b/stand/i386/libi386/multiboot.c index 71fd63bb4c88..2d0a4a9b2274 100644 --- a/stand/i386/libi386/multiboot.c +++ b/stand/i386/libi386/multiboot.c @@ -274,7 +274,7 @@ multiboot_exec(struct preloaded_file *fp) bzero(mb_mod, sizeof(struct multiboot_mod_list) * NUM_MODULES); - error = bi_load64(fp->f_args, 0, &modulep, &kernend, 0); + error = bi_load64(fp->f_args, &modulep, &kernend, 0); if (error != 0) { printf("bi_load64 failed: %d\n", error); goto error;