[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <48B226CE.2060700@cs.columbia.edu>
Date: Sun, 24 Aug 2008 23:28:14 -0400
From: Oren Laadan <orenl@...columbia.edu>
To: Louis.Rilling@...labs.com
CC: dave@...ux.vnet.ibm.com, arnd@...db.de, jeremy@...p.org,
linux-kernel@...r.kernel.org, containers@...ts.linux-foundation.org
Subject: Re: [RFC v2][PATCH 8/9] File descriprtors - dump state
Louis Rilling wrote:
> On Wed, Aug 20, 2008 at 11:07:16PM -0400, Oren Laadan wrote:
>> Dump the files_struct of a task with 'struct cr_hdr_files', followed by
>> all open file descriptors. Since FDs can be shared, they are assigned a
>> tag and registered in the object hash.
>>
>> For each open FD there is a 'struct cr_hdr_fd_ent' with the FD, its tag
>> and its close-on-exec property. If the FD is to be saved (first time)
>> then this is followed by a 'struct cr_hdr_fd_data' with the FD state.
>> Then will come the next FD and so on.
>>
>> This patch only handles basic FDs - regular files, directories and also
>> symbolic links.
>
> [...]
[...]
>
>> @@ -114,4 +125,24 @@ struct cr_hdr_vma {
>>
>> } __attribute__ ((aligned (8)));
>>
>> +struct cr_hdr_files {
>> + __u32 tag; /* sharing identifier */
>> + __u32 nfds;
>> +} __attribute__ ((aligned (8)));
>> +
>> +struct cr_hdr_fd_ent {
>> + __u32 tag;
>> + __u16 fd;
>> + __u16 close_on_exec;
>> +} __attribute__ ((aligned (8)));
>> +
>> +struct cr_hdr_fd_data {
>> + __u16 how;
>> + __u16 f_mode;
>> + __u32 f_flags;
>> + __u32 f_uid, f_gid;
>
> We are not at a 64bits boundary here. Should add one __32 padding or reorder the
> fields.
Actually this is ok - note there are two fields on fourth row. I'll
change to one field per row so it's easily seen.
Thanks,
Oren.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists