On Thu, Sep 10, 2015 at 11:20 AM, Jon Hunter wrote:
>
> On 10/09/15 18:10, Grant Grundler wrote:
...
>> struct mmc_ioc_multi_cmd {
>> __u64 num_of_cmds;
>> struct mmc_ioc_cmd cmds[0];
>> };
>>
>> I think this would work just as well. But doesn't "pointer to an
>> array"
On 10/09/15 18:10, Grant Grundler wrote:
> On Thu, Sep 10, 2015 at 1:24 AM, Jon Hunter wrote:
> ...
>>> - you have some implicit padding after the structure and should replace that
>>> with explictit pad bytes to extend the structure to a multiple of its
>>> alignment (8 bytes).
>>
>> Would
On Thu, Sep 10, 2015 at 1:24 AM, Jon Hunter wrote:
...
>> - you have some implicit padding after the structure and should replace that
>> with explictit pad bytes to extend the structure to a multiple of its
>> alignment (8 bytes).
>
> Would padding with __u32 at the end be sufficient here? I
On Thursday 10 September 2015 09:24:19 Jon Hunter wrote:
> On 09/09/15 21:22, Arnd Bergmann wrote:
> > On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
> >> On 09/09/15 16:56, Arnd Bergmann wrote:
> >>> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
>
> > - you have some
Hi Arnd,
On 09/09/15 21:22, Arnd Bergmann wrote:
> On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
>>
>> On 09/09/15 16:56, Arnd Bergmann wrote:
>>> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
+
+ idata = kcalloc(mcci.num_of_cmds, sizeof(*idata),
Hi Arnd,
On 09/09/15 21:22, Arnd Bergmann wrote:
> On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
>>
>> On 09/09/15 16:56, Arnd Bergmann wrote:
>>> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
+
+ idata = kcalloc(mcci.num_of_cmds, sizeof(*idata),
On Thursday 10 September 2015 09:24:19 Jon Hunter wrote:
> On 09/09/15 21:22, Arnd Bergmann wrote:
> > On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
> >> On 09/09/15 16:56, Arnd Bergmann wrote:
> >>> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
>
> > - you have some
On Thu, Sep 10, 2015 at 1:24 AM, Jon Hunter wrote:
...
>> - you have some implicit padding after the structure and should replace that
>> with explictit pad bytes to extend the structure to a multiple of its
>> alignment (8 bytes).
>
> Would padding with __u32 at the end
On 10/09/15 18:10, Grant Grundler wrote:
> On Thu, Sep 10, 2015 at 1:24 AM, Jon Hunter wrote:
> ...
>>> - you have some implicit padding after the structure and should replace that
>>> with explictit pad bytes to extend the structure to a multiple of its
>>> alignment
On Thu, Sep 10, 2015 at 11:20 AM, Jon Hunter wrote:
>
> On 10/09/15 18:10, Grant Grundler wrote:
...
>> struct mmc_ioc_multi_cmd {
>> __u64 num_of_cmds;
>> struct mmc_ioc_cmd cmds[0];
>> };
>>
>> I think this would work just as well. But doesn't
On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
>
> On 09/09/15 16:56, Arnd Bergmann wrote:
> > On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
> >> +
> >> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
> >> + if (!idata) {
> >> +
On 09/09/15 16:56, Arnd Bergmann wrote:
> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
>> +
>> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
>> + if (!idata) {
>> + err = -ENOMEM;
>> + goto cmd_err;
>> + }
>> +
>> +
On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
> +
> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
> + if (!idata) {
> + err = -ENOMEM;
> + goto cmd_err;
> + }
> +
> + cmds = (struct mmc_ioc_cmd __user *)(unsigned
From: Seshagiri Holi
Certain eMMC devices allow vendor specific device information to be read
via a sequence of vendor commands. These vendor commands must be issued
in sequence and an atomic fashion. One way to support this would be to
add an ioctl function for sending a sequence of commands to
On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
> +
> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
> + if (!idata) {
> + err = -ENOMEM;
> + goto cmd_err;
> + }
> +
> + cmds = (struct mmc_ioc_cmd __user *)(unsigned
From: Seshagiri Holi
Certain eMMC devices allow vendor specific device information to be read
via a sequence of vendor commands. These vendor commands must be issued
in sequence and an atomic fashion. One way to support this would be to
add an ioctl function for sending a
On 09/09/15 16:56, Arnd Bergmann wrote:
> On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
>> +
>> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
>> + if (!idata) {
>> + err = -ENOMEM;
>> + goto cmd_err;
>> + }
>> +
>> +
On Wednesday 09 September 2015 17:44:54 Jon Hunter wrote:
>
> On 09/09/15 16:56, Arnd Bergmann wrote:
> > On Wednesday 09 September 2015 16:06:01 Jon Hunter wrote:
> >> +
> >> + idata = kcalloc(mcci.num_of_cmds, sizeof(*idata), GFP_KERNEL);
> >> + if (!idata) {
> >> +
18 matches
Mail list logo