Messages in this thread | | | Subject | Re: [RFC PATCH 0/19] A kernel-level configfs enabled generic target engine for Linux v2.6.32 | From | "Nicholas A. Bellinger" <> | Date | Fri, 11 Sep 2009 20:08:04 -0700 |
| |
On Fri, 2009-09-11 at 19:37 -0700, Daniel Walker wrote: > On Fri, 2009-09-11 at 18:55 -0700, Nicholas A. Bellinger wrote: > > There is cleanup work continuing to make the TCM code to use > > dprintk(), > > and some include cleanups, and other minor fixes. The code has also > > been run through checkpatch again since the posting in April, and all > > errors have been removed, with the exception of one ConfigFS macro > > define triggering a checkpatch false positive. >
Greetings Daniel,
> > One show stopping issue is that your adding new semaphore usage in patch > 5 and 16 (and in 16 it's a LOCKED one wtf!!). Which you should not be > doing .. You need to really evaluate the warnings from checkpatch, cause > those still have meaning.. >
Ok, I will have a look at converting the HBA semaphore to a mutex in patch 5, and use some single use waitqueues in patch 16. Btw, the ones using init_MUTEX_LOCKED() in the latter source file are being used when shutting down target_core_mod allocated kernel threads to sychronize between the stopping processing threads and configfs process context that waitqueues would work better for. Thanks for pointing this out!
> It's basically telling you what you should be doing in the warnings.. > For instance, don't add new semaphore usage, and it tells you what you > need to be using in place of them..
Ok, so I will also do the struct sempahore -> struct mutex conversion where it makes sense to protect lists and see about the rest..
> Also typedefs are frowned upon, so > you could remove those.. >
I was thinking about going through the typedefs with a sed chainsaw, but I decided against for this TCM patch series because it makes keeping backward ports to <= v2.6.31 slightly easier to maintain for me for the current users of LIO kernel code.
However, I have been starting to not use typedef for the newest ALUA and pieces of PR code, and I would be happy to have another look at typedef style comments on this once the other larger interest items are on the way to being resolved.
Thanks!
--nab
> Daniel > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
| |