Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2020-01-26 Thread Samuel Thibault
Joan Lledó via Bug reports for the GNU Hurd, le dim. 26 janv. 2020 09:52:12 +0100, a ecrit: > I'm glad to say that our patches for both picutils and libpciaccess are > now upstream. Yay \o/ Samuel

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2020-01-26 Thread Joan Lledó via Bug reports for the GNU Hurd
Hello Hurd, I'm glad to say that our patches for both picutils and libpciaccess are now upstream. El 10/11/19 a les 2:44, Samuel Thibault ha escrit: > Joan Lledó, le sam. 09 nov. 2019 17:36:19 +0100, a ecrit: >> I was also wondering if after your changes, libpciaccess would support >> nested

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Samuel Thibault
Joan Lledó, le sam. 09 nov. 2019 17:36:19 +0100, a ecrit: > I was also wondering if after your changes, libpciaccess would support > nested arbiters, or if is that something we want at all. We'd want this, yes. > About pciutils, I've seen our patch doesn't use the RPC get_ndevs(), so > no need

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Joan Lledó
Hi, Thanks for your explanations, Samuel. Then I'll take your changes on the libpciaccess patch, fix the closedir() issue and send it to upstream. I was also wondering if after your changes, libpciaccess would support nested arbiters, or if is that something we want at all. About pciutils, I've

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Almudena Garcia
> That's exactly it, no manual compilation needed. Just make sure to be > getting upgrades from the "unreleased" distrib. Thanks. This week I'll try to test It El sáb., 9 nov. 2019 a las 15:21, Samuel Thibault () escribió: > Almudena Garcia, le sam. 09 nov. 2019 15:15:31 +0100, a ecrit: > > but

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Samuel Thibault
Almudena Garcia, le sam. 09 nov. 2019 15:15:31 +0100, a ecrit: > but I don't know if I simply have to "apt full-upgrade", That's exactly it, no manual compilation needed. Just make sure to be getting upgrades from the "unreleased" distrib. Samuel

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Almudena Garcia
Then, does Debian repositories includes all new PCI Arbitrer and libpciaccess0 updates? Samuel asked me to test the latest versions of netdde, libpciaccess0 and hurd, in my Thinkpad T60, but I don't know if I simply have to "apt full-upgrade", or I need to do a manual compilation from upstream El

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Guillem Jover
Hi! [ BTW it seems like something on the gnu.org side is mandling UTF-8 characters? :/ ] On Sat, 2019-11-09 at 11:54:15 +0100, Samuel Thibault wrote: > Joan Lledó via Bug reports for the GNU Hurd, le sam. 09 nov. 2019 10:50:18 > +0100, a ecrit: > > And, about pciutils, what did you do? >

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Samuel Thibault
Concerning packages uploaded to the "unreleased" distribution, a repository if there was one would be on https://salsa.debian.org/hurd-team I however don't create one for each and very package I upload to "unreleased". Normally the patchs uploaded there are already submitted upstream or to a

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Joan Lledó via Bug reports for the GNU Hurd
El 9/11/19 a les 10:48, Joan Lledó via Bug reports for the GNU Hurd ha escrit: > Now I have some questions: > > - You said libpciaccess upstream code for the Hurd doesn't match the one > in the Debian package. Where is the debian repo for the package? I found > [1] but it doesn't seem to have

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-09 Thread Joan Lledó via Bug reports for the GNU Hurd
Hi, El 3/11/19 a les 21:48, Samuel Thibault ha escrit: > Hello, > > Uploaded fixed netdde, pciutils, libpciaccess, and commited this > pci-arbiter cleanup! > Great! Now I have some questions: - You said libpciaccess upstream code for the Hurd doesn't match the one in the Debian package.

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-05 Thread Damien Zammit
On 4/11/19 7:48 am, Samuel Thibault wrote: > Hello, > > Uploaded fixed netdde, pciutils, libpciaccess, and commited this > pci-arbiter cleanup! > > Samuel > Thank you for doing all this! wOOt!! I will test it soon. Damien

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-03 Thread Samuel Thibault
Hello, Uploaded fixed netdde, pciutils, libpciaccess, and commited this pci-arbiter cleanup! Samuel

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-03 Thread Joan Lledó
Strange, the patches were not attached... I send them again.

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-11-03 Thread Joan Lledó via Bug reports for the GNU Hurd
Hello, El 27/10/19 a les 16:32, Samuel Thibault ha escrit:> Hello, > Could you try to split your changes in separate patches? I splitted the patch into four patches, the first one is the Damien's original patch adapted to the current head + a changelog. The other three patches are my changes.

Re: [PATCH] pci-arbiter: Remove embedded pciaccess code

2019-10-27 Thread Samuel Thibault
Hello, Thanks for your work? Could you try to split your changes in separate patches? Otherwise the review is harder since the reviewer has to work out which pieces are intended to what part of the changes, and applying the parts that look ok while other parts are getting repolished is harder