[PATCH] MPSAFE/LOOKUP_SHARED cd9660
Paul B. Mahol
onemda at gmail.com
Wed Dec 10 23:54:23 UTC 2008
On 12/10/08, John Baldwin <jhb at freebsd.org> wrote:
> On Wednesday 10 December 2008 12:22:43 pm Paul B. Mahol wrote:
>> On 12/9/08, John Baldwin <jhb at freebsd.org> wrote:
>> > The RRIP stuff is all done in cd9660_vget_internal() under an exclusive
>> > lock.
>> > It could be a property of the ISO image. "PX" holds permissions (owner,
>> > etc.). Do you get the same messages w/o the patch with the same ISO
> image /
>> > CD?
>> > --
>> > John Baldwin
>> I searched little for this message and found kern/63446 PR interesting
>> Caused by cd9660_vnops.c rev. 1.77. VOP_READDIR returns bogus
>> d_fileno, VFS_VGET on this value returns bogus vnode with zeroed
>> I think that whatever locking is done is done wrong.
> That issue isn't a locking issue, it's an issue with VOP_READDIR() using a
> different meaning for i-node numbers than VFS_VGET(), and would happen
> regardless of any Giant or vnode locking.
Yes, I know that that issue/PR doesnt have anything to do with locking.
But displaying bogus messages and still having
functional cd9660 for general use means that something
is not correctly locked.
More information about the freebsd-current