[syslinux] [PATCH 2/2] core/fs: Add support for Unix File system 1/2.

Raphael S Carvalho raphael.scarv at gmail.com
Thu May 29 08:29:47 PDT 2014


On Thu, May 29, 2014 at 11:40 AM, H. Peter Anvin <hpa at zytor.com> wrote:
> On 05/29/2014 07:36 AM, Raphael S Carvalho wrote:
>> On Thu, May 29, 2014 at 11:30 AM, H. Peter Anvin <hpa at zytor.com> wrote:
>>> On 05/29/2014 07:20 AM, Raphael S.Carvalho wrote:
>>>> +static int ufs_readlink(struct inode *inode, char *buf)
>>>> +{
>>>> +    ufs_debug("ufs_readlink\n");
>>>> +    return inode->size;
>>>> +}
>>>
>>> Something missing here?
>> Yes, implementation. It's just a placeholder until I implement it. You
>> can see that I didn't even set it into fs_ops. At that time, I think
>> Matt removed it himself as compiler complained about it being unused.
>>>
>
> Yes, it did.  However, it would be better to implement it.  As far as I
> know, UFS symlinks are very similar to ext* symlinks (not surprising
> since ext2 was heavily inspired by UFS): symlinks below a specific size
> is stored in the inode overlaying the block pointers, and otherwise the
> symlink is just an ordinary file.
According to the ufs1/2 specification, a symbolic link stores the path
of the destination file in a fragment (1024 bytes), or in the 60/120
bytes used for ufs1/2 block pointers respectively.
>
>         -hpa
>
>



-- 
Raphael S. Carvalho


More information about the Syslinux mailing list