Skip to content

Prove assume in Entry::replace - #202

Closed
Marsman1996 wants to merge 1 commit into
asterinas:mainfrom
Marsman1996:entry_replace
Closed

Prove assume in Entry::replace#202
Marsman1996 wants to merge 1 commit into
asterinas:mainfrom
Marsman1996:entry_replace

Conversation

@Marsman1996

Copy link
Copy Markdown
Collaborator

No description provided.

@Marsman1996
Marsman1996 marked this pull request as draft September 26, 2025 11:12
@Marsman1996

Copy link
Copy Markdown
Collaborator Author

For

assume(self.pte.pte_paddr_spec() == index_pte_paddr(
            self.node.paddr() as int,
            self.idx as int,
        ));
  • Model: GPT-5-Codex
  • Repeat: 8
  • Input: 85,542
  • Output: 1,824

@Marsman1996

Copy link
Copy Markdown
Collaborator Author

One bad news is that LLM would add exec code sometimes, and in this case it adds

let pte_addr: usize = self.node.paddr() + self.idx * exec::SIZEOF_PAGETABLEENTRY;
self.pte.set_paddr(pte_addr);

I'm working on this...

@Marsman1996 Marsman1996 closed this Dec 5, 2025
@Marsman1996
Marsman1996 deleted the entry_replace branch January 8, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant