* Roman Zippel <[email protected]> wrote:
> > Sorry if it was previously your idea and if we didnt credit you for it.
> > I did not keep track of each word said in these endless mail threads. We
> > credited every suggestion and idea which we picked up from you, see our
> > previous emails. If we missed one, it was definitely not intentional.
>
> http://marc.theaimsgroup.com/?l=linux-kernel&m=112755827327101
> http://marc.theaimsgroup.com/?l=linux-kernel&m=112760582427537
>
> A bit later ktime_t looked pretty much like the 64bit part of my
> ktimespec.
and Thomas credited you for that point in his announcement:
" Roman pointed out that the penalty for some architectures
would be quite big when using the nsec_t (64bit) scalar time
storage format. "
http://marc.theaimsgroup.com/?l=linux-kernel&m=112794069605537&w=2
also, once you came up with actual modifications to the ktimers concept
(the ptimer queue) we noticed a further refinement of ktime_t in that
code: the elimination of the plain scalar type. We gave you credit
again:
" - eliminate the plain s64 scalar type, and always use the union.
This simplifies the arithmetics. Idea from Roman Zippel. "
see:
http://marc.theaimsgroup.com/?l=linux-kernel&m=113339663027117&w=2
http://marc.theaimsgroup.com/?l=linux-kernel&m=113382965626004&w=2
we couldnt take your actual patch/code though, due to the way you
created the ptimer queue: you took our ktimer queue, added a dozen
changes to it (intermixed, without keeping/disclosing the changes), then
you split up the resulting queue. This structure made it largely
incompatible with our queue, the diff between ktimers and ptimers was
larger than the two patches themselves, due to the stacked changed! This
is not a complaint - we are happy you are writing ktimer based code -
this is just an explanation of why we couldnt take the code/patch as-is
but had to redo that portion from scratch, based on your idea.
> I don't won't to imply any intention, but please try to see this from
> my perspective, after this happens a number of times.
What the hell are you talking about? I bloody well know how it all
happened, because i did those simplifications to ktime.h myself, and i
added the changelog too, crediting you for the idea.
Ingo
-
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]
