Re: [PATCH 0/8] scsi: convert tasklets to use new tasklet_setup()

quoted

quoted

quoted

Commit 12cc923f1ccc ("tasklet: Introduce new initialization
API")' introduced a new tasklet initialization API. This series
converts all the scsi drivers to use the new tasklet_setup() API

I've got to say I agree with Jens, this was a silly obfuscation:

+#define from_tasklet(var, callback_tasklet, tasklet_fieldname) \
+       container_of(callback_tasklet, typeof(*var),
tasklet_fieldname)

Just use container_of directly since we all understand what it
does.

But then the lines get really long, wrapped, etc.

I really don't think that's a problem but if you want to add a new
generic container_of that does typeof instead of insisting on the type,
I'd be sort of OK with that ... provided you don't gratuitously alter
the argument order.

The thing I object to is that this encourages everyone to roll their
own unnecessary container_of type macros in spite of the fact that it's
function is wholly generic.  It's fine if you're eliminating one of the
arguments, or actually making the macro specific to the type, but in
this case you're not, you're making a completely generic macro where
the name is the only thing that's specific to this case.

quoted

 This is what the timer_struct conversion did too (added a
container_of wrapper), so I think it makes sense here too.

I didn't see that one to object to it ...

Since we could not get the generic API accepted, can I send out V2
which would use container_of()?

Thanks,

-- 
       - Allen