Load a png on IOS causes an "Attempting to JIT compile method while running in aot-only mode" #776

Activity

1. vicfergar commented on Nov 14, 2018

   

   vicfergar

   on Nov 14, 2018

   Author

   More actions

   This is related with #383 and #762
2. antonfirsov commented on Nov 22, 2018

   

   antonfirsov

   on Nov 22, 2018

   Member

   More actions

   @vicfergar I wonder how should we extend the AOT seeding utility introduced in #767 to make this work.

   We lack both the knowledge and the time to investigate this, so any hints from the community would be very helpful.

   👍React with 👍1Reacted by James Jackson-South
3. JimBobSquarePants

   added

   platform:ios

   on Nov 23, 2018
4. JimBobSquarePants

   closed this as completedon Jan 28, 2019
5. davilovick commented on Jan 31, 2019

   

   davilovick

   on Jan 31, 2019

   More actions

   I've just tried to load a PNG in iOS with the latest Release (1.0.0-beta0006), and the issue is still happening.

   The stacktrace is exatly the same that @vicfergar has described.
6. James-Jackson-South commented on Jan 31, 2019

   

   James-Jackson-South

   on Jan 31, 2019

   More actions

   Did you use the seed utility method?
7. davilovick commented on Feb 4, 2019

   

   davilovick

   on Feb 4, 2019

   More actions

   Sorry for the delay.

   Yes, I've used the Seed method. Here is an example of how I've used this to reproduce the problem:

           private static void SeedAOT()
           {
               AotCompilerTools.Seed<Rgba32, Rgb24, Bgr24>();
               AotCompilerTools.Seed<Rgb48, Bgra32, Gray8>();
           }
   
           public ViewController (IntPtr handle) : base (handle)
           {
               SeedAOT();
           }
   
8. leonluc-dev commented on Jul 11, 2019

   

   leonluc-dev

   on Jul 11, 2019

   Last edited by leonluc-dev

   More actions

   I'm encountering a similar issue in my iOS app.
   I use the following code in a .NET Standard project (referenced by an Xamarin.iOS and Xamarin.Android app) to load a PNG or JPEG image from a byte array.

   var image = Image.Load(imageBytes)
   

   This works fine for JPEG images (and on the Android app), but as soon as I try to load PNG image data on iOS the following exception occurs:

   Unhandled managed exception: Attempting to JIT compile method 'SixLabors.ImageSharp.Formats.Png.PngScanlineProcessor:ProcessRgbaScanline<SixLabors.ImageSharp.PixelFormats.Rgba32> (SixLabors.ImageSharp.Configuration,SixLabors.ImageSharp.Formats.Png.PngHeader&,System.ReadOnlySpan`1<byte>,System.Span`1<SixLabors.ImageSharp.PixelFormats.Rgba32>,int,int)' while running in aot-only mode. See https://docs.microsoft.com/xamarin/ios/internals/limitations for more information.
    (System.ExecutionEngineException)
     at SixLabors.ImageSharp.Formats.Png.PngDecoderCore.ProcessDefilteredScanline[TPixel] (System.ReadOnlySpan`1[T] defilteredScanline, SixLabors.ImageSharp.ImageFrame`1[TPixel] pixels, SixLabors.ImageSharp.Formats.Png.PngMetaData pngMetaData) [0x00135] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Formats.Png.PngDecoderCore.DecodePixelData[TPixel] (System.IO.Stream compressedStream, SixLabors.ImageSharp.ImageFrame`1[TPixel] image, SixLabor
   s.ImageSharp.Formats.Png.PngMetaData pngMetaData) [0x000e5] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Formats.Png.PngDecoderCore.ReadScanlines[TPixel] (System.IO.Stream dataStream, SixLabors.ImageSharp.ImageFrame`1[TPixel] image, SixLabors.ImageSharp.Formats.Png.PngMetaData pngMetaData) [0x00018] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Formats.Png.PngDecoderCore.Decode[TPixel] (System.IO.Stream stream) [0x00171] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Formats.Png.PngDecoder.Decode[TPixel] (SixLabors.ImageSharp.Configuration configuration, System.IO.Stream stream) [0x00008] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image.Decode[TPixel] (System.IO.Stream stream, SixLabors.ImageSharp.Configuration config) [0x00015] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image+<>c__DisplayClass55_0`1[TPixel].<Load>b__0 (System.IO.Stream s) [0x00000] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLa
   bors.ImageSharp.Image.WithSeekableStream[T] (SixLabors.ImageSharp.Configuration config, System.IO.Stream stream, System.Func`2[T,TResult] action) [0x0002b] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image.Load[TPixel] (SixLabors.ImageSharp.Configuration config, System.IO.Stream stream, SixLabors.ImageSharp.Formats.IImageFormat& format) [0x00022] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image.Load[TPixel] (SixLabors.ImageSharp.Configuration config, System.IO.Stream stream) [0x00000] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image.Load[TPixel] (SixLabors.ImageSharp.Configuration config, System.Byte[] data) [0x00007] in <7430a37e6bca4100b1763856ff8f7867>:0 
     at SixLabors.ImageSharp.Image.Load (System.Byte[] data) [0x00005] in <7430a37e6bca4100b1763856ff8f7867>:0 
   

   This exception seems close to identical to the one described in this issue. The major difference being the method name ProcessRgbaScanline vs ProcessRgbScanline

   I've already tried using the AotCompilerTools to seed the Rgba32 pixel type my code uses, but this did not solve the issue.

   private static void NeverCallThisMethod()
   {
       AotCompilerTools.Seed<Rgba32>();
       throw new Exception("Don't call AotCompilerTools.NeverCallThisMethod(), it only exists to preseed the AOT compiler.");
           }
   
9. leonluc-dev commented on Jul 12, 2019

   

   leonluc-dev

   on Jul 12, 2019

   More actions

   Looking into it, it seems the following code is not properly seeded for some reason (possibly due to the inferred generics, but I'm not 100% sure):

   ImageSharp/src/ImageSharp/Formats/Png/PngDecoderCore.cs

   Lines 649 to 705 in 48bac7b

   	switch (this.pngColorType)
   	{
   	case PngColorType.Grayscale:
   	PngScanlineProcessor.ProcessGrayscaleScanline(
   	this.header,
   	scanlineSpan,
   	rowSpan,
   	pngMetadata.HasTrans,
   	pngMetadata.TransparentGray16.GetValueOrDefault(),
   	pngMetadata.TransparentGray8.GetValueOrDefault());
   	break;
   	case PngColorType.GrayscaleWithAlpha:
   	PngScanlineProcessor.ProcessGrayscaleWithAlphaScanline(
   	this.header,
   	scanlineSpan,
   	rowSpan,
   	this.bytesPerPixel,
   	this.bytesPerSample);
   	break;
   	case PngColorType.Palette:
   	PngScanlineProcessor.ProcessPaletteScanline(
   	this.header,
   	scanlineSpan,
   	rowSpan,
   	this.palette,
   	this.paletteAlpha);
   	break;
   	case PngColorType.Rgb:
   	PngScanlineProcessor.ProcessRgbScanline(
   	this.configuration,
   	this.header,
   	scanlineSpan,
   	rowSpan,
   	this.bytesPerPixel,
   	this.bytesPerSample,
   	pngMetadata.HasTrans,
   	pngMetadata.TransparentRgb48.GetValueOrDefault(),
   	pngMetadata.TransparentRgb24.GetValueOrDefault());
   	break;
   	case PngColorType.RgbWithAlpha:
   	PngScanlineProcessor.ProcessRgbaScanline(
   	this.configuration,
   	this.header,
   	scanlineSpan,
   	rowSpan,
   	this.bytesPerPixel,
   	this.bytesPerSample);
   	break;

   Since the code is marked assembly internal I cannot directly seed these methods without importing/modifying the source code.

   👍React with 👍1Reacted by James Jackson-South
10. antonfirsov commented on Jul 14, 2019

    

    antonfirsov

    on Jul 14, 2019

    Last edited by antonfirsov

    Member

    More actions

    @gameleon-dev thans for the investigations you made so far! 👍
    I have just opened #946 to address this issue at a higher level. Let's continue the discussion there!

    👍React with 👍1Reacted by James Jackson-South