Re: Configfs usb mass_storage device always reports 8 LUNs ?

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

 





On 07/02/2015 09:14 PM, Felipe Balbi wrote:
On Thu, Jul 02, 2015 at 09:12:30PM +0200, Krzysztof Opasiak wrote:


On 07/02/2015 07:16 PM, Michal Nazarewicz wrote:
On 07/02/2015 05:25 PM, Michal Nazarewicz wrote:
Since I think we agreed that luns should just be turned into an array
with static length the above won’t be too far from that goal.

On Thu, Jul 02 2015, Krzysztof Opasiak wrote:
True, just started working on it.

I just got this to compile but haven’t tested it past that.  Feel free
to grab it or whatever:


Ohh, I was quite busy with testing (and debugging) my patches and didn't
notice this message, sorry...

I have run through this patch and didn't find any serious issues.
But in my opinion instead of editing set_nluns() we could simply drop it and
simplify legacy gadgets code as we do almost nothing in this function.

for the -rc cycle we need a minimal fix, though. Simplification,
refactoring and whatever else, needs to be merged during merge window.


Ok, sure. So the easiest way to fix problems in previously applied patch would be as Michal suggested here: [1]. Putting 1 and 2 together should be fine as a minimal fix for GET_MAX_LUNS issue.

1 - http://marc.info/?l=linux-usb&m=143585072403772&w=2
2 - http://marc.info/?l=linux-usb&m=143584114100542&w=2

Best regards,

--
Krzysztof Opasiak
Samsung R&D Institute Poland
Samsung Electronics
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux