Groups | Search | Server Info | Keyboard shortcuts | Login | Register


Groups > gnu.hurd.bug > #12035

Re: PCI arbiter crash on last qemu image

Path csiph.com!3.eu.feeder.erje.net!feeder.erje.net!goblin2!goblin1!goblin.stu.neva.ru!usenet.stanford.edu!not-for-mail
From Damien Zammit <damien@zamaudio.com>
Newsgroups gnu.hurd.bug
Subject Re: PCI arbiter crash on last qemu image
Date Tue, 18 Aug 2020 14:28:26 +1000
Lines 21
Approved bug-hurd@gnu.org
Message-ID <mailman.131.1597724918.2469.bug-hurd@gnu.org> (permalink)
References <78d17df2-64dd-692e-4151-ad14e25a97d6@mailfence.com> <2f03cf83-9dff-1b92-a6e0-96ae20bfda49@zamaudio.com> <e3d57b71-af84-c035-7799-0c4dc6bfdae4@mailfence.com> <c5ae2597-dd25-58f6-8aa2-18b60b91fe5f@zamaudio.com> <ce86a983-886d-cf4d-bb44-106a17ad9282@mailfence.com> <36514972-223d-5a32-d5d9-c102caa9d715@zamaudio.com>
NNTP-Posting-Host lists.gnu.org
Mime-Version 1.0
Content-Type text/plain; charset=utf-8
Content-Transfer-Encoding 8bit
X-Trace usenet.stanford.edu 1597724918 25461 209.51.188.17 (18 Aug 2020 04:28:38 GMT)
X-Complaints-To action@cs.stanford.edu
To jlledom@member.fsf.org, bug-hurd@gnu.org
Envelope-to bug-hurd@gnu.org
DKIM-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=zamaudio-com.20150623.gappssmtp.com; s=20150623; h=subject:to:references:from:autocrypt:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=PPe+CcYJG3HCxCeD37MyQWB6B5Oj0zWPryg372NmXH8=; b=aU2Sk8sFmF9IEdxXDIIWyXZldnlqLqOgeyL/flIzWLZymeooSqiLnRFR3QqlBslnQv V6axNfBlta9dWOrmb25SuJNsF0msMtLqV9XKkkwbyWBbRj6s23zgzULeinJIjA8fqLUc +Hbcs+1g5iLuBByvRr9yUBGSXBG1KB0C8h/ESCKeDFuhs/9TbJl0wtPMgVkd7PF2ZU/q doN4jzII+aLyyyc2XzYjA0PNFOVPfw+AiOSIc8BiDsHJJFmtrwQP3KkgSrvho5SauRUf R5zcZhIy58AB0UAx1ppx454A4T5MliVdd4OpX/XiJGAqZ16MbRsuC/lZZ53Hp2wGtue8 SB2Q==
X-Google-DKIM-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:from:autocrypt:message-id :date:user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=PPe+CcYJG3HCxCeD37MyQWB6B5Oj0zWPryg372NmXH8=; b=azYERnGfPdzN5p4Tfv1epNMNLpNSxlpnMJhaqsngIHNWEZr6+jmPWdTEvYh30NDeGJ D17ub66aX4G+MhzhSxdYfneiG8NjhiBKFSYXsQzrJtdQyqXmYBCMC9SHK4JFVgplqJtq 7HQHd41hpXkDpVPlF+uetyy7aFOvC21clfSujUOvgXBkD0VeCPCAox32kbxSwANw/jU2 eZutfffEfP37w5m2F5cjdVjZed/eyemCedlW1D8Gzzu5b2JzOfdzVIKPgSA6VZ8ShXCs YNYO4oauw7lc8DWgYD3Urwyy1aihllaED8iIzbBuB+ILl4kIYR+2Zx5BepJW+E3wnYED SLvw==
X-Gm-Message-State AOAM532p0eqx9fFJzC8bQOIg+fR897bQBImUzGVf7B8mxCmMmvD2Qb05 ZjEVDEajB+7lYUWloulEsbodQ1FkFenQwQ==
X-Google-Smtp-Source ABdhPJxh61L047yQvHZ2uJq3cZP+cEJe/WkVEHVOK20dZUgGUaRGjlTh3poo/5dZEIg6X4e5i2l1KA==
X-Received by 2002:a63:2a96:: with SMTP id q144mr11397287pgq.87.1597724912080; Mon, 17 Aug 2020 21:28:32 -0700 (PDT)
Autocrypt addr=damien@zamaudio.com; prefer-encrypt=mutual; keydata= mQENBFKCKqwBCACxNliUKB4XUVDmLbY9bBx+XONng5jqnGz2Hheo/1vfdNtuigTaYBWML2ma lDAImfihvPpHS7eyISb7SuCrUZ84RV21wIf03IO9hqDumM0xE6GNxWosRvegjYKAiiGsN9gn 8F7tmeQEuDEqzt5HLnktOSwlrQ3CFk+aiqrYjYhTIduF1MQPgYarRjiiIJtSFoinfRTeHKW4 bgxIIYKvwa4C3McCqPHkTJbkHxITWnlnOuskxFszglDt9UiGtlbapBRvmFSe+9xOrjOkOJe/ 9riyZ04mpO/xADjOsDnXaT+OcQIiZgift+nQs1IP0HEIf2TK5c0Gfrqn1FLiU8gMk+zjABEB AAG0I0RhbWllbiBaYW1taXQgPGRhbWllbkB6YW1hdWRpby5jb20+iQE4BBMBAgAiBQJSgiqs AhsDBgsJCAcDAgYVCAIJCgsEFgIDAQIeAQIXgAAKCRBXfBc5WF+pII7uB/9iMYVbS7AhnYJi DiCV9X6C27E4fYxA6/mGdy6MgjZ3kDAnBnBLYcbxtvcjMiCBiPNsyE//yQr12CyoyNqjh9BQ 4AXDdkhJZ6oW1HeP0Dqo+q1+uThlyXEMHC5qKxH2RWFnStmwR6qEr35By3LD8RQO5h7C2mpT VzzArYz5896qpqZizkPcVztbH4eymwQoZBCD0aF/76extAymQ8y7CxQueSrtDLedLKVPAANC msuaLNYaw5CdqkFo16iA0eBadjrrnTyX/FAnbnUKLbnVVE56gPLebz4ay9ZmQXJpLdjnHfkb m21z8IQ0K2ct5cZQRbA5ge27V87ts+OmutBlSk6zuQENBFKCKqwBCADLGxUiBOiRNXroZp1a PwgzXRAhrHDpp5qlXJ3vpqU6HAJYVc6Ozp65ADy2e6Du5GYi3KOmc+YITfLEIZ/CDrzz4IfZ whnoYd1kpMeTasq3wP44WOYzVx/1/Rmo9JlSOdaFi4TVlEX7qF4kUfWLqt0wSFYgPg/OpQeL uNO9e3GIL4hk5UtD633PMVjmjShO6H52TosckQ4jiNt5yUJun8ABkDxBbcfycMrLlAMUGydL S1WAz+/fp1P0VL2bmnsOif+ZO19EvN7JhohQEovfaZ3R7UCtcJm6ZfjH1e1Zw80HQtZ+MnTN 0PmCRydmWJzyDecBffoRVlhwes+ff6tx+9RVABEBAAGJAR8EGAECAAkFAlKCKqwCGwwACgkQ V3wXOVhfqSD5lAf/aAMTm5qknUBqfyE8Z6PuXa3hwcH1ZmkXYrm8u3V4SJUTz69iixihi/kI hzM63n6IpllQvxYjT6bvSH5KNzJYUlqdA3pLun0J+cNMda4VQXWUuLRtc2NyYJE0liyKRdm+ op65BkRqr6wP5Af2Wj9hfMrCn2eFaiYh9ymFmg2Ee4yfFdwU5BFiYNMPbx9mbf78Q4f4VSBG 1XO6Ob55JEbC/7UhbnPkEG18MrNLx1ea5YpaRbKPXVVSonKk8YVIzCJxcDcse4ZRZogSnrJF C+GZpY4WoQQ/5kWRmNFQbIw38KfJGf2BmLa4dr9Kl/Qg288nbo66HJxqv+pWaWwmaja5Sg==
User-Agent Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0
In-Reply-To <ce86a983-886d-cf4d-bb44-106a17ad9282@mailfence.com>
Content-Language en-US
Received-SPF none client-ip=2607:f8b0:4864:20::430; envelope-from=damien@zamaudio.com; helo=mail-pf1-x430.google.com
X-detected-operating-system by eggs.gnu.org: No matching host in p0f cache. That's all we know.
X-Spam_score_int -18
X-Spam_score -1.9
X-Spam_bar -
X-Spam_report (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no
X-Spam_action no action
X-BeenThere bug-hurd@gnu.org
X-Mailman-Version 2.1.23
Precedence list
List-Id Bug reports for the GNU Hurd <bug-hurd.gnu.org>
List-Unsubscribe <https://lists.gnu.org/mailman/options/bug-hurd>, <mailto:bug-hurd-request@gnu.org?subject=unsubscribe>
List-Archive <https://lists.gnu.org/archive/html/bug-hurd>
List-Post <mailto:bug-hurd@gnu.org>
List-Help <mailto:bug-hurd-request@gnu.org?subject=help>
List-Subscribe <https://lists.gnu.org/mailman/listinfo/bug-hurd>, <mailto:bug-hurd-request@gnu.org?subject=subscribe>
X-Mailman-Original-Message-ID <36514972-223d-5a32-d5d9-c102caa9d715@zamaudio.com>
X-Mailman-Original-References <78d17df2-64dd-692e-4151-ad14e25a97d6@mailfence.com> <2f03cf83-9dff-1b92-a6e0-96ae20bfda49@zamaudio.com> <e3d57b71-af84-c035-7799-0c4dc6bfdae4@mailfence.com> <c5ae2597-dd25-58f6-8aa2-18b60b91fe5f@zamaudio.com> <ce86a983-886d-cf4d-bb44-106a17ad9282@mailfence.com>
Xref csiph.com gnu.hurd.bug:12035

Show key headers only | View raw


On 18/8/20 6:51 am, Joan Lledó wrote:
> El 17/8/20 a les 1:51, Damien Zammit ha escrit:
>> Perhaps a better way to fix the mapping problem I encountered
>> is by removing the check for previous mappings when trying to map regions,
> 
> I could check the pointer before reading from it at func_files.c, and if
> it happens to be null, then call the libpciaccess mapping function from
> there, i.e. mapping the memory in the first access instead of doing it
> during the startup. So there's no need to make any changes in your
> patch, do you think that'd work?
> 

That would probably work, but I don't want to break other usages of pciaccess
by including my change upstream, doing it your way would mean every other use case
for pciaccess would need to do the same thing as you are suggesting.

Perhaps we can just remove the check instead after all in pciaccess,
as that would be compatible with existing code.

Damien

Back to gnu.hurd.bug | Previous | Next | Find similar


Thread

Re: PCI arbiter crash on last qemu image Damien Zammit <damien@zamaudio.com> - 2020-08-18 14:28 +1000

csiph-web