perhaps i'm just being clueless about the authorship protocol here,
but i'm a bit hacked off by noticing that at least one submitted patch
of mine was apparently re-submitted (albeit slightly modified) a few
days later by another poster and applied under that poster's name.
on sun, dec 3, i submitted to the list:
http://marc.theaimsgroup.com/?l=linux-kernel&m=116516635728664&w=2
and yet, just by accident this morning, i see the log for that file
ipc/sem.c contains:
======================================================
commit 4668edc334ee90cf50c382c3e423cfc510b5a126
Author: Burman Yan <[email protected]>
Date: Wed Dec 6 20:38:51 2006 -0800
[PATCH] kernel core: replace kmalloc+memset with kzalloc
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
diff --git a/ipc/sem.c b/ipc/sem.c
index 21b3289..d3e12ef 100644
--- a/ipc/sem.c
+++ b/ipc/sem.c
@@ -1070,14 +1070,13 @@ static struct sem_undo *find_undo(struct
ipc_rcu_getref(sma);
sem_unlock(sma);
- new = (struct sem_undo *) kmalloc(sizeof(struct sem_undo) + sizeof(short)*nsems, GFP_KERNEL);
+ new = kzalloc(sizeof(struct sem_undo) + sizeof(short)*nsems, GFP_KERNEL);
if (!new) {
ipc_lock_by_ptr(&sma->sem_perm);
ipc_rcu_putref(sma);
sem_unlock(sma);
return ERR_PTR(-ENOMEM);
}
- memset(new, 0, sizeof(struct sem_undo) + sizeof(short)*nsems);
new->semadj = (short *) &new[1];
new->semid = semid;
======================================================
admittedly, mr. yan's patch is technically cleaner since it removes
the superfluous cast applied to kzalloc(). however, i very
*deliberately* left that cast in, and i explained why a couple days
later here in another context:
http://marc.theaimsgroup.com/?l=linux-kernel&m=116553652920469&w=2
quite simply, as per the guidelines given for creating and
submitting kernel patches, i'm trying to keep each submission
well-defined, as i was going to follow up the above with another
submission to remove *all* superfluous casts in one fell swoop. but
it's not just a matter of proper patch attribution.
i've submitted a number of patches recently and, every time i do a
"git pull", i check the log to see if any of them have been applied so
i can delete them from my personal "submitted but not applied"
directory. if they've been applied by another author, then naturally
i'll never notice and i'll keep wondering about the delay.
so what's the protocol here? are more senior kernel developers
allowed to poach on my patch submissions, tidy them up slightly, then
drop any attribution to me? enquiring minds *definitely* want this
cleared up.
rday
p.s. it's possible that this is all just a wild coincidence, of
course. stranger things have happened.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]