Re: pci-arbiter: Implement mapping for ROM files

2022-08-16 Thread Samuel Thibault
Joan Lledó, le mar. 16 août 2022 18:44:40 +0200, a ecrit:
> El 16/8/22 a les 18:10, Samuel Thibault ha escrit:
> > ? Any memory allocated by a process will be freed when the process shuts
> > down.
> 
> Yes, I know, just wondering if there was a good practice about malloc/free
> in the Hurd.

For globally-referenced data ("still reachable") it's not that much
useful to spend time on implementing freeing it.

Samuel



Re: pci-arbiter: Implement mapping for ROM files

2022-08-16 Thread Joan Lledó

Hi,

El 16/8/22 a les 18:10, Samuel Thibault ha escrit:

? Any memory allocated by a process will be freed when the process shuts
down.



Yes, I know, just wondering if there was a good practice about 
malloc/free in the Hurd.




Re: pci-arbiter: Implement mapping for ROM files

2022-08-16 Thread Samuel Thibault
Joan Lledó, le mar. 16 août 2022 10:38:28 +0200, a ecrit:
> I just though... shouldn't this allocated memory be freed somewhere when the
> arbiter shuts down?

? Any memory allocated by a process will be freed when the process shuts
down.

Samuel



Re: pci-arbiter: Implement mapping for ROM files

2022-08-16 Thread Joan Lledó

Hi,

I just though... shouldn't this allocated memory be freed somewhere when 
the arbiter shuts down?


El 15/8/22 a les 20:33, Samuel Thibault ha escrit:

Samuel Thibault, le lun. 15 août 2022 20:20:16 +0200, a ecrit:

Joan Lledó, le lun. 15 août 2022 20:07:09 +0200, a ecrit:

El 15/8/22 a les 19:37, Samuel Thibault ha escrit:

Mmm, but doesn't libpciaccess allow to map the BAR?Then pci-arbiter
could create a read-only memory proxy of the mapping?


Yes, the arbiter exposes the BAR regions as files called "region0",
"region1", etc. And they can be mapped through a proxy, that's already
working. But this is about the "rom" files which expose the content of the
expansion ROM for each device.


Sorry, I meant: why can't we do that for ROMs? Does libpciaccess not
provide a way to map ROMs?


Ah, no it doesn't.

Applied, thanks!

Samuel