Re: [PATCH 1/9] exofs: osd Swiss army knife

From: Boaz Harrosh
Date: Wed Dec 31 2008 - 10:33:57 EST


Andrew Morton wrote:
>> +int prepare_get_attr_list_add_entry(struct osd_request *req,
>> + uint32_t page_num,
>> + uint32_t attr_num,
>> + uint32_t attr_len)
>> +{
>> + struct osd_attr attr = {
>> + .page = page_num,
>
> Kernel developers expect a field called "page" to have type `struct
> page *'. osd_attr.page is thus designed to confuse.
>
>> ...
>>
>

Rant below (can be ignored):
This single fix will cause a massive change to the open-osd
initiator patchset, (18 patches), and resubmission .I made the mistake
because this name originates from a file that all naming conventions
are taken from the OSD standard text. However this is no excuse
for using a well known Kernel construct name. I will fix it. And
will be more careful in the future.

Thanks
Boaz

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/