[v2,01/22] block: add eMMC block device type

Commit Message

Sai Pavan Boddu Feb. 22, 2021, 8:20 a.m. UTC

From: Vincent Palatin <[e-mail]>

Add new block device type.

Signed-off-by: Vincent Palatin <[e-mail]>
[SPB: Rebased over 5.1 version]
Signed-off-by: Sai Pavan Boddu <[e-mail]>
Signed-off-by: Joel Stanley <[e-mail]>
Signed-off-by: Cédric Le Goater <[e-mail]>
Reviewed-by: Alistair Francis <[e-mail]>
---
 include/sysemu/blockdev.h | 1 +
 blockdev.c                | 1 +
 2 files changed, 2 insertions(+)

Comments

Philippe Mathieu-Daudé Feb. 22, 2021, 12:04 p.m. UTC | #1

On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> From: Vincent Palatin <[e-mail]>
> 
> Add new block device type.
> 
> Signed-off-by: Vincent Palatin <[e-mail]>
> [SPB: Rebased over 5.1 version]
> Signed-off-by: Sai Pavan Boddu <[e-mail]>
> Signed-off-by: Joel Stanley <[e-mail]>
> Signed-off-by: Cédric Le Goater <[e-mail]>
> Reviewed-by: Alistair Francis <[e-mail]>
> ---
>  include/sysemu/blockdev.h | 1 +
>  blockdev.c                | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> index 3b5fcda..eefae9f 100644
> --- a/include/sysemu/blockdev.h
> +++ b/include/sysemu/blockdev.h
> @@ -24,6 +24,7 @@ typedef enum {
>       */
>      IF_NONE = 0,
>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
> +    IF_EMMC,
>      IF_COUNT
>  } BlockInterfaceType;
>  
> diff --git a/blockdev.c b/blockdev.c
> index cd438e6..390d43c 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
>      [IF_SD] = "sd",
>      [IF_VIRTIO] = "virtio",
>      [IF_XEN] = "xen",
> +    [IF_EMMC] = "emmc",
>  };

We don't need to introduce support for the legacy -drive magic.

-device should be enough for this device, right?

Markus Armbruster Feb. 22, 2021, 12:19 p.m. UTC | #2

Philippe Mathieu-Daudé <[e-mail]> writes:

> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
>> From: Vincent Palatin <[e-mail]>
>> 
>> Add new block device type.
>> 
>> Signed-off-by: Vincent Palatin <[e-mail]>
>> [SPB: Rebased over 5.1 version]
>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
>> Signed-off-by: Joel Stanley <[e-mail]>
>> Signed-off-by: Cédric Le Goater <[e-mail]>
>> Reviewed-by: Alistair Francis <[e-mail]>
>> ---
>>  include/sysemu/blockdev.h | 1 +
>>  blockdev.c                | 1 +
>>  2 files changed, 2 insertions(+)
>> 
>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
>> index 3b5fcda..eefae9f 100644
>> --- a/include/sysemu/blockdev.h
>> +++ b/include/sysemu/blockdev.h
>> @@ -24,6 +24,7 @@ typedef enum {
>>       */
>>      IF_NONE = 0,
>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
>> +    IF_EMMC,
>>      IF_COUNT
>>  } BlockInterfaceType;
>>  
>> diff --git a/blockdev.c b/blockdev.c
>> index cd438e6..390d43c 100644
>> --- a/blockdev.c
>> +++ b/blockdev.c
>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
>>      [IF_SD] = "sd",
>>      [IF_VIRTIO] = "virtio",
>>      [IF_XEN] = "xen",
>> +    [IF_EMMC] = "emmc",
>>  };
>
> We don't need to introduce support for the legacy -drive magic.
>
> -device should be enough for this device, right?

External interface extensions need rationale: why do we want / need it?
The commit message neglects to provide one.

Even more so when the interface in question is in a state like -drive
is.

Dr. David Alan Gilbert Feb. 22, 2021, 1:16 p.m. UTC | #3

* Markus Armbruster ([e-mail]) wrote:
> Philippe Mathieu-Daudé <[e-mail]> writes:
> 
> > On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> >> From: Vincent Palatin <[e-mail]>
> >> 
> >> Add new block device type.
> >> 
> >> Signed-off-by: Vincent Palatin <[e-mail]>
> >> [SPB: Rebased over 5.1 version]
> >> Signed-off-by: Sai Pavan Boddu <[e-mail]>
> >> Signed-off-by: Joel Stanley <[e-mail]>
> >> Signed-off-by: Cédric Le Goater <[e-mail]>
> >> Reviewed-by: Alistair Francis <[e-mail]>
> >> ---
> >>  include/sysemu/blockdev.h | 1 +
> >>  blockdev.c                | 1 +
> >>  2 files changed, 2 insertions(+)
> >> 
> >> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> >> index 3b5fcda..eefae9f 100644
> >> --- a/include/sysemu/blockdev.h
> >> +++ b/include/sysemu/blockdev.h
> >> @@ -24,6 +24,7 @@ typedef enum {
> >>       */
> >>      IF_NONE = 0,
> >>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
> >> +    IF_EMMC,
> >>      IF_COUNT
> >>  } BlockInterfaceType;
> >>  
> >> diff --git a/blockdev.c b/blockdev.c
> >> index cd438e6..390d43c 100644
> >> --- a/blockdev.c
> >> +++ b/blockdev.c
> >> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> >>      [IF_SD] = "sd",
> >>      [IF_VIRTIO] = "virtio",
> >>      [IF_XEN] = "xen",
> >> +    [IF_EMMC] = "emmc",
> >>  };
> >
> > We don't need to introduce support for the legacy -drive magic.
> >
> > -device should be enough for this device, right?
> 
> External interface extensions need rationale: why do we want / need it?
> The commit message neglects to provide one.
> 
> Even more so when the interface in question is in a state like -drive
> is.

I wouldn't be too nasty about -drive;  for me I still find it the
easiest way to start a VM.

Dave

Philippe Mathieu-Daudé Feb. 22, 2021, 1:23 p.m. UTC | #4

On 2/22/21 2:16 PM, Dr. David Alan Gilbert wrote:
> * Markus Armbruster ([e-mail]) wrote:
>> Philippe Mathieu-Daudé <[e-mail]> writes:
>>
>>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
>>>> From: Vincent Palatin <[e-mail]>
>>>>
>>>> Add new block device type.
>>>>
>>>> Signed-off-by: Vincent Palatin <[e-mail]>
>>>> [SPB: Rebased over 5.1 version]
>>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
>>>> Signed-off-by: Joel Stanley <[e-mail]>
>>>> Signed-off-by: Cédric Le Goater <[e-mail]>
>>>> Reviewed-by: Alistair Francis <[e-mail]>
>>>> ---
>>>>  include/sysemu/blockdev.h | 1 +
>>>>  blockdev.c                | 1 +
>>>>  2 files changed, 2 insertions(+)
>>>>
>>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
>>>> index 3b5fcda..eefae9f 100644
>>>> --- a/include/sysemu/blockdev.h
>>>> +++ b/include/sysemu/blockdev.h
>>>> @@ -24,6 +24,7 @@ typedef enum {
>>>>       */
>>>>      IF_NONE = 0,
>>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
>>>> +    IF_EMMC,
>>>>      IF_COUNT
>>>>  } BlockInterfaceType;
>>>>  
>>>> diff --git a/blockdev.c b/blockdev.c
>>>> index cd438e6..390d43c 100644
>>>> --- a/blockdev.c
>>>> +++ b/blockdev.c
>>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
>>>>      [IF_SD] = "sd",
>>>>      [IF_VIRTIO] = "virtio",
>>>>      [IF_XEN] = "xen",
>>>> +    [IF_EMMC] = "emmc",
>>>>  };
>>>
>>> We don't need to introduce support for the legacy -drive magic.
>>>
>>> -device should be enough for this device, right?
>>
>> External interface extensions need rationale: why do we want / need it?
>> The commit message neglects to provide one.
>>
>> Even more so when the interface in question is in a state like -drive
>> is.
> 
> I wouldn't be too nasty about -drive;  for me I still find it the
> easiest way to start a VM.

But eMMC isn't a bus where you can plug drives, it is soldered
on-board and is mmio mapped to a fixed address. I don't see the
point of having a drive interface for it...

Sai Pavan Boddu Feb. 22, 2021, 4:17 p.m. UTC | #5

Hi Philippe,

> -----Original Message-----
> From: Philippe Mathieu-Daudé <[e-mail]>
> Sent: Monday, February 22, 2021 6:54 PM
> To: Dr. David Alan Gilbert <[e-mail]>; Markus Armbruster
> <[e-mail]>
> Cc: Sai Pavan Boddu <[e-mail]>; Kevin Wolf <[e-mail]>;
> Max Reitz <[e-mail]>; Vladimir Sementsov-Ogievskiy
> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
> <[e-mail]>; Thomas Huth <[e-mail]>; Stefan
> Hajnoczi <[e-mail]>; Peter Maydell <[e-mail]>;
> Alistair Francis <[e-mail]>; Edgar Iglesias <[e-mail]>;
> Luc Michel <[e-mail]>; Paolo Bonzini
> <[e-mail]>; Sai Pavan Boddu <[e-mail]>; qemu-
> [e-mail]; [e-mail]
> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> 
> On 2/22/21 2:16 PM, Dr. David Alan Gilbert wrote:
> > * Markus Armbruster ([e-mail]) wrote:
> >> Philippe Mathieu-Daudé <[e-mail]> writes:
> >>
> >>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> >>>> From: Vincent Palatin <[e-mail]>
> >>>>
> >>>> Add new block device type.
> >>>>
> >>>> Signed-off-by: Vincent Palatin <[e-mail]>
> >>>> [SPB: Rebased over 5.1 version]
> >>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
> >>>> Signed-off-by: Joel Stanley <[e-mail]>
> >>>> Signed-off-by: Cédric Le Goater <[e-mail]>
> >>>> Reviewed-by: Alistair Francis <[e-mail]>
> >>>> ---
> >>>>  include/sysemu/blockdev.h | 1 +
> >>>>  blockdev.c                | 1 +
> >>>>  2 files changed, 2 insertions(+)
> >>>>
> >>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> >>>> index 3b5fcda..eefae9f 100644
> >>>> --- a/include/sysemu/blockdev.h
> >>>> +++ b/include/sysemu/blockdev.h
> >>>> @@ -24,6 +24,7 @@ typedef enum {
> >>>>       */
> >>>>      IF_NONE = 0,
> >>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD,
> >>>> IF_VIRTIO, IF_XEN,
> >>>> +    IF_EMMC,
> >>>>      IF_COUNT
> >>>>  } BlockInterfaceType;
> >>>>
> >>>> diff --git a/blockdev.c b/blockdev.c index cd438e6..390d43c 100644
> >>>> --- a/blockdev.c
> >>>> +++ b/blockdev.c
> >>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> >>>>      [IF_SD] = "sd",
> >>>>      [IF_VIRTIO] = "virtio",
> >>>>      [IF_XEN] = "xen",
> >>>> +    [IF_EMMC] = "emmc",
> >>>>  };
> >>>
> >>> We don't need to introduce support for the legacy -drive magic.
> >>>
> >>> -device should be enough for this device, right?
> >>
> >> External interface extensions need rationale: why do we want / need it?
> >> The commit message neglects to provide one.
> >>
> >> Even more so when the interface in question is in a state like -drive
> >> is.
> >
> > I wouldn't be too nasty about -drive;  for me I still find it the
> > easiest way to start a VM.
> 
> But eMMC isn't a bus where you can plug drives, it is soldered on-board and is
> mmio mapped to a fixed address. I don't see the point of having a drive
> interface for it...
[Sai Pavan Boddu] Yeah, this makes sense but having a drive would be a simple implementation without disturbing much in the sd card emulation code. And its just easy to use, just as how sd cards are inserted.

I need to see, how easy it would be with -device.

Thanks,
Sai Pavan

Dr. David Alan Gilbert Feb. 22, 2021, 4:26 p.m. UTC | #6

* Sai Pavan Boddu ([e-mail]) wrote:
> Hi Philippe,
> 
> > -----Original Message-----
> > From: Philippe Mathieu-Daudé <[e-mail]>
> > Sent: Monday, February 22, 2021 6:54 PM
> > To: Dr. David Alan Gilbert <[e-mail]>; Markus Armbruster
> > <[e-mail]>
> > Cc: Sai Pavan Boddu <[e-mail]>; Kevin Wolf <[e-mail]>;
> > Max Reitz <[e-mail]>; Vladimir Sementsov-Ogievskiy
> > <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> > <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
> > <[e-mail]>; Thomas Huth <[e-mail]>; Stefan
> > Hajnoczi <[e-mail]>; Peter Maydell <[e-mail]>;
> > Alistair Francis <[e-mail]>; Edgar Iglesias <[e-mail]>;
> > Luc Michel <[e-mail]>; Paolo Bonzini
> > <[e-mail]>; Sai Pavan Boddu <[e-mail]>; qemu-
> > [e-mail]; [e-mail]
> > Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> > 
> > On 2/22/21 2:16 PM, Dr. David Alan Gilbert wrote:
> > > * Markus Armbruster ([e-mail]) wrote:
> > >> Philippe Mathieu-Daudé <[e-mail]> writes:
> > >>
> > >>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> > >>>> From: Vincent Palatin <[e-mail]>
> > >>>>
> > >>>> Add new block device type.
> > >>>>
> > >>>> Signed-off-by: Vincent Palatin <[e-mail]>
> > >>>> [SPB: Rebased over 5.1 version]
> > >>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
> > >>>> Signed-off-by: Joel Stanley <[e-mail]>
> > >>>> Signed-off-by: Cédric Le Goater <[e-mail]>
> > >>>> Reviewed-by: Alistair Francis <[e-mail]>
> > >>>> ---
> > >>>>  include/sysemu/blockdev.h | 1 +
> > >>>>  blockdev.c                | 1 +
> > >>>>  2 files changed, 2 insertions(+)
> > >>>>
> > >>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> > >>>> index 3b5fcda..eefae9f 100644
> > >>>> --- a/include/sysemu/blockdev.h
> > >>>> +++ b/include/sysemu/blockdev.h
> > >>>> @@ -24,6 +24,7 @@ typedef enum {
> > >>>>       */
> > >>>>      IF_NONE = 0,
> > >>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD,
> > >>>> IF_VIRTIO, IF_XEN,
> > >>>> +    IF_EMMC,
> > >>>>      IF_COUNT
> > >>>>  } BlockInterfaceType;
> > >>>>
> > >>>> diff --git a/blockdev.c b/blockdev.c index cd438e6..390d43c 100644
> > >>>> --- a/blockdev.c
> > >>>> +++ b/blockdev.c
> > >>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> > >>>>      [IF_SD] = "sd",
> > >>>>      [IF_VIRTIO] = "virtio",
> > >>>>      [IF_XEN] = "xen",
> > >>>> +    [IF_EMMC] = "emmc",
> > >>>>  };
> > >>>
> > >>> We don't need to introduce support for the legacy -drive magic.
> > >>>
> > >>> -device should be enough for this device, right?
> > >>
> > >> External interface extensions need rationale: why do we want / need it?
> > >> The commit message neglects to provide one.
> > >>
> > >> Even more so when the interface in question is in a state like -drive
> > >> is.
> > >
> > > I wouldn't be too nasty about -drive;  for me I still find it the
> > > easiest way to start a VM.
> > 
> > But eMMC isn't a bus where you can plug drives, it is soldered on-board and is
> > mmio mapped to a fixed address. I don't see the point of having a drive
> > interface for it...
> [Sai Pavan Boddu] Yeah, this makes sense but having a drive would be a simple implementation without disturbing much in the sd card emulation code. And its just easy to use, just as how sd cards are inserted.
> 
> I need to see, how easy it would be with -device.


Lets see what your command line looks like for starting it with emmc.

Dave

> Thanks,
> Sai Pavan
>

Sai Pavan Boddu Feb. 23, 2021, 5:35 p.m. UTC | #7

Hi Philippe,

> -----Original Message-----
> From: Philippe Mathieu-Daudé <[e-mail]>
> Sent: Monday, February 22, 2021 5:34 PM
> To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> <[e-mail]>; Vladimir Sementsov-Ogievskiy
> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
> <[e-mail]>; Dr. David Alan Gilbert <[e-mail]>;
> Thomas Huth <[e-mail]>; Stefan Hajnoczi <[e-mail]>;
> Peter Maydell <[e-mail]>; Alistair Francis
> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc Michel
> <[e-mail]>; Paolo Bonzini <[e-mail]>
> Cc: Sai Pavan Boddu <[e-mail]>; [e-mail]; qemu-
> [e-mail]
> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> 
> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> > From: Vincent Palatin <[e-mail]>
> >
> > Add new block device type.
> >
> > Signed-off-by: Vincent Palatin <[e-mail]>
> > [SPB: Rebased over 5.1 version]
> > Signed-off-by: Sai Pavan Boddu <[e-mail]>
> > Signed-off-by: Joel Stanley <[e-mail]>
> > Signed-off-by: Cédric Le Goater <[e-mail]>
> > Reviewed-by: Alistair Francis <[e-mail]>
> > ---
> >  include/sysemu/blockdev.h | 1 +
> >  blockdev.c                | 1 +
> >  2 files changed, 2 insertions(+)
> >
> > diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> > index 3b5fcda..eefae9f 100644
> > --- a/include/sysemu/blockdev.h
> > +++ b/include/sysemu/blockdev.h
> > @@ -24,6 +24,7 @@ typedef enum {
> >       */
> >      IF_NONE = 0,
> >      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO,
> > IF_XEN,
> > +    IF_EMMC,
> >      IF_COUNT
> >  } BlockInterfaceType;
> >
> > diff --git a/blockdev.c b/blockdev.c
> > index cd438e6..390d43c 100644
> > --- a/blockdev.c
> > +++ b/blockdev.c
> > @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> >      [IF_SD] = "sd",
> >      [IF_VIRTIO] = "virtio",
> >      [IF_XEN] = "xen",
> > +    [IF_EMMC] = "emmc",
> >  };
> 
> We don't need to introduce support for the legacy -drive magic.
> 
> -device should be enough for this device, right?
[Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we anyway need blockdev support for this, which would require us the use -drive.

Can you give some pointers, how to approach this ?

Regards,
Sai Pavan

Stefan Hajnoczi Feb. 24, 2021, 11:40 a.m. UTC | #8

On Tue, Feb 23, 2021 at 05:35:20PM +0000, Sai Pavan Boddu wrote:
> Hi Philippe,
> 
> > -----Original Message-----
> > From: Philippe Mathieu-Daudé <[e-mail]>
> > Sent: Monday, February 22, 2021 5:34 PM
> > To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
> > <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> > <[e-mail]>; Vladimir Sementsov-Ogievskiy
> > <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> > <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
> > <[e-mail]>; Dr. David Alan Gilbert <[e-mail]>;
> > Thomas Huth <[e-mail]>; Stefan Hajnoczi <[e-mail]>;
> > Peter Maydell <[e-mail]>; Alistair Francis
> > <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc Michel
> > <[e-mail]>; Paolo Bonzini <[e-mail]>
> > Cc: Sai Pavan Boddu <[e-mail]>; [e-mail]; qemu-
> > [e-mail]
> > Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> > 
> > On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> > > From: Vincent Palatin <[e-mail]>
> > >
> > > Add new block device type.
> > >
> > > Signed-off-by: Vincent Palatin <[e-mail]>
> > > [SPB: Rebased over 5.1 version]
> > > Signed-off-by: Sai Pavan Boddu <[e-mail]>
> > > Signed-off-by: Joel Stanley <[e-mail]>
> > > Signed-off-by: Cédric Le Goater <[e-mail]>
> > > Reviewed-by: Alistair Francis <[e-mail]>
> > > ---
> > >  include/sysemu/blockdev.h | 1 +
> > >  blockdev.c                | 1 +
> > >  2 files changed, 2 insertions(+)
> > >
> > > diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> > > index 3b5fcda..eefae9f 100644
> > > --- a/include/sysemu/blockdev.h
> > > +++ b/include/sysemu/blockdev.h
> > > @@ -24,6 +24,7 @@ typedef enum {
> > >       */
> > >      IF_NONE = 0,
> > >      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO,
> > > IF_XEN,
> > > +    IF_EMMC,
> > >      IF_COUNT
> > >  } BlockInterfaceType;
> > >
> > > diff --git a/blockdev.c b/blockdev.c
> > > index cd438e6..390d43c 100644
> > > --- a/blockdev.c
> > > +++ b/blockdev.c
> > > @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> > >      [IF_SD] = "sd",
> > >      [IF_VIRTIO] = "virtio",
> > >      [IF_XEN] = "xen",
> > > +    [IF_EMMC] = "emmc",
> > >  };
> > 
> > We don't need to introduce support for the legacy -drive magic.
> > 
> > -device should be enough for this device, right?
> [Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we anyway need blockdev support for this, which would require us the use -drive.
> 
> Can you give some pointers, how to approach this ?

It is probably not necessary to add a new IF_ constant. Would this work:

  -drive if=none,id=emmc0,file=test.img,format=raw
  -device emmc,...,drive=emmc0

Or the more modern:

  -blockdev node-name=emmc0,driver=file,filename=test.img
  -device emmc,...,drive=emmc0

?

(The syntax might need small tweaks but is shows the general idea.)

Stefan

Cédric Le Goater Feb. 24, 2021, 1:55 p.m. UTC | #9

On 2/24/21 12:40 PM, Stefan Hajnoczi wrote:
> On Tue, Feb 23, 2021 at 05:35:20PM +0000, Sai Pavan Boddu wrote:
>> Hi Philippe,
>>
>>> -----Original Message-----
>>> From: Philippe Mathieu-Daudé <[e-mail]>
>>> Sent: Monday, February 22, 2021 5:34 PM
>>> To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
>>> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
>>> <[e-mail]>; Vladimir Sementsov-Ogievskiy
>>> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
>>> <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
>>> <[e-mail]>; Dr. David Alan Gilbert <[e-mail]>;
>>> Thomas Huth <[e-mail]>; Stefan Hajnoczi <[e-mail]>;
>>> Peter Maydell <[e-mail]>; Alistair Francis
>>> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc Michel
>>> <[e-mail]>; Paolo Bonzini <[e-mail]>
>>> Cc: Sai Pavan Boddu <[e-mail]>; [e-mail]; qemu-
>>> [e-mail]
>>> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
>>>
>>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
>>>> From: Vincent Palatin <[e-mail]>
>>>>
>>>> Add new block device type.
>>>>
>>>> Signed-off-by: Vincent Palatin <[e-mail]>
>>>> [SPB: Rebased over 5.1 version]
>>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
>>>> Signed-off-by: Joel Stanley <[e-mail]>
>>>> Signed-off-by: Cédric Le Goater <[e-mail]>
>>>> Reviewed-by: Alistair Francis <[e-mail]>
>>>> ---
>>>>  include/sysemu/blockdev.h | 1 +
>>>>  blockdev.c                | 1 +
>>>>  2 files changed, 2 insertions(+)
>>>>
>>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
>>>> index 3b5fcda..eefae9f 100644
>>>> --- a/include/sysemu/blockdev.h
>>>> +++ b/include/sysemu/blockdev.h
>>>> @@ -24,6 +24,7 @@ typedef enum {
>>>>       */
>>>>      IF_NONE = 0,
>>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO,
>>>> IF_XEN,
>>>> +    IF_EMMC,
>>>>      IF_COUNT
>>>>  } BlockInterfaceType;
>>>>
>>>> diff --git a/blockdev.c b/blockdev.c
>>>> index cd438e6..390d43c 100644
>>>> --- a/blockdev.c
>>>> +++ b/blockdev.c
>>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
>>>>      [IF_SD] = "sd",
>>>>      [IF_VIRTIO] = "virtio",
>>>>      [IF_XEN] = "xen",
>>>> +    [IF_EMMC] = "emmc",
>>>>  };
>>>
>>> We don't need to introduce support for the legacy -drive magic.
>>>
>>> -device should be enough for this device, right?
>> [Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we anyway need blockdev support for this, which would require us the use -drive.
>>
>> Can you give some pointers, how to approach this ?
> 
> It is probably not necessary to add a new IF_ constant. Would this work:
> 
>   -drive if=none,id=emmc0,file=test.img,format=raw
>   -device emmc,...,drive=emmc0
> 
> Or the more modern:
> 
>   -blockdev node-name=emmc0,driver=file,filename=test.img
>   -device emmc,...,drive=emmc0
> 
> ?
> 
> (The syntax might need small tweaks but is shows the general idea.)

Yes. This is better. 

We could have an "emmc" device inheriting from "sd-card". The "emmc" 
property would not be necessary anymore and may be, we could cleanup 
up some parts doing : 

    if (sd->emmc) { /* eMMC */
        ...
    } else {

    }

with SDCardClass handlers. the SWITCH_FUNCTION command is a good 
candidate, CMD8 also.

C.

Sai Pavan Boddu Feb. 24, 2021, 7:06 p.m. UTC | #10

Hi Stefan

> -----Original Message-----
> From: Stefan Hajnoczi <[e-mail]>
> Sent: Wednesday, February 24, 2021 5:10 PM
> To: Sai Pavan Boddu <[e-mail]>
> Cc: Philippe Mathieu-Daudé <[e-mail]>; Markus Armbruster
> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> <[e-mail]>; Vladimir Sementsov-Ogievskiy
> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent Palatin
> <[e-mail]>; Dr. David Alan Gilbert <[e-mail]>;
> Thomas Huth <[e-mail]>; Peter Maydell
> <[e-mail]>; Alistair Francis <[e-mail]>; Edgar
> Iglesias <[e-mail]>; Luc Michel <[e-mail]>; Paolo
> Bonzini <[e-mail]>; [e-mail]; qemu-
> [e-mail]
> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> 
> On Tue, Feb 23, 2021 at 05:35:20PM +0000, Sai Pavan Boddu wrote:
> > Hi Philippe,
> >
> > > -----Original Message-----
> > > From: Philippe Mathieu-Daudé <[e-mail]>
> > > Sent: Monday, February 22, 2021 5:34 PM
> > > To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
> > > <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> > > <[e-mail]>; Vladimir Sementsov-Ogievskiy
> > > <[e-mail]>; Eric Blake <[e-mail]>; Joel
> > > Stanley <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent
> > > Palatin <[e-mail]>; Dr. David Alan Gilbert
> > > <[e-mail]>; Thomas Huth <[e-mail]>; Stefan
> > > Hajnoczi <[e-mail]>; Peter Maydell
> > > <[e-mail]>; Alistair Francis
> > > <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc
> > > Michel <[e-mail]>; Paolo Bonzini
> > > <[e-mail]>
> > > Cc: Sai Pavan Boddu <[e-mail]>; [e-mail];
> > > qemu- [e-mail]
> > > Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> > >
> > > On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> > > > From: Vincent Palatin <[e-mail]>
> > > >
> > > > Add new block device type.
> > > >
> > > > Signed-off-by: Vincent Palatin <[e-mail]>
> > > > [SPB: Rebased over 5.1 version]
> > > > Signed-off-by: Sai Pavan Boddu <[e-mail]>
> > > > Signed-off-by: Joel Stanley <[e-mail]>
> > > > Signed-off-by: Cédric Le Goater <[e-mail]>
> > > > Reviewed-by: Alistair Francis <[e-mail]>
> > > > ---
> > > >  include/sysemu/blockdev.h | 1 +
> > > >  blockdev.c                | 1 +
> > > >  2 files changed, 2 insertions(+)
> > > >
> > > > diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> > > > index 3b5fcda..eefae9f 100644
> > > > --- a/include/sysemu/blockdev.h
> > > > +++ b/include/sysemu/blockdev.h
> > > > @@ -24,6 +24,7 @@ typedef enum {
> > > >       */
> > > >      IF_NONE = 0,
> > > >      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD,
> > > > IF_VIRTIO, IF_XEN,
> > > > +    IF_EMMC,
> > > >      IF_COUNT
> > > >  } BlockInterfaceType;
> > > >
> > > > diff --git a/blockdev.c b/blockdev.c index cd438e6..390d43c 100644
> > > > --- a/blockdev.c
> > > > +++ b/blockdev.c
> > > > @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> > > >      [IF_SD] = "sd",
> > > >      [IF_VIRTIO] = "virtio",
> > > >      [IF_XEN] = "xen",
> > > > +    [IF_EMMC] = "emmc",
> > > >  };
> > >
> > > We don't need to introduce support for the legacy -drive magic.
> > >
> > > -device should be enough for this device, right?
> > [Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we anyway
> need blockdev support for this, which would require us the use -drive.
> >
> > Can you give some pointers, how to approach this ?
> 
> It is probably not necessary to add a new IF_ constant. Would this work:
> 
>   -drive if=none,id=emmc0,file=test.img,format=raw
>   -device emmc,...,drive=emmc0
[Sai Pavan Boddu] Great, this works for me.
> 
> Or the more modern:
> 
>   -blockdev node-name=emmc0,driver=file,filename=test.img
>   -device emmc,...,drive=emmc0
[Sai Pavan Boddu] Thanks, I would try to follow the modern approach then!

Regards,
Sai Pavan
> 
> ?
> 
> (The syntax might need small tweaks but is shows the general idea.)
> 
> Stefan

Sai Pavan Boddu Feb. 24, 2021, 7:13 p.m. UTC | #11

Hi Cedric,


> -----Original Message-----
> From: Cédric Le Goater <[e-mail]>
> Sent: Wednesday, February 24, 2021 7:25 PM
> To: Stefan Hajnoczi <[e-mail]>; Sai Pavan Boddu
> <[e-mail]>
> Cc: Philippe Mathieu-Daudé <[e-mail]>; Markus Armbruster
> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> <[e-mail]>; Vladimir Sementsov-Ogievskiy
> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
> <[e-mail]>; Vincent Palatin <[e-mail]>; Dr. David Alan
> Gilbert <[e-mail]>; Thomas Huth <[e-mail]>; Peter
> Maydell <[e-mail]>; Alistair Francis
> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc Michel
> <[e-mail]>; Paolo Bonzini <[e-mail]>; qemu-
> [e-mail]; [e-mail]
> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> 
> On 2/24/21 12:40 PM, Stefan Hajnoczi wrote:
> > On Tue, Feb 23, 2021 at 05:35:20PM +0000, Sai Pavan Boddu wrote:
> >> Hi Philippe,
> >>
> >>> -----Original Message-----
> >>> From: Philippe Mathieu-Daudé <[e-mail]>
> >>> Sent: Monday, February 22, 2021 5:34 PM
> >>> To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
> >>> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
> >>> <[e-mail]>; Vladimir Sementsov-Ogievskiy
> >>> <[e-mail]>; Eric Blake <[e-mail]>; Joel
> >>> Stanley <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent
> >>> Palatin <[e-mail]>; Dr. David Alan Gilbert
> >>> <[e-mail]>; Thomas Huth <[e-mail]>; Stefan
> >>> Hajnoczi <[e-mail]>; Peter Maydell
> >>> <[e-mail]>; Alistair Francis
> >>> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc
> >>> Michel <[e-mail]>; Paolo Bonzini
> >>> <[e-mail]>
> >>> Cc: Sai Pavan Boddu <[e-mail]>; [e-mail];
> >>> qemu- [e-mail]
> >>> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
> >>>
> >>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
> >>>> From: Vincent Palatin <[e-mail]>
> >>>>
> >>>> Add new block device type.
> >>>>
> >>>> Signed-off-by: Vincent Palatin <[e-mail]>
> >>>> [SPB: Rebased over 5.1 version]
> >>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
> >>>> Signed-off-by: Joel Stanley <[e-mail]>
> >>>> Signed-off-by: Cédric Le Goater <[e-mail]>
> >>>> Reviewed-by: Alistair Francis <[e-mail]>
> >>>> ---
> >>>>  include/sysemu/blockdev.h | 1 +
> >>>>  blockdev.c                | 1 +
> >>>>  2 files changed, 2 insertions(+)
> >>>>
> >>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
> >>>> index 3b5fcda..eefae9f 100644
> >>>> --- a/include/sysemu/blockdev.h
> >>>> +++ b/include/sysemu/blockdev.h
> >>>> @@ -24,6 +24,7 @@ typedef enum {
> >>>>       */
> >>>>      IF_NONE = 0,
> >>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD,
> >>>> IF_VIRTIO, IF_XEN,
> >>>> +    IF_EMMC,
> >>>>      IF_COUNT
> >>>>  } BlockInterfaceType;
> >>>>
> >>>> diff --git a/blockdev.c b/blockdev.c index cd438e6..390d43c 100644
> >>>> --- a/blockdev.c
> >>>> +++ b/blockdev.c
> >>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
> >>>>      [IF_SD] = "sd",
> >>>>      [IF_VIRTIO] = "virtio",
> >>>>      [IF_XEN] = "xen",
> >>>> +    [IF_EMMC] = "emmc",
> >>>>  };
> >>>
> >>> We don't need to introduce support for the legacy -drive magic.
> >>>
> >>> -device should be enough for this device, right?
> >> [Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we
> anyway need blockdev support for this, which would require us the use -drive.
> >>
> >> Can you give some pointers, how to approach this ?
> >
> > It is probably not necessary to add a new IF_ constant. Would this work:
> >
> >   -drive if=none,id=emmc0,file=test.img,format=raw
> >   -device emmc,...,drive=emmc0
> >
> > Or the more modern:
> >
> >   -blockdev node-name=emmc0,driver=file,filename=test.img
> >   -device emmc,...,drive=emmc0
> >
> > ?
> >
> > (The syntax might need small tweaks but is shows the general idea.)
> 
> Yes. This is better.
> 
> We could have an "emmc" device inheriting from "sd-card". The "emmc"
> property would not be necessary anymore and may be, we could cleanup up
> some parts doing :
> 
>     if (sd->emmc) { /* eMMC */
>         ...
>     } else {
> 
>     }
> 
> with SDCardClass handlers. the SWITCH_FUNCTION command is a good
> candidate, CMD8 also.
[Sai Pavan Boddu] Nice, this approach looks clean.
But we still may be depending on emmc property. Not sure!

I would get back with v3, your review on those patches would be great.

Thanks & Regards,
Sai Pavan
> 
> C.

Cédric Le Goater Feb. 25, 2021, 2:43 p.m. UTC | #12

On 2/24/21 8:13 PM, Sai Pavan Boddu wrote:
> Hi Cedric,
> 
> 
>> -----Original Message-----
>> From: Cédric Le Goater <[e-mail]>
>> Sent: Wednesday, February 24, 2021 7:25 PM
>> To: Stefan Hajnoczi <[e-mail]>; Sai Pavan Boddu
>> <[e-mail]>
>> Cc: Philippe Mathieu-Daudé <[e-mail]>; Markus Armbruster
>> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
>> <[e-mail]>; Vladimir Sementsov-Ogievskiy
>> <[e-mail]>; Eric Blake <[e-mail]>; Joel Stanley
>> <[e-mail]>; Vincent Palatin <[e-mail]>; Dr. David Alan
>> Gilbert <[e-mail]>; Thomas Huth <[e-mail]>; Peter
>> Maydell <[e-mail]>; Alistair Francis
>> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc Michel
>> <[e-mail]>; Paolo Bonzini <[e-mail]>; qemu-
>> [e-mail]; [e-mail]
>> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
>>
>> On 2/24/21 12:40 PM, Stefan Hajnoczi wrote:
>>> On Tue, Feb 23, 2021 at 05:35:20PM +0000, Sai Pavan Boddu wrote:
>>>> Hi Philippe,
>>>>
>>>>> -----Original Message-----
>>>>> From: Philippe Mathieu-Daudé <[e-mail]>
>>>>> Sent: Monday, February 22, 2021 5:34 PM
>>>>> To: Sai Pavan Boddu <[e-mail]>; Markus Armbruster
>>>>> <[e-mail]>; Kevin Wolf <[e-mail]>; Max Reitz
>>>>> <[e-mail]>; Vladimir Sementsov-Ogievskiy
>>>>> <[e-mail]>; Eric Blake <[e-mail]>; Joel
>>>>> Stanley <[e-mail]>; Cédric Le Goater <[e-mail]>; Vincent
>>>>> Palatin <[e-mail]>; Dr. David Alan Gilbert
>>>>> <[e-mail]>; Thomas Huth <[e-mail]>; Stefan
>>>>> Hajnoczi <[e-mail]>; Peter Maydell
>>>>> <[e-mail]>; Alistair Francis
>>>>> <[e-mail]>; Edgar Iglesias <[e-mail]>; Luc
>>>>> Michel <[e-mail]>; Paolo Bonzini
>>>>> <[e-mail]>
>>>>> Cc: Sai Pavan Boddu <[e-mail]>; [e-mail];
>>>>> qemu- [e-mail]
>>>>> Subject: Re: [PATCH v2 01/22] block: add eMMC block device type
>>>>>
>>>>> On 2/22/21 9:20 AM, Sai Pavan Boddu wrote:
>>>>>> From: Vincent Palatin <[e-mail]>
>>>>>>
>>>>>> Add new block device type.
>>>>>>
>>>>>> Signed-off-by: Vincent Palatin <[e-mail]>
>>>>>> [SPB: Rebased over 5.1 version]
>>>>>> Signed-off-by: Sai Pavan Boddu <[e-mail]>
>>>>>> Signed-off-by: Joel Stanley <[e-mail]>
>>>>>> Signed-off-by: Cédric Le Goater <[e-mail]>
>>>>>> Reviewed-by: Alistair Francis <[e-mail]>
>>>>>> ---
>>>>>>  include/sysemu/blockdev.h | 1 +
>>>>>>  blockdev.c                | 1 +
>>>>>>  2 files changed, 2 insertions(+)
>>>>>>
>>>>>> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
>>>>>> index 3b5fcda..eefae9f 100644
>>>>>> --- a/include/sysemu/blockdev.h
>>>>>> +++ b/include/sysemu/blockdev.h
>>>>>> @@ -24,6 +24,7 @@ typedef enum {
>>>>>>       */
>>>>>>      IF_NONE = 0,
>>>>>>      IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD,
>>>>>> IF_VIRTIO, IF_XEN,
>>>>>> +    IF_EMMC,
>>>>>>      IF_COUNT
>>>>>>  } BlockInterfaceType;
>>>>>>
>>>>>> diff --git a/blockdev.c b/blockdev.c index cd438e6..390d43c 100644
>>>>>> --- a/blockdev.c
>>>>>> +++ b/blockdev.c
>>>>>> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = {
>>>>>>      [IF_SD] = "sd",
>>>>>>      [IF_VIRTIO] = "virtio",
>>>>>>      [IF_XEN] = "xen",
>>>>>> +    [IF_EMMC] = "emmc",
>>>>>>  };
>>>>>
>>>>> We don't need to introduce support for the legacy -drive magic.
>>>>>
>>>>> -device should be enough for this device, right?
>>>> [Sai Pavan Boddu] I was seeing to use -device for emmc. But I see we
>> anyway need blockdev support for this, which would require us the use -drive.
>>>>
>>>> Can you give some pointers, how to approach this ?
>>>
>>> It is probably not necessary to add a new IF_ constant. Would this work:
>>>
>>>   -drive if=none,id=emmc0,file=test.img,format=raw
>>>   -device emmc,...,drive=emmc0
>>>
>>> Or the more modern:
>>>
>>>   -blockdev node-name=emmc0,driver=file,filename=test.img
>>>   -device emmc,...,drive=emmc0
>>>
>>> ?
>>>
>>> (The syntax might need small tweaks but is shows the general idea.)
>>
>> Yes. This is better.
>>
>> We could have an "emmc" device inheriting from "sd-card". The "emmc"
>> property would not be necessary anymore and may be, we could cleanup up
>> some parts doing :
>>
>>     if (sd->emmc) { /* eMMC */
>>         ...
>>     } else {
>>
>>     }
>>
>> with SDCardClass handlers. the SWITCH_FUNCTION command is a good
>> candidate, CMD8 also.
> [Sai Pavan Boddu] Nice, this approach looks clean.
> But we still may be depending on emmc property. Not sure!

It could be a SDCardClass attribute instead since it's a constant for
the emmc device.

C. 

> 
> I would get back with v3, your review on those patches would be great.
> 
> Thanks & Regards,
> Sai Pavan
>>
>> C.

Patch

diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h
index 3b5fcda..eefae9f 100644
--- a/include/sysemu/blockdev.h
+++ b/include/sysemu/blockdev.h
@@ -24,6 +24,7 @@  typedef enum {
      */
     IF_NONE = 0,
     IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN,
+    IF_EMMC,
     IF_COUNT
 } BlockInterfaceType;
 
diff --git a/blockdev.c b/blockdev.c
index cd438e6..390d43c 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -83,6 +83,7 @@  static const char *const if_name[IF_COUNT] = {
     [IF_SD] = "sd",
     [IF_VIRTIO] = "virtio",
     [IF_XEN] = "xen",
+    [IF_EMMC] = "emmc",
 };
 
 static int if_max_devs[IF_COUNT] = {