ch4中os/src/mm/page_table.rs内find_pte函数需要调换2个if块顺序避免潜在BUG

[PassingThroughStarDust]

os/mm/page_table.rs中的find_pte函数，如果参数是一个没有被映射过的VPN，函数也会返回PTE，调用该函数的translate函数（page_table.rs内）没有检查PTE合法的手段，原OS代码不报错是因为所查询的PPN都在合法范围内。
原代码：

fn find_pte(&self, vpn: VirtPageNum) -> Option<&mut PageTableEntry> {
        let idxs = vpn.indexes();
        let mut ppn = self.root_ppn;
        let mut result: Option<&mut PageTableEntry> = None;
        for (i, idx) in idxs.iter().enumerate() {
            let pte = &mut ppn.get_pte_array()[*idx];
            if i == 2 {
                result = Some(pte);
                break;
            }
            if !pte.is_valid() {
                return None;
            }
            ppn = pte.ppn();
        }
        result
    }

修改后：

fn find_pte(&self, vpn: VirtPageNum) -> Option<&mut PageTableEntry> {
        let idxs = vpn.indexes();
        let mut ppn = self.root_ppn;
        let mut result: Option<&mut PageTableEntry> = None;
        for (i, idx) in idxs.iter().enumerate() {
            let pte = &mut ppn.get_pte_array()[*idx];
            if !pte.is_valid() {
                return None;
            }
            if i == 2 {
                result = Some(pte);
                break;
            }
            ppn = pte.ppn();
        }
        result
    }

[wyfcyx]

这里确实有问题，多谢指出。不过translate是可以检查PTE的合法性的，直接调用pte.is_valid就行了，目前没有检查。我需要想一下这个检查是放在find_pte里面还是translate里面。

[PassingThroughStarDust]

这里确实有问题，多谢指出。不过translate是可以检查PTE的合法性的，直接调用pte.is_valid就行了，目前没有检查。我需要想一下这个检查是放在find_pte里面还是translate里面。

好的，谢谢。另外还有个小的问题，os/src/syscall/fs.rs中的sys_write函数打印非ASCII字符可能会失败，因为是将translate_byte_buffer返回的字符串碎片分别打印的，对于超过一个字节表示的文字编码（比如中文），可能会在编码中间截断导致打印错误。