[PATCH v3] staging: lustre: libcfs: Remove new typedef in bitmap.h

Lisa Nguyen lisa at xenapiadmin.com
Sun Oct 20 16:08:29 UTC 2013


On Sun, Oct 20, 2013 at 12:43 AM, Greg KH <gregkh at linuxfoundation.org> wrote:
> On Sat, Oct 19, 2013 at 04:15:25PM -0700, Lisa Nguyen wrote:
>> On Sat, Oct 19, 2013 at 2:39 PM, Dan Carpenter <dan.carpenter at oracle.com> wrote:
>> > Your patch is great, but actually, are any of the functions in this
>> > header used?  `grep cfs_bitmap drivers/staging/lustre/ -Rl` only shows
>> > this header file.
>> >
>> > Perhaps we can delete the whole thing?
>> >
>> > regards,
>> > dan carpenter
>>
>> Ah, good point. I didn't think about that. I ran your grep command and
>> got the same results.
>>
>> Since that is the case, is there a specific protocol I have to follow
>> and suggest for this particular header file to be deleted? It's my
>> first time running into this situation.
>
> Just email saying, "Ignore this patch, please replace it with this newer
> one", or just do a "v4" that does this as well.
>
> thanks,
>
> greg k-h

Greg, please ignore this patch as I have submitted a new one to be
applied. The new patch title should say, "Remove bitmap.h file".

Dan, thank you for your suggestion.

--Lisa


More information about the devel mailing list