Re: [PATCH] Mass Storage Gadget: Handle eject request

[MichaÅ Nazarewicz]

> 2010/4/21 MichaÅ Nazarewicz <m.nazarewicz@xxxxxxxxxxx>
> > Clearly, suspend seems like a state of the mass storage function
> > as a whole rather then attribute of each logical unit so I think
> > it'll be better to make it global for the mass storage function
> > rather then per-LUN.
> >
> > Even more so, it's a state of the whole gadget so in my opinion the
> > whole code should be moved the the gadget rather then kept in
> > a function!
> >
> > Going even further, I would propose an sysfs entry to be added to
> > the composite framework as a single generic interface rather then
> > hacking it in each gadget/function that might need to export this
> > information to user space.
> >
> > This provided that there is no easy way of gaining that information
> > in user space through same other sysfs entry.

On Wed, 21 Apr 2010 14:35:57 +0200, Chouteau Fabien <fabien.chouteau@xxxxxxxxx> wrote:
> You're right, the suspended state should be handled in the composite
> framework, I'm going resend the patch with this modification.

That'll be great.

> I have a question about the mass/file storage gadget, why is there a
> g_mass_storage gadget and a g_file_storage gadget? Code and feature seems
> redundant.

My Mass Storage Function is relatively young and thus not so mature as
File Storage Gadget.  As a matter of fact, if one were to choose between
FSG and MSG then FSG is probably a better choice.  MSG was provided to
test the MSF and show how to write composite gadgets using it.  The
strength of MSF is of course that it is a composite function hence can
be mixed with other functions.  Maybe in the future, when MSF (and MSG)
proves stability, FSG will be removed from the kernel but for now it
has been decided to let it be there.  You may refer to my discussion with
Alan when I was posting the code.

> btw, the eject code of this patch comes from file_storage.c

It may be a good idea to point that in a comment.

Or maybe even extract the common do_*() functions to storage_common.c.
I always felt like the do_*() functions should be in the
storage_common.c but unfortunately there are many subtle differences,
which make those functions differ in little details between MSF and
FSG.

--
Best regards,                                           _     _
 .o. | Liege of Serenely Enlightened Majesty of       o' \,=./ `o
 ..o | Computer Science,  MichaÅ "mina86" Nazarewicz     (o o)
 ooo +---[mina86@xxxxxxxxxx]---[mina86@jabber.org]---ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/