Re: [PATCH 2.6.13 5/14] sas-class: sas_discover.c Discover process (end devices)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/13/05 17:37, Stefan Richter wrote:
> Luben Tuikov wrote:
> 
>>I've never seen the symbols "lun".
> 
> 
> It is merely an encoding of a variable name or struct member name 
> according to the coding style spec.

I appreciate your insight.

I've never seen a "lun" in _any_ spec.  It is always abbreviated
"LUN".

"lun" exists only in the SCSI Core.  Even other OSs use "LUN".

So after a while, after someone reads enough specs, they see
only "LUN".  "lun" seems foreign.

> 
>>"task->ssp_task.LUN"
> 
> 
> But SSP is a TLA too, isn't it? ;-)

Not quite.  I can actually _see_ "LUN" in the frame.

As I said, after a while it becomes second nature to you,
due to the layout of the frame you're working with.

The pattern that the brain sees is "LUN": in the
transport frame and in the code.

	Luben

P.S. Trust me, using "lun" would be quite ugly and it would
show that whoever coded it has had little experience reading
SCSI specs.  What you want to use is "u8 LUN[8]".

PPS. I hope I don't have to put up this sort of convincing
emails back and forth for each and every little thing.  We'd
get nowhere, no code will be written and no hardware would
work.


-
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]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux