Re: A few quick mmap questions

2024-12-22 Thread Corinna Vinschen
On Dec 20 17:48, Ken Brown wrote: > On 12/20/2024 10:37 AM, Corinna Vinschen wrote: > > No worries. The expression > > > >size_t commit_len = u_len - (start_addr - u_addr); > > > > may be right, but it's not generally intelligible. If you'd like > > to take a stab of changing the code to be

Re: A few quick mmap questions

2024-12-20 Thread Ken Brown
On 12/20/2024 10:37 AM, Corinna Vinschen wrote: No worries. The expression size_t commit_len = u_len - (start_addr - u_addr); may be right, but it's not generally intelligible. If you'd like to take a stab of changing the code to be more clear, I'm happy. I'll give it a try. By the way, t

Re: A few quick mmap questions

2024-12-20 Thread Corinna Vinschen
On Dec 20 10:12, Ken Brown wrote: > On 12/20/2024 8:13 AM, Corinna Vinschen wrote: > > In case we're called via fhandler_base::raw_read(): > > > >Assuming the region to read is enclosing the noreserve() region... > > > >- addr = 0x1, len = 256K > >- rec->get_address() = 0x1000

Re: A few quick mmap questions

2024-12-20 Thread Ken Brown
On 12/20/2024 8:13 AM, Corinna Vinschen wrote: On Dec 19 18:36, Ken Brown wrote: I think I'm seeing a similar confusion in mmap_is_attached_or_noreserve().  I'm tired now and am having trouble sorting out exactly what that function is doing. noreserve() pages are MAP_RESERVE pages. Trying to

Re: A few quick mmap questions

2024-12-20 Thread Corinna Vinschen
On Dec 19 18:36, Ken Brown wrote: > Hi Corinna, > > I plan to keep working on mmap issues while you're on vacation. A few > questions have come up while I've been looking at the code. > > 1. The function is_mmapped_region() doesn't seem to be used anywhere. Is it > OK to remove it? Yes. Commit

A few quick mmap questions

2024-12-19 Thread Ken Brown
Hi Corinna, I plan to keep working on mmap issues while you're on vacation. A few questions have come up while I've been looking at the code. 1. The function is_mmapped_region() doesn't seem to be used anywhere. Is it OK to remove it? 2. I can't find any uses of filler pages, in spite of t